From mboxrd@z Thu Jan 1 00:00:00 1970 From: Theodore Ts'o Subject: Re: [PATCH 01/18] e2fsck: reserve blocks for root/lost+found directory repair Date: Sat, 26 Jul 2014 15:47:03 -0400 Message-ID: <20140726194703.GI6725@thunk.org> References: <20140726003339.28334.54447.stgit@birch.djwong.org> <20140726003345.28334.14376.stgit@birch.djwong.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: linux-ext4@vger.kernel.org To: "Darrick J. Wong" Return-path: Received: from imap.thunk.org ([74.207.234.97]:58523 "EHLO imap.thunk.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755451AbaGZTrG (ORCPT ); Sat, 26 Jul 2014 15:47:06 -0400 Content-Disposition: inline In-Reply-To: <20140726003345.28334.14376.stgit@birch.djwong.org> Sender: linux-ext4-owner@vger.kernel.org List-ID: On Fri, Jul 25, 2014 at 05:33:45PM -0700, Darrick J. Wong wrote: > +static void reserve_block_for_lnf_repair(e2fsck_t ctx) > +{ > + blk64_t blk = 0; > + errcode_t err; > + ext2_filsys fs = ctx->fs; > + const char *name = "lost+found"; > + ext2_ino_t ino; > + > + ctx->lnf_repair_block = 0; > + if (!ext2fs_lookup(fs, EXT2_ROOT_INO, name, sizeof(name)-1, 0, &ino)) > + return; Let me guess... this originally read: const char name[] = "lost+found"; But you changed it without rerunning the regression tests. :-( Another reason why there is no such thing as not running the regression tests too many times, even after the most trivial changes. I'll fix this up and commit it, with the following comment added: [ Fixed up an obvious C trap: const char * and const char [] are not the same thing when you are taking the size of the parameter. People, run your regression tests! Like spinache, it's good for you. :-) -- tytso ] - Ted