diff options
| author | Danny Al-Gaaf <danny.al-gaaf@bisect.de> | 2014-03-19 22:47:14 +0100 |
|---|---|---|
| committer | Danny Al-Gaaf <danny.al-gaaf@bisect.de> | 2014-04-22 20:08:26 +0200 |
| commit | 3b6364e5f20e0a5a6e3aa8dd143e5fdd8fc867ad (patch) | |
| tree | f807d6d5dca77d7f2fbfccd85dd150bc23e2c13f /src | |
| parent | 86b994df05ce792b792411b813f1c11f077ddb52 (diff) | |
| download | gf-complete-3b6364e5f20e0a5a6e3aa8dd143e5fdd8fc867ad.tar.gz | |
gf_w4.c: add missing breaks
Since there is no comment indicating fallthrough on purpose added a
break in switch value 5 and 6.
Fix for coverity issue from Ceph project:
CID 1193082 (#1 of 1): Missing break in switch (MISSING_BREAK)
unterminated_case: This case (value 5) is not terminated by a 'break'
statement.
CID 1193083 (#1 of 1): Missing break in switch (MISSING_BREAK)
unterminated_case: This case (value 6) is not terminated by a 'break'
statement.
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Diffstat (limited to 'src')
| -rw-r--r-- | src/gf_w4.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/gf_w4.c b/src/gf_w4.c index 2397d82..11d21e8 100644 --- a/src/gf_w4.c +++ b/src/gf_w4.c @@ -1584,6 +1584,7 @@ gf_w4_bytwo_b_nosse_multiply_region(gf_t *gf, void *src, void *dest, gf_val_32_t s64++; } } + break; case 6: if (xor) { while (d64 < (uint64_t *) rd.d_top) { @@ -1606,6 +1607,7 @@ gf_w4_bytwo_b_nosse_multiply_region(gf_t *gf, void *src, void *dest, gf_val_32_t s64++; } } + break; case 7: if (xor) { while (d64 < (uint64_t *) rd.d_top) { |