diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2013-08-15 01:54:56 -0400 |
|---|---|---|
| committer | Jaegeuk Kim <jaegeuk.kim@samsung.com> | 2013-08-18 20:42:29 -0400 |
| commit | e27dae4d663762da2020e93885be2219f0608ec6 (patch) | |
| tree | ee60baa9daf0636ce7daf8c1562088696145d8f0 /fs/f2fs/recovery.c | |
| parent | 479bd73ac425ff117efeea051077b4277baab52e (diff) | |
f2fs: alloc_page() doesn't return an ERR_PTR
alloc_page() returns a NULL on failure, it never returns an ERR_PTR.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Jaegeuk Kim <jaegeuk.kim@samsung.com>
Diffstat (limited to 'fs/f2fs/recovery.c')
| -rw-r--r-- | fs/f2fs/recovery.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/fs/f2fs/recovery.c b/fs/f2fs/recovery.c index c6908b5d9d84..fa493bb64167 100644 --- a/fs/f2fs/recovery.c +++ b/fs/f2fs/recovery.c | |||
| @@ -129,8 +129,8 @@ static int find_fsync_dnodes(struct f2fs_sb_info *sbi, struct list_head *head) | |||
| 129 | 129 | ||
| 130 | /* read node page */ | 130 | /* read node page */ |
| 131 | page = alloc_page(GFP_F2FS_ZERO); | 131 | page = alloc_page(GFP_F2FS_ZERO); |
| 132 | if (IS_ERR(page)) | 132 | if (!page) |
| 133 | return PTR_ERR(page); | 133 | return -ENOMEM; |
| 134 | lock_page(page); | 134 | lock_page(page); |
| 135 | 135 | ||
| 136 | while (1) { | 136 | while (1) { |
| @@ -367,7 +367,7 @@ static int recover_data(struct f2fs_sb_info *sbi, | |||
| 367 | 367 | ||
| 368 | /* read node page */ | 368 | /* read node page */ |
| 369 | page = alloc_page(GFP_NOFS | __GFP_ZERO); | 369 | page = alloc_page(GFP_NOFS | __GFP_ZERO); |
| 370 | if (IS_ERR(page)) | 370 | if (!page) |
| 371 | return -ENOMEM; | 371 | return -ENOMEM; |
| 372 | 372 | ||
| 373 | lock_page(page); | 373 | lock_page(page); |