summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNikita Popov <nikita.ppv@gmail.com>2020-08-05 18:36:27 +0200
committerNikita Popov <nikita.ppv@gmail.com>2020-08-05 18:40:23 +0200
commit2965c8f86890c106ed34266e7bd32142cd247aea (patch)
tree74562d0deb10a0880eb5fdc8d90da66267a462be
parent74b285d78caf92364584397c2d986ad35cdeb856 (diff)
downloadphp-git-2965c8f86890c106ed34266e7bd32142cd247aea.tar.gz
Prefer strtoll over atoll
Both are specified by C99, but strtoll has specified overflow behavior while atoll does not, so prefer using it.
-rw-r--r--ext/date/php_date.c6
-rw-r--r--main/rfc1867.c9
2 files changed, 1 insertions, 14 deletions
diff --git a/ext/date/php_date.c b/ext/date/php_date.c
index 8f755d887f..d81bc1659a 100644
--- a/ext/date/php_date.c
+++ b/ext/date/php_date.c
@@ -51,11 +51,7 @@ static inline long long php_date_llabs( long long i ) { return i >= 0 ? i : -i;
int st = snprintf(s, len, "%lld", i); \
s[st] = '\0'; \
} while (0);
-#ifdef HAVE_ATOLL
-# define DATE_A64I(i, s) i = atoll(s)
-#else
-# define DATE_A64I(i, s) i = strtoll(s, NULL, 10)
-#endif
+#define DATE_A64I(i, s) i = strtoll(s, NULL, 10)
#endif
PHPAPI time_t php_time(void)
diff --git a/main/rfc1867.c b/main/rfc1867.c
index 9dcd5ad07f..8ef83f443d 100644
--- a/main/rfc1867.c
+++ b/main/rfc1867.c
@@ -31,11 +31,6 @@
#include "ext/standard/php_string.h"
#include "zend_smart_string.h"
-#if defined(PHP_WIN32) && !defined(HAVE_ATOLL)
-# define atoll(s) _atoi64(s)
-# define HAVE_ATOLL 1
-#endif
-
#ifndef DEBUG_FILE_UPLOAD
# define DEBUG_FILE_UPLOAD 0
#endif
@@ -903,11 +898,7 @@ SAPI_API SAPI_POST_HANDLER_FUNC(rfc1867_post_handler) /* {{{ */
}
if (!strcasecmp(param, "MAX_FILE_SIZE")) {
-#ifdef HAVE_ATOLL
- max_file_size = atoll(value);
-#else
max_file_size = strtoll(value, NULL, 10);
-#endif
}
efree(param);