diff options
| author | Younger Liu <[email protected]> | 2014-01-23 15:53:47 -0800 |
|---|---|---|
| committer | Linus Torvalds <[email protected]> | 2014-01-23 16:36:54 -0800 |
| commit | 2252b62a56601c9e31396da230b4ce792f167fb4 (patch) | |
| tree | 02a0cf2f7ba9a1367d8c13752b67d3819149b921 | |
| parent | a3b25d9b774fbda2e6add28cf792941fd98fa999 (diff) | |
logfs: check for the return value after calling find_or_create_page()
In get_mapping_page(), after calling find_or_create_page(), the return
value should be checked.
This patch has been provided:
http://www.spinics.net/lists/linux-fsdevel/msg66948.html but not been
applied now.
Signed-off-by: Younger Liu <[email protected]>
Cc: Younger Liu <[email protected]>
Cc: Vyacheslav Dubeyko <[email protected]>
Reviewed-by: Prasad Joshi <[email protected]>
Cc: Jörn Engel <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
| -rw-r--r-- | fs/logfs/segment.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/fs/logfs/segment.c b/fs/logfs/segment.c index d448a777166b..7f9b096d8d57 100644 --- a/fs/logfs/segment.c +++ b/fs/logfs/segment.c @@ -62,7 +62,8 @@ static struct page *get_mapping_page(struct super_block *sb, pgoff_t index, page = read_cache_page(mapping, index, filler, sb); else { page = find_or_create_page(mapping, index, GFP_NOFS); - unlock_page(page); + if (page) + unlock_page(page); } return page; } |