Unverified Commit 74590f83 authored by Viktor Söderqvist's avatar Viktor Söderqvist Committed by GitHub
Browse files

redis-cli: Assert > 0 before dividing, to silence warning by tool (#9396)

Also make sure function can't return NULL by another assert.
parent 0835f596
...@@ -4679,6 +4679,7 @@ static clusterManagerNode *clusterManagerNodeMasterRandom() { ...@@ -4679,6 +4679,7 @@ static clusterManagerNode *clusterManagerNodeMasterRandom() {
master_count++; master_count++;
} }
assert(master_count > 0);
srand(time(NULL)); srand(time(NULL));
idx = rand() % master_count; idx = rand() % master_count;
listRewind(cluster_manager.nodes, &li); listRewind(cluster_manager.nodes, &li);
...@@ -4690,7 +4691,7 @@ static clusterManagerNode *clusterManagerNodeMasterRandom() { ...@@ -4690,7 +4691,7 @@ static clusterManagerNode *clusterManagerNodeMasterRandom() {
} }
} }
/* Can not be reached */ /* Can not be reached */
return NULL; assert(0);
} }
static int clusterManagerFixSlotsCoverage(char *all_slots) { static int clusterManagerFixSlotsCoverage(char *all_slots) {
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment