about summary refs log tree commit diff
diff options
context:
space:
mode:
authorRich Felker <dalias@aerifal.cx>2011-11-10 20:44:44 -0500
committerRich Felker <dalias@aerifal.cx>2011-11-10 20:44:44 -0500
commit0c4188f6d76fad021a93eb1012630c717bda80a1 (patch)
treecf750ce90574fa55b066de623c0ce42005a26545
parent3ed8c9f2df0b5f0bfe1006037c46d4f32ec6ca7b (diff)
downloadmusl-0c4188f6d76fad021a93eb1012630c717bda80a1.tar.gz
musl-0c4188f6d76fad021a93eb1012630c717bda80a1.tar.xz
musl-0c4188f6d76fad021a93eb1012630c717bda80a1.zip
fix signed overflows at most-negative values in ato(i|l|ll)
patch by Pascal Cuoq (with minor tweaks to comments)
-rw-r--r--src/stdlib/atoi.c5
-rw-r--r--src/stdlib/atol.c5
-rw-r--r--src/stdlib/atoll.c5
3 files changed, 9 insertions, 6 deletions
diff --git a/src/stdlib/atoi.c b/src/stdlib/atoi.c
index 648b154f..9baca7b8 100644
--- a/src/stdlib/atoi.c
+++ b/src/stdlib/atoi.c
@@ -9,7 +9,8 @@ int atoi(const char *s)
 	case '-': neg=1;
 	case '+': s++;
 	}
+	/* Compute n as a negative number to avoid overflow on INT_MIN */
 	while (isdigit(*s))
-		n = 10*n + *s++ - '0';
-	return neg ? -n : n;
+		n = 10*n - (*s++ - '0');
+	return neg ? n : -n;
 }
diff --git a/src/stdlib/atol.c b/src/stdlib/atol.c
index 9c91bba9..140ea3ea 100644
--- a/src/stdlib/atol.c
+++ b/src/stdlib/atol.c
@@ -10,7 +10,8 @@ long atol(const char *s)
 	case '-': neg=1;
 	case '+': s++;
 	}
+	/* Compute n as a negative number to avoid overflow on LONG_MIN */
 	while (isdigit(*s))
-		n = 10*n + *s++ - '0';
-	return neg ? -n : n;
+		n = 10*n - (*s++ - '0');
+	return neg ? n : -n;
 }
diff --git a/src/stdlib/atoll.c b/src/stdlib/atoll.c
index 0e03e0a1..b6930489 100644
--- a/src/stdlib/atoll.c
+++ b/src/stdlib/atoll.c
@@ -10,7 +10,8 @@ long long atoll(const char *s)
 	case '-': neg=1;
 	case '+': s++;
 	}
+	/* Compute n as a negative number to avoid overflow on LLONG_MIN */
 	while (isdigit(*s))
-		n = 10*n + *s++ - '0';
-	return neg ? -n : n;
+		n = 10*n - (*s++ - '0');
+	return neg ? n : -n;
 }