On Sat, Jan 19, 2013 at 08:27:45AM -0700, Cong Ding wrote: > The bug happens when rb_node == NULL. It causes variable node to be NULL and > then the NULL pointer is dereferenced this line: > BUG_ON((struct btrfs_root *)node->data != root); > > Based on my analysis, function tree_search should not return NULL to variable > rb_node in this case (otherwise here has to be something unknown thing wrong), > so I replace "if (rb_node)" with UG_ON(!rb_node). > > Signed-off-by: Cong Ding <ding...@gmail.com>
I don't want to add more BUG_ON()'s, just return an error. Josef -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/