Commit 273946a5 authored Apr 01, 2012 by Dan Carpenter Committed by Artem Bityutskiy May 03, 2012

UBIFS: remove douple initialization in change_category()

"heap" is initialized twice. I removed the first one, because it makes
Smatch complain that we use "new_cat" as an offset before checking it.

This doesn't change how the code works, it's just a cleanup.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>

parent 69964ea4

fs/ubifs/lprops.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -447,7 +447,7 @@ static void change_category(struct ubifs_info c, struct ubifs_lprops lprops)
		int new_cat = ubifs_categorize_lprops(c, lprops);

		if (old_cat == new_cat) {
		struct ubifs_lpt_heap *heap = &c->lpt_heap[new_cat - 1];
		struct ubifs_lpt_heap *heap;

		/* lprops on a heap now must be moved up or down */
		if (new_cat < 1 \|\| new_cat > LPROPS_HEAP_CNT)