release-bh-in-make_indexed_dir

   1 ext4: release bh in make_indexed_dir
   2
   3 From: gmail <yngsion@gmail.com>
   4
   5 The commit 6050d47adcad: "ext4: bail out from make_indexed_dir() on
   6 first error" could end up leaking bh2 in the error path.
   7
   8 [ Also avoid renaming bh2 to bh, which just confuses things --tytso ]
   9
  10 Signed-off-by: yangsheng <yngsion@gmail.com>
  11 Signed-off-by: Theodore Ts'o <tytso@mit.edu>
  12 ---
  13  fs/ext4/namei.c | 14 ++++++--------
  14  1 file changed, 6 insertions(+), 8 deletions(-)
  15
  16 diff --git a/fs/ext4/namei.c b/fs/ext4/namei.c
  17 index 2243ae2..c344b81 100644
  18 --- a/fs/ext4/namei.c
  19 +++ b/fs/ext4/namei.c
  20 @@ -2044,33 +2044,31 @@ static int make_indexed_dir(handle_t *handle, struct ext4_filename *fname,
  21         frame->entries = entries;
  22         frame->at = entries;
  23         frame->bh = bh;
  24 -       bh = bh2;
  25
  26         retval = ext4_handle_dirty_dx_node(handle, dir, frame->bh);
  27         if (retval)
  28                 goto out_frames;
  29 -       retval = ext4_handle_dirty_dirent_node(handle, dir, bh);
  30 +       retval = ext4_handle_dirty_dirent_node(handle, dir, bh2);
  31         if (retval)
  32                 goto out_frames;
  33
  34 -       de = do_split(handle,dir, &bh, frame, &fname->hinfo);
  35 +       de = do_split(handle,dir, &bh2, frame, &fname->hinfo);
  36         if (IS_ERR(de)) {
  37                 retval = PTR_ERR(de);
  38                 goto out_frames;
  39         }
  40 -       dx_release(frames);
  41
  42 -       retval = add_dirent_to_buf(handle, fname, dir, inode, de, bh);
  43 -       brelse(bh);
  44 -       return retval;
  45 +       retval = add_dirent_to_buf(handle, fname, dir, inode, de, bh2);
  46  out_frames:
  47         /*
  48          * Even if the block split failed, we have to properly write
  49          * out all the changes we did so far. Otherwise we can end up
  50          * with corrupted filesystem.
  51          */
  52 -       ext4_mark_inode_dirty(handle, dir);
  53 +       if (retval)
  54 +               ext4_mark_inode_dirty(handle, dir);
  55         dx_release(frames);
  56 +       brelse(bh2);
  57         return retval;
  58  }
  59