summaryrefslogtreecommitdiffstats
path: root/fs/ubifs/lprops.c
diff options
context:
space:
mode:
authorDan Carpenter <dan.carpenter@oracle.com>2012-04-01 18:57:59 +0300
committerArtem Bityutskiy <artem.bityutskiy@linux.intel.com>2012-05-03 14:11:11 +0300
commit273946a5c507ff5bce12349143470e8629bbe0a7 (patch)
treec060ab6ab237d8f4f24f80d1242384f71ca51e34 /fs/ubifs/lprops.c
parent69964ea4c7b68c9399f7977aa5b9aa6539a6a98a (diff)
downloadop-kernel-dev-273946a5c507ff5bce12349143470e8629bbe0a7.zip
op-kernel-dev-273946a5c507ff5bce12349143470e8629bbe0a7.tar.gz
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>
Diffstat (limited to 'fs/ubifs/lprops.c')
-rw-r--r--fs/ubifs/lprops.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/ubifs/lprops.c b/fs/ubifs/lprops.c
index f8a181e..b5c95a6 100644
--- a/fs/ubifs/lprops.c
+++ b/fs/ubifs/lprops.c
@@ -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)
OpenPOWER on IntegriCloud