diff options
author | Danny Al-Gaaf <danny.al-gaaf@bisect.de> | 2014-03-19 22:32:35 +0100 |
---|---|---|
committer | Danny Al-Gaaf <danny.al-gaaf@bisect.de> | 2014-04-22 20:08:26 +0200 |
commit | 86b994df05ce792b792411b813f1c11f077ddb52 (patch) | |
tree | 3d2944c90a7ffc70595ba98baac92183007598d6 /src | |
parent | e958b0437dd91b66457522b414091ea06deb5677 (diff) | |
download | gf-complete-86b994df05ce792b792411b813f1c11f077ddb52.tar.gz |
gf_w32.c: add missing breaks
Since there is no comment indicating fallthrough on purpose added a
break in switch value 3 and 5/before default.
Fix for coverity issue from Ceph project:
CID 1193080 (#1 of 1): Missing break in switch (MISSING_BREAK)
unterminated_case: This case (value 3) is not terminated by a 'break'
statement.
CID 1193081 (#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_w32.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/gf_w32.c b/src/gf_w32.c index 1503c72..8f7790c 100644 --- a/src/gf_w32.c +++ b/src/gf_w32.c @@ -1096,6 +1096,7 @@ gf_w32_bytwo_b_nosse_multiply_region(gf_t *gf, void *src, void *dest, gf_val_32_ s64++; } } + break; case 4: if (xor) { while (d64 < (uint64_t *) rd.d_top) { @@ -1139,6 +1140,7 @@ gf_w32_bytwo_b_nosse_multiply_region(gf_t *gf, void *src, void *dest, gf_val_32_ s64++; } } + break; default: if (xor) { while (d64 < (uint64_t *) rd.d_top) { |