UBIFS: remove douple initialization in change_category()
authorDan Carpenter <dan.carpenter@oracle.com>
Sun, 1 Apr 2012 15:57:59 +0000 (18:57 +0300)
committerArtem Bityutskiy <artem.bityutskiy@linux.intel.com>
Thu, 3 May 2012 11:11:11 +0000 (14:11 +0300)
"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>
fs/ubifs/lprops.c

index f8a181e647ccfb4bb12d08ce3dac798e6bc7822c..b5c95a69d95fcea5a4ff71d3545529da8ffb3d0d 100644 (file)
@@ -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)