diff options
author | Ashish Pandey <aspandey@redhat.com> | 2018-08-30 15:08:02 +0530 |
---|---|---|
committer | Amar Tumballi <amarts@redhat.com> | 2018-08-31 12:56:49 +0000 |
commit | 67cfacaa9d5ce31240a4d39e81b2984a57ac8865 (patch) | |
tree | 020f142e630a5cbe20c7cae45fe2971b5bd2ed74 /xlators/cluster/ec/src/ec-types.h | |
parent | 19436f2e5fa6dc7f33b7d32d170d3c18f5fdd0d4 (diff) |
cluster/ec: Fix Coverity issue
Fix following coverity issues-
CID:
1382378
1382459
https://scan6.coverity.com/reports.htm#v42607/p10714/fileInstanceId=85091670&defectInstanceId=25915064&mergedDefectId=1382459
https://scan6.coverity.com/reports.htm#v42607/p10714/fileInstanceId=85091670&defectInstanceId=25915063&mergedDefectId=1382378
Problem:
ASSERT_LOCAL(this, healer) function is supposed to get the local healer
so that we can take advantage of it while healing and reading data.
However, we are not using healer->local anywhere.
Also, this is not as useful in context of EC as it is in AFR.
In EC we have to raed fragments from 4 bricks to heal a bad
fragment on other brick.
Change-Id: Iea8ce127ea02cc84e3823cb2be82a47872217b33
updates: bz#789278
Signed-off-by: Ashish Pandey <aspandey@redhat.com>
Diffstat (limited to 'xlators/cluster/ec/src/ec-types.h')
-rw-r--r-- | xlators/cluster/ec/src/ec-types.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/xlators/cluster/ec/src/ec-types.h b/xlators/cluster/ec/src/ec-types.h index 2724da6415f..0682a73fd6b 100644 --- a/xlators/cluster/ec/src/ec-types.h +++ b/xlators/cluster/ec/src/ec-types.h @@ -596,7 +596,6 @@ struct _ec_heal { struct subvol_healer { xlator_t *this; int subvol; - gf_boolean_t local; gf_boolean_t running; gf_boolean_t rerun; pthread_mutex_t mutex; |