| From 6a66356c4993294ea7de22483c224acc0d44511b Mon Sep 17 00:00:00 2001 |
| From: Allison Henderson <achender@linux.vnet.ibm.com> |
| Date: Sun, 15 May 2011 00:19:41 -0400 |
| Subject: [PATCH] ext4: don't dereference null pointer when make_indexed_dir() |
| fails |
| |
| commit 6976a6f2acde2b0443cd64f1d08af90630e4ce81 upstream. |
| |
| Fix for a null pointer bug found while running punch hole tests |
| |
| Signed-off-by: Allison Henderson <achender@us.ibm.com> |
| Signed-off-by: "Theodore Ts'o" <tytso@mit.edu> |
| Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com> |
| |
| diff --git a/fs/ext4/namei.c b/fs/ext4/namei.c |
| index caa3c77..41198b3 100644 |
| --- a/fs/ext4/namei.c |
| +++ b/fs/ext4/namei.c |
| @@ -1451,6 +1451,10 @@ static int make_indexed_dir(handle_t *handle, struct dentry *dentry, |
| frame->at = entries; |
| frame->bh = bh; |
| bh = bh2; |
| + |
| + ext4_handle_dirty_metadata(handle, dir, frame->bh); |
| + ext4_handle_dirty_metadata(handle, dir, bh); |
| + |
| de = do_split(handle,dir, &bh, frame, &hinfo, &retval); |
| if (!de) { |
| /* |
| @@ -1459,8 +1463,6 @@ static int make_indexed_dir(handle_t *handle, struct dentry *dentry, |
| * with corrupted filesystem. |
| */ |
| ext4_mark_inode_dirty(handle, dir); |
| - ext4_handle_dirty_metadata(handle, dir, frame->bh); |
| - ext4_handle_dirty_metadata(handle, dir, bh); |
| dx_release(frames); |
| return retval; |
| } |
| -- |
| 1.8.1.2 |
| |