diff options
| author | Michael Scherer <misc@redhat.com> | 2017-02-19 11:06:56 +0100 |
|---|---|---|
| committer | Shyamsundar Ranganathan <srangana@redhat.com> | 2017-02-24 09:52:45 -0500 |
| commit | ff2ba49d02dc99b0c3de5712348ae9d588be7ad7 (patch) | |
| tree | 6f38131e3b941ab99d3d8f4417728b65715d888e /xlators/debug/error-gen | |
| parent | f08f5bfef923259fe6de6d41a0a2b1b9140fc547 (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/error-gen')
| -rw-r--r-- | xlators/debug/error-gen/src/error-gen.c | 2 |
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; |