korg/linux
0
0
Fork 0
mirror of https://mirrors.bfsu.edu.cn/git/linux.git synced 2024-11-11 12:28:41 +08:00
Code Issues Packages Projects Releases Wiki Activity

FS-Cache: Remove unneeded null checks

Browse Source 
fscache_write_op() makes unnecessary checks of the page variable to see if it
is NULL.  It can't be NULL at those points as the kernel would already have
crashed a little higher up where we examined page->index.

Furthermore, unless radix_tree_gang_lookup_tag() can return 1 but no page, a
NULL pointer crash should not be encountered there as we can only get there if
r_t_g_l_t() returned 1.

Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: David Howells <dhowells@redhat.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
This commit is contained in:
Dan Carpenter 2010-06-01 20:58:22 +01:00 committed by Linus Torvalds
parent 0a6531ebea
commit 08a66859e6
1 changed files with 16 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

36
fs/fscache/page.c
View File

@ -710,30 +710,26 @@ static void fscache_write_op(struct fscache_operation *_op)
goto superseded; goto superseded;
} }
if (page) { radix_tree_tag_set(&cookie->stores, page->index,
radix_tree_tag_set(&cookie->stores, page->index, FSCACHE_COOKIE_STORING_TAG);
FSCACHE_COOKIE_STORING_TAG); radix_tree_tag_clear(&cookie->stores, page->index,
radix_tree_tag_clear(&cookie->stores, page->index, FSCACHE_COOKIE_PENDING_TAG);
FSCACHE_COOKIE_PENDING_TAG);
}
spin_unlock(&cookie->stores_lock); spin_unlock(&cookie->stores_lock);
spin_unlock(&object->lock); spin_unlock(&object->lock);
if (page) { fscache_set_op_state(&op->op, "Store");
fscache_set_op_state(&op->op, "Store"); fscache_stat(&fscache_n_store_pages);
fscache_stat(&fscache_n_store_pages); fscache_stat(&fscache_n_cop_write_page);
fscache_stat(&fscache_n_cop_write_page); ret = object->cache->ops->write_page(op, page);
ret = object->cache->ops->write_page(op, page); fscache_stat_d(&fscache_n_cop_write_page);
fscache_stat_d(&fscache_n_cop_write_page); fscache_set_op_state(&op->op, "EndWrite");
fscache_set_op_state(&op->op, "EndWrite"); fscache_end_page_write(object, page);
fscache_end_page_write(object, page); if (ret < 0) {
if (ret < 0) { fscache_set_op_state(&op->op, "Abort");
fscache_set_op_state(&op->op, "Abort"); fscache_abort_object(object);
fscache_abort_object(object); } else {
} else { fscache_enqueue_operation(&op->op);
fscache_enqueue_operation(&op->op);
}
} }
_leave(""); _leave("");