diff options
author | hujianyang <hujianyang@huawei.com> | 2014-06-10 22:41:17 -0400 |
---|---|---|
committer | Artem Bityutskiy <artem.bityutskiy@linux.intel.com> | 2014-07-19 02:53:51 -0400 |
commit | ce6ebdb87ef94b332fa0bfafd62830bf50e6e2c6 (patch) | |
tree | 7c36dd8b16715c5d77a7d4f590f38c6acccaf1ea | |
parent | 5a95741a574f69c8219d3164d1fbf1c855566afe (diff) |
UBIFS: Add missing break statements in dbg_chk_pnode()
This is a minor fix. These two branches in 'dbg_chk_pnode()'
are dealing with different conditions. Although there is
no fault in current state, I think adding "break"s in
each end of branch is better.
Signed-off-by: hujianyang <hujianyang@huawei.com>
Signed-off-by: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
-rw-r--r-- | fs/ubifs/lpt.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/fs/ubifs/lpt.c b/fs/ubifs/lpt.c index d46b19ec1815..b4fb422e5b8d 100644 --- a/fs/ubifs/lpt.c +++ b/fs/ubifs/lpt.c | |||
@@ -2198,6 +2198,7 @@ static int dbg_chk_pnode(struct ubifs_info *c, struct ubifs_pnode *pnode, | |||
2198 | lprops->dirty); | 2198 | lprops->dirty); |
2199 | return -EINVAL; | 2199 | return -EINVAL; |
2200 | } | 2200 | } |
2201 | break; | ||
2201 | case LPROPS_FREEABLE: | 2202 | case LPROPS_FREEABLE: |
2202 | case LPROPS_FRDI_IDX: | 2203 | case LPROPS_FRDI_IDX: |
2203 | if (lprops->free + lprops->dirty != c->leb_size) { | 2204 | if (lprops->free + lprops->dirty != c->leb_size) { |
@@ -2206,6 +2207,7 @@ static int dbg_chk_pnode(struct ubifs_info *c, struct ubifs_pnode *pnode, | |||
2206 | lprops->dirty); | 2207 | lprops->dirty); |
2207 | return -EINVAL; | 2208 | return -EINVAL; |
2208 | } | 2209 | } |
2210 | break; | ||
2209 | } | 2211 | } |
2210 | } | 2212 | } |
2211 | return 0; | 2213 | return 0; |