f2fs: handle errors correctly during f2fs_reserve_block
authorJaegeuk Kim <jaegeuk.kim@samsung.com>
Fri, 27 Dec 2013 08:04:17 +0000 (17:04 +0900)
committerJaegeuk Kim <jaegeuk.kim@samsung.com>
Mon, 6 Jan 2014 07:42:21 +0000 (16:42 +0900)
commita8865372a8414298982e07f4ac8d6dc0ab1e0a3d
treed854711c56be4c135701610710c0fda95a297d13
parent1e1bb4baf10be371f72150e2801d97a04d40b3b9
f2fs: handle errors correctly during f2fs_reserve_block

The get_dnode_of_data nullifies inode and node page when error is occurred.

There are two cases that passes inode page into get_dnode_of_data().

1. make_empty_dir()
    -> get_new_data_page()
      -> f2fs_reserve_block(ipage)
-> get_dnode_of_data()

2. f2fs_convert_inline_data()
    -> __f2fs_convert_inline_data()
      -> f2fs_reserve_block(ipage)
-> get_dnode_of_data()

This patch adds correct error handling codes when get_dnode_of_data() returns
an error.

At first, f2fs_reserve_block() calls f2fs_put_dnode() whenever reserve_new_block
returns an error.
So, the rule of f2fs_reserve_block() is to nullify inode page when there is any
error internally.

Finally, two callers of f2fs_reserve_block() should call f2fs_put_dnode()
appropriately if they got an error since successful f2fs_reserve_block().

Signed-off-by: Jaegeuk Kim <jaegeuk.kim@samsung.com>
fs/f2fs/data.c
fs/f2fs/dir.c
fs/f2fs/inline.c