diff options
| author | Danny Al-Gaaf <danny.al-gaaf@bisect.de> | 2014-03-19 22:57:27 +0100 |
|---|---|---|
| committer | Danny Al-Gaaf <danny.al-gaaf@bisect.de> | 2014-04-22 20:08:26 +0200 |
| commit | fa5ec8112b66471559a4e983085703401e2fd866 (patch) | |
| tree | 95d97a139f9869f727cfaa8b994793acce814ea6 /src | |
| parent | 3b6364e5f20e0a5a6e3aa8dd143e5fdd8fc867ad (diff) | |
| download | gf-complete-fa5ec8112b66471559a4e983085703401e2fd866.tar.gz | |
gf_w8.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 1193084 (#1 of 1): Missing break in switch (MISSING_BREAK)
unterminated_case: This case (value 5) 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_w8.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/gf_w8.c b/src/gf_w8.c index d3e2bf6..a2a8600 100644 --- a/src/gf_w8.c +++ b/src/gf_w8.c @@ -1932,6 +1932,7 @@ gf_w8_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) { @@ -1954,6 +1955,7 @@ gf_w8_bytwo_b_nosse_multiply_region(gf_t *gf, void *src, void *dest, gf_val_32_t s64++; } } + break; /* case 7: if (xor) { |