summaryrefslogtreecommitdiffstats
path: root/xlators/debug
diff options
context:
space:
mode:
authorMichael Scherer <misc@redhat.com>2017-02-19 11:06:56 +0100
committerShyamsundar Ranganathan <srangana@redhat.com>2017-02-24 09:52:45 -0500
commitff2ba49d02dc99b0c3de5712348ae9d588be7ad7 (patch)
tree6f38131e3b941ab99d3d8f4417728b65715d888e /xlators/debug
parentf08f5bfef923259fe6de6d41a0a2b1b9140fc547 (diff)
Filter coverty false positives
Since random is not used for anything cryptographically related, it is fine to use it in those instances. Change-Id: I720172285f60d6bc477c7169c7286fc018ebdf8f BUG: 1424764 Signed-off-by: Michael Scherer <misc@redhat.com> Reviewed-on: https://review.gluster.org/16668 Smoke: Gluster Build System <jenkins@build.gluster.org> Tested-by: Nigel Babu <nigelb@redhat.com> NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org> CentOS-regression: Gluster Build System <jenkins@build.gluster.org> Reviewed-by: Shyamsundar Ranganathan <srangana@redhat.com>
Diffstat (limited to 'xlators/debug')
-rw-r--r--xlators/debug/error-gen/src/error-gen.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/xlators/debug/error-gen/src/error-gen.c b/xlators/debug/error-gen/src/error-gen.c
index b6b17baa87f..103c302d3ba 100644
--- a/xlators/debug/error-gen/src/error-gen.c
+++ b/xlators/debug/error-gen/src/error-gen.c
@@ -178,6 +178,7 @@ generate_rand_no (int op_no)
int rand_no = 0;
if (op_no < GF_FOP_MAXVALUE)
+ /* coverty[DC.WEAK_CRYPTO] */
rand_no = rand () % error_no_list[op_no].error_no_count;
return rand_no;
}
@@ -367,6 +368,7 @@ error_gen (xlator_t *this, int op_no)
ret = error_no_list[op_no].error_no[rand_no];
}
if (egp->random_failure == _gf_true)
+ /* coverty[DC.WEAK_CRYPTO] */
egp->failure_iter_no = 3 + (rand () % GF_UNIVERSAL_ANSWER);
}
return ret;