We should put the inode page when error was occurred.
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
} else if (page->index == 0) {
err = f2fs_convert_inline_page(&dn, page);
if (err)
- goto unlock_fail;
+ goto put_fail;
} else {
struct page *p = grab_cache_page(inode->i_mapping, 0);
if (!p) {
err = -ENOMEM;
- goto unlock_fail;
+ goto put_fail;
}
err = f2fs_convert_inline_page(&dn, p);
f2fs_put_page(p, 1);
if (err)
- goto unlock_fail;
+ goto put_fail;
}
}
err = f2fs_reserve_block(&dn, index);
if (err)
- goto unlock_fail;
+ goto put_fail;
put_next:
f2fs_put_dnode(&dn);
f2fs_unlock_op(sbi);
clear_cold_data(page);
return 0;
+put_fail:
+ f2fs_put_dnode(&dn);
unlock_fail:
f2fs_unlock_op(sbi);
f2fs_put_page(page, 1);