From ff2ba49d02dc99b0c3de5712348ae9d588be7ad7 Mon Sep 17 00:00:00 2001 From: Michael Scherer Date: Sun, 19 Feb 2017 11:06:56 +0100 Subject: 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 Reviewed-on: https://review.gluster.org/16668 Smoke: Gluster Build System Tested-by: Nigel Babu NetBSD-regression: NetBSD Build System CentOS-regression: Gluster Build System Reviewed-by: Shyamsundar Ranganathan --- xlators/cluster/dht/src/tier.c | 1 + xlators/debug/error-gen/src/error-gen.c | 2 ++ xlators/features/locks/src/common.c | 1 + 3 files changed, 4 insertions(+) diff --git a/xlators/cluster/dht/src/tier.c b/xlators/cluster/dht/src/tier.c index 41032743c04..b13c41cd2ab 100644 --- a/xlators/cluster/dht/src/tier.c +++ b/xlators/cluster/dht/src/tier.c @@ -445,6 +445,7 @@ tier_do_migration (xlator_t *this, int promote) migrate = promote ? 0 : 1; break; case TIER_WM_MID: + /* coverty[DC.WEAK_CRYPTO] */ rand = random() % 100; if (promote) { migrate = (rand > tier_conf->percent_full); 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; diff --git a/xlators/features/locks/src/common.c b/xlators/features/locks/src/common.c index 8cdff0d61ef..68f253d92ad 100644 --- a/xlators/features/locks/src/common.c +++ b/xlators/features/locks/src/common.c @@ -1129,6 +1129,7 @@ pl_does_monkey_want_stuck_lock() long int monkey_unlock_rand = 0; long int monkey_unlock_rand_rem = 0; + /* coverty[DC.WEAK_CRYPTO] */ monkey_unlock_rand = random (); monkey_unlock_rand_rem = monkey_unlock_rand % 100; if (monkey_unlock_rand_rem == 0) -- cgit