aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDaniel Stenberg <daniel@haxx.se>2004-11-11 09:26:09 +0000
committerDaniel Stenberg <daniel@haxx.se>2004-11-11 09:26:09 +0000
commitcd73a733c78b7ab7b4e3ccaca9350bbcce124142 (patch)
tree4d9772c476213fa9c7f4034d7860c75f97b83dc5
parent8ef8e949bd07a72918565f4482f8250a55a2f2dc (diff)
dates from 2038 or later now return 0x7fffffff when 32 bit time_t is used
-rw-r--r--CHANGES5
-rw-r--r--configure.ac1
-rw-r--r--docs/libcurl/curl_getdate.33
-rw-r--r--lib/parsedate.c6
-rw-r--r--lib/setup.h5
5 files changed, 20 insertions, 0 deletions
diff --git a/CHANGES b/CHANGES
index d66143eb7..890f64b9e 100644
--- a/CHANGES
+++ b/CHANGES
@@ -6,6 +6,11 @@
Changelog
+Daniel (11 November 2004)
+- Jeff Phillips found out that a date string with a year beyond 2038 could
+ crash the new date parser on systems with 32bit time_t. We now check for
+ this case and deal with it.
+
Daniel (10 November 2004)
- I installed Heimdal on my Debian box (using the debian package) and noticed
that configure --with-gssapi failed to create a nice build. Fixed now.
diff --git a/configure.ac b/configure.ac
index d8aaf0238..78e80a5e7 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1171,6 +1171,7 @@ AC_CHECK_SIZEOF(curl_off_t, ,[
])
AC_CHECK_SIZEOF(size_t)
AC_CHECK_SIZEOF(long)
+AC_CHECK_SIZEOF(time_t)
AC_CHECK_TYPE(long long,
[AC_DEFINE(HAVE_LONGLONG, 1, [if your compiler supports long long])]
diff --git a/docs/libcurl/curl_getdate.3 b/docs/libcurl/curl_getdate.3
index 65a0e738b..9d13433a3 100644
--- a/docs/libcurl/curl_getdate.3
+++ b/docs/libcurl/curl_getdate.3
@@ -83,6 +83,9 @@ only ones RFC2616 says HTTP applications may use.
.SH RETURN VALUE
This function returns -1 when it fails to parse the date string. Otherwise it
returns the number of seconds as described.
+
+If the year is larger than 2037 on systems with 32 bit time_t, this function
+will return 0x7fffffff (since that is the largest possible 31 bit number).
.SH REWRITE
The former version of this function was built with yacc and was not only very
large, it was also never quite understood and it wasn't possible to build with
diff --git a/lib/parsedate.c b/lib/parsedate.c
index 836340c07..e9e860fe6 100644
--- a/lib/parsedate.c
+++ b/lib/parsedate.c
@@ -350,6 +350,12 @@ static time_t Curl_parsedate(const char *date)
/* lacks vital info, fail */
return -1;
+#if SIZEOF_TIME_T < 5
+ /* 32 bit time_t can only hold dates to the beginning of 2038 */
+ if(yearnum > 2037)
+ return 0x7fffffff;
+#endif
+
tm.tm_sec = secnum;
tm.tm_min = minnum;
tm.tm_hour = hournum;
diff --git a/lib/setup.h b/lib/setup.h
index e535279c8..2969b936a 100644
--- a/lib/setup.h
+++ b/lib/setup.h
@@ -287,6 +287,11 @@ typedef int curl_socket_t;
#define USE_LIBIDN
#endif
+#ifndef SIZEOF_TIME_T
+/* assume default size of time_t to be 32 bit */
+#define SIZEOF_TIME_T 4
+#endif
+
#define LIBIDN_REQUIRED_VERSION "0.4.1"
#endif /* __CONFIG_H */