diff options
author | Guy Harris <guy@alum.mit.edu> | 2016-09-07 10:43:15 -0700 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2016-09-07 20:41:53 +0000 |
commit | c7fc2802221877d939bb939f04766a5a30cfdb9f (patch) | |
tree | 7762d3a9dbf0cd8caffdfed880ce576c3e5cb1cd /wsutil/strtoi.c | |
parent | a66aa4c9c90017f0c03412b02a2be2b2cef3ac5d (diff) | |
download | wireshark-c7fc2802221877d939bb939f04766a5a30cfdb9f.tar.gz |
Make the ws_strto* routines more like the strto* routines.
Not all uses of atoi() or various strto* routines in Wireshark expect
the string to contain *only* a number, so not all uses should require
that the byte after the number be a '\0'. Have the ws_strto* routines
take a "pointer a pointer set to point to the character after the
number" argument, and have the callers do the appropriate checks of the
character after that.
This fixes the VMS trace reading code so that it can read those files
again.
The get_ routines are handed command-line arguments, so they *do* expect
the string to contain only a number; have them check to make sure the
byte after the number is a '\0'.
Change-Id: I46fc1bea7912b9278e385fe38491a0a2ad60d697
Reviewed-on: https://code.wireshark.org/review/17560
Reviewed-by: Guy Harris <guy@alum.mit.edu>
Diffstat (limited to 'wsutil/strtoi.c')
-rw-r--r-- | wsutil/strtoi.c | 38 |
1 files changed, 26 insertions, 12 deletions
diff --git a/wsutil/strtoi.c b/wsutil/strtoi.c index 96683d9d43..26e756d444 100644 --- a/wsutil/strtoi.c +++ b/wsutil/strtoi.c @@ -27,15 +27,17 @@ #include "strtoi.h" #include <errno.h> -gboolean ws_strtoi64(const gchar* str, gint64* cint) +gboolean ws_strtoi64(const gchar* str, const gchar** endptr, gint64* cint) { - gchar* endptr; + gchar* end; gint64 val; errno = 0; - val = g_ascii_strtoll(str, &endptr, 10); - if ((val == 0 && endptr == str) || (*endptr != 0)) { + val = g_ascii_strtoll(str, &end, 10); + if (val == 0 && end == str) { *cint = 0; + if (endptr != NULL) + *endptr = end; errno = EINVAL; return FALSE; } @@ -45,16 +47,20 @@ gboolean ws_strtoi64(const gchar* str, gint64* cint) * report the value as "too small" or "too large". */ *cint = val; + if (endptr != NULL) + *endptr = end; /* errno is already set */ return FALSE; } + if (endptr != NULL) + *endptr = end; *cint = val; return TRUE; } -gboolean ws_strtou64(const gchar* str, guint64* cint) +gboolean ws_strtou64(const gchar* str, const gchar** endptr, guint64* cint) { - gchar* endptr; + gchar* end; guint64 val; if (str[0] == '-' || str[0] == '+') { @@ -62,13 +68,17 @@ gboolean ws_strtou64(const gchar* str, guint64* cint) * Unsigned numbers don't have a sign. */ *cint = 0; + if (endptr != NULL) + *endptr = str; errno = EINVAL; return FALSE; } errno = 0; - val = g_ascii_strtoull(str, &endptr, 10); - if ((val == 0 && endptr == str) || (*endptr != 0)) { + val = g_ascii_strtoull(str, &end, 10); + if (val == 0 && end == str) { *cint = 0; + if (endptr != NULL) + *endptr = end; errno = EINVAL; return FALSE; } @@ -77,18 +87,22 @@ gboolean ws_strtou64(const gchar* str, guint64* cint) * Return the value, because ws_strtoi64() does. */ *cint = val; + if (endptr != NULL) + *endptr = end; /* errno is already set */ return FALSE; } + if (endptr != NULL) + *endptr = end; *cint = val; return TRUE; } #define DEFINE_WS_STRTOI_BITS(bits) \ -gboolean ws_strtoi##bits(const gchar* str, gint##bits* cint) \ +gboolean ws_strtoi##bits(const gchar* str, const gchar** endptr, gint##bits* cint) \ { \ gint64 val; \ - if (!ws_strtoi64(str, &val)) { \ + if (!ws_strtoi64(str, endptr, &val)) { \ /* \ * For ERANGE, return either G_MININT##bits or \ * G_MAXINT##bits so our caller knows whether \ @@ -134,10 +148,10 @@ DEFINE_WS_STRTOI_BITS(16) DEFINE_WS_STRTOI_BITS(8) #define DEFINE_WS_STRTOU_BITS(bits) \ -int ws_strtou##bits(const gchar* str, guint##bits* cint) \ +int ws_strtou##bits(const gchar* str, const gchar** endptr, guint##bits* cint) \ { \ guint64 val; \ - if (!ws_strtou64(str, &val)) { \ + if (!ws_strtou64(str, endptr, &val)) { \ /* \ * For ERANGE, return G_MAXUINT##bits for parallelism \ * with ws_strtoi##bits(). \ |