diff mbox series

[01/16] lib: fix match_string() helper when array size is positive

Message ID 20190508112842.11654-3-alexandru.ardelean@analog.com (mailing list archive)
State New, archived
Headers show
Series [01/16] lib: fix match_string() helper when array size is positive | expand

Commit Message

Alexandru Ardelean May 8, 2019, 11:28 a.m. UTC
The documentation the `_match_string()` helper mentions that `n`
(size of the given array) should be:
 * @n: number of strings in the array or -1 for NULL terminated arrays

The behavior of the function is different, in the sense that it exits on
the first NULL element in the array, regardless of whether `n` is -1 or a
positive number.

This patch changes the behavior, to exit the loop when a NULL element is
found and n == -1. Essentially, this aligns the behavior with the

There are currently many users of `match_string()`, and so, in order to go
through them, the next patches in the series will focus on doing some
cosmetic changes, which are aimed at grouping the users of

Signed-off-by: Alexandru Ardelean <alexandru.ardelean@analog.com>
 lib/string.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)
diff mbox series


diff --git a/lib/string.c b/lib/string.c
index 3ab861c1a857..76edb7bf76cb 100644
--- a/lib/string.c
+++ b/lib/string.c
@@ -648,8 +648,11 @@  int match_string(const char * const *array, size_t n, const char *string)
 	for (index = 0; index < n; index++) {
 		item = array[index];
-		if (!item)
+		if (!item) {
+			if (n != (size_t)-1)
+				continue;
+		}
 		if (!strcmp(item, string))
 			return index;