lib/list.c: is_on_list(): Move break condition to loop controlling expression

This change executes `i++` one more time before breaking, so we need to
update the `i+1` after the loop to just `i`.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

1 files changed, 2 insertions, 4 deletions

diff --git a/lib/list.c b/lib/list.c
index b88f5c5a..28f53068 100644
--- a/lib/list.c
+++ b/lib/list.c
@@ -232,15 +232,13 @@ bool is_on_list (char *const *list, const char *member)
 	 * array of pointers.
 	 */
 
-	for (cp = members, i = 0;; i++) {
+	for (cp = members, i = 0; cp != NULL; i++) {
 		array[i] = cp;
 		cp = strchr(cp, ',');
 		if (NULL != cp)
 			*cp++ = '\0';
-		else
-			break;
 	}
-	array[i+1] = NULL;
+	array[i] = NULL;
 
 	/*
 	 * Return the new array of pointers