On Sat, Jan 05, 2013 at 03:43:00PM +0800, Guo Chao wrote: > If checksum fails, we should also release the buffer > read from previous iteration. > > Cc: Darrick J. Wong <darrick.wong@xxxxxxxxxx> > Signed-off-by: Guo Chao <yan@xxxxxxxxxxxxxxxxxx> > --- > fs/ext4/namei.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/fs/ext4/namei.c b/fs/ext4/namei.c > index 843e29f..e249a47 100644 > --- a/fs/ext4/namei.c > +++ b/fs/ext4/namei.c > @@ -722,7 +722,7 @@ dx_probe(const struct qstr *d_name, struct inode *dir, > ext4_warning(dir->i_sb, "Node failed checksum"); > brelse(bh); > *err = ERR_BAD_DX_DIR; > - goto fail; > + goto fail2; Good catch! Acked-by: Darrick J. Wong <darrick.wong@xxxxxxxxxx> --D > } > set_buffer_verified(bh); > > -- > 1.7.9.5 > -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
