d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Detect range errors when using the %s specifier. Previously, LONG_MAX

Browse Source 
was rejected as a range error, while any values less than LONG_MIN
were silently substituted with LONG_MIN.  Furthermore, on some
platforms `time_t' has less range than `long' (e.g. alpha), which may
give incorrect results when parsing some strings.
This commit is contained in:
Jacques Vidrine 2003-11-17 04:19:15 +00:00
parent 81bbee5996
commit c91e947dbd
1 changed files with 10 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
lib/libc/stdtime/strptime.c
View File

@ -64,7 +64,7 @@ __FBSDID("$FreeBSD$");
#include "namespace.h"
#include <time.h>
#include <ctype.h>
#include <limits.h>
#include <errno.h>
#include <stdlib.h>
#include <string.h>
#include <pthread.h>
@ -444,11 +444,18 @@ label:
case 's':
{
char *cp;
int sverrno;
long n;
time_t t;
t = strtol(buf, &cp, 10);
if (t == LONG_MAX)
sverrno = errno;
errno = 0;
n = strtol(buf, &cp, 10);
if (errno == ERANGE || (long)(t = n) != n) {
errno = sverrno;
return 0;
}
errno = sverrno;
buf = cp;
gmtime_r(&t, tm);
*GMTp = 1;