From: Alejandro Colomar <alx@kernel.org>
Date: Sun, 30 Jun 2024 01:00:40 +0000 (+0200)
Subject: lib/atoi/strtoi/, tests/: strto[iu]_(): 1 is an invalid base
X-Git-Tag: 4.17.0-rc1~112
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=63297e836d000db7928b02cf9608796346705611;p=thirdparty%2Fshadow.git

lib/atoi/strtoi/, tests/: strto[iu]_(): 1 is an invalid base

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

diff --git a/lib/atoi/strtoi/strtoi.h b/lib/atoi/strtoi/strtoi.h
index a8f6f5408..8fa6b62f1 100644
--- a/lib/atoi/strtoi/strtoi.h
+++ b/lib/atoi/strtoi/strtoi.h
@@ -36,7 +36,7 @@ strtoi_(const char *s, char **restrict endp, int base,
 	if (status == NULL)
 		status = &st;
 
-	if (base != 0 && (base < 0 || base > 36)) {
+	if (base != 0 && (base < 2 || base > 36)) {
 		*status = EINVAL;
 		return MAX(min, MIN(max, 0));
 	}
diff --git a/lib/atoi/strtoi/strtou.h b/lib/atoi/strtoi/strtou.h
index 5ce7bd6a1..7ca6b346f 100644
--- a/lib/atoi/strtoi/strtou.h
+++ b/lib/atoi/strtoi/strtou.h
@@ -36,7 +36,7 @@ strtou_(const char *s, char **restrict endp, int base,
 	if (status == NULL)
 		status = &st;
 
-	if (base != 0 && (base < 0 || base > 36)) {
+	if (base != 0 && (base < 2 || base > 36)) {
 		*status = EINVAL;
 		return MAX(min, 0);
 	}
diff --git a/tests/unit/test_atoi_strtoi.c b/tests/unit/test_atoi_strtoi.c
index 02d62cf3d..782c8a4c6 100644
--- a/tests/unit/test_atoi_strtoi.c
+++ b/tests/unit/test_atoi_strtoi.c
@@ -46,6 +46,9 @@ test_strtoi(void **state)
 	assert_true(strtoi_("42", NULL, -1, 1, 2, &status) == 1);
 	assert_true(status == EINVAL);
 
+	assert_true(strtoi_("42", NULL, 1, 1, 2, &status) == 1);
+	assert_true(status == EINVAL);
+
 	assert_true(strtoi_("40", &end, 5, INTMAX_MIN, INTMAX_MAX, &status) == 20);
 	assert_true(status == 0);
 	assert_true(strcmp(end, "") == 0);
@@ -84,6 +87,9 @@ test_strtou(void **state)
 	assert_true(strtou_("42", NULL, -1, 1, 2, &status) == 1);
 	assert_true(status == EINVAL);
 
+	assert_true(strtou_("42", NULL, 1, 1, 2, &status) == 1);
+	assert_true(status == EINVAL);
+
 	assert_true(strtou_("40", &end, 5, 0, UINTMAX_MAX, &status) == 20);
 	assert_true(status == 0);
 	assert_true(strcmp(end, "") == 0);
@@ -123,6 +129,10 @@ test_strtou_noneg(void **state)
 	            == 1);
 	assert_true(status == EINVAL);
 
+	assert_true(strtou_noneg("42", NULL, 1, 1, 2, &status)
+	            == 1);
+	assert_true(status == EINVAL);
+
 	assert_true(strtou_noneg("40", &end, 5, 0, UINTMAX_MAX, &status)
 	            == 20);
 	assert_true(status == 0);