Commit Diff

Commit:
c9fac28daf953636d570ceb3b0a730af6c0a2355
From:
Markus Uhlin <markus@nifty-networks.net>
Date:
Message:
Fixed usage of possibly uninitialized variable
Actions:
Patch | Tree
commit - 9ea5c578303493721723650ebb10063c27db91b0
commit + c9fac28daf953636d570ceb3b0a730af6c0a2355
blob - 4fe252b8554260159ac1d881575aa73683647f4c
blob + a37e60a8da29b5a14ee02de4e14da22b1fb17522
--- FICS/comproc.c
+++ FICS/comproc.c
@@ -1170,6 +1170,10 @@ alpha_cmp(const void *pp1, const void *pp2)
 PRIVATE void
 sort_players(int players[PARRAY_SIZE], who_cmp_t cmp_func)
 {
+	if (p_num <= 0) {
+		warnx("%s: p_num <= 0", __func__);
+		return;
+	}
 	for (int i = 0; i < p_num; i++)
 		players[i] = i;
 	qsort(players, p_num, sizeof(int), cmp_func);