diff options
| author | Xavier Hernandez <xhernandez@datalab.es> | 2014-07-16 13:50:53 +0200 |
|---|---|---|
| committer | Vijay Bellur <vbellur@redhat.com> | 2014-07-21 18:39:46 -0700 |
| commit | 464ff8f7592e7d6a9239ca1fab1928d4a608f253 (patch) | |
| tree | ede9747e4b9c79ad0da3598eb43ff922d132e3dc /xlators/cluster/ec/src/ec-method.c | |
| parent | f6ddb4675c096dc81420ed84fb2a6fefa9fea563 (diff) | |
ec: Fixed coveriry scan issues
CID list:
1226163 Logically dead code
1226166 Missing break in switch
1226167 Missing break in switch
1226168 Missing break in switch
1226169 Missing break in switch
1226170 Missing break in switch
1226171 Missing break in switch
1226172 Missing break in switch
1226173 Missing break in switch
1226174 Missing break in switch
1226175 Missing break in switch
1226176 Missing break in switch
1226177 Missing break in switch
1226178 Data race condition
1226179 Data race condition
1226180 Data race condition
1226181 Thread deadlock
1226182 Uninitialized pointer read
1226183 Uninitialized pointer read
1226184 Read from pointer after free
Change-Id: I4d33aa42289371927175c43bb29e018df64fb943
BUG: 789278
Signed-off-by: Xavier Hernandez <xhernandez@datalab.es>
Reviewed-on: http://review.gluster.org/8317
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Vijay Bellur <vbellur@redhat.com>
Diffstat (limited to 'xlators/cluster/ec/src/ec-method.c')
| -rw-r--r-- | xlators/cluster/ec/src/ec-method.c | 9 |
1 files changed, 1 insertions, 8 deletions
diff --git a/xlators/cluster/ec/src/ec-method.c b/xlators/cluster/ec/src/ec-method.c index deedf2a21f2..83b603bd14a 100644 --- a/xlators/cluster/ec/src/ec-method.c +++ b/xlators/cluster/ec/src/ec-method.c @@ -96,7 +96,7 @@ size_t ec_method_decode(size_t size, uint32_t columns, uint32_t * rows, uint8_t ** in, uint8_t * out) { uint32_t i, j, k; - uint32_t f, off, mask; + uint32_t f, off; uint8_t inv[EC_METHOD_MAX_FRAGMENTS][EC_METHOD_MAX_FRAGMENTS + 1]; uint8_t mtx[EC_METHOD_MAX_FRAGMENTS][EC_METHOD_MAX_FRAGMENTS]; uint8_t * p[EC_METHOD_MAX_FRAGMENTS]; @@ -105,7 +105,6 @@ size_t ec_method_decode(size_t size, uint32_t columns, uint32_t * rows, memset(inv, 0, sizeof(inv)); memset(mtx, 0, sizeof(mtx)); - mask = 0; for (i = 0; i < columns; i++) { inv[i][i] = 1; @@ -114,11 +113,6 @@ size_t ec_method_decode(size_t size, uint32_t columns, uint32_t * rows, k = 0; for (i = 0; i < columns; i++) { - while ((mask & 1) != 0) - { - k++; - mask >>= 1; - } mtx[k][columns - 1] = 1; for (j = columns - 1; j > 0; j--) { @@ -126,7 +120,6 @@ size_t ec_method_decode(size_t size, uint32_t columns, uint32_t * rows, } p[k] = in[i]; k++; - mask >>= 1; } for (i = 0; i < columns; i++) |