diff options
author | PJ Waskiewicz <peter.p.waskiewicz.jr@intel.com> | 2009-12-23 23:22:08 -0800 |
---|---|---|
committer | Jeff Garzik <jgarzik@redhat.com> | 2009-12-24 03:29:18 -0500 |
commit | e2b0502a2fc578d2d3ffa41d0ac4ca6080d66386 (patch) | |
tree | a7c3e7417f9fb3e9cc59d8f31e2a1df01e61bb06 /ethtool.c | |
parent | 1d83d1bf255bb0ac3fda6f18041ae81675ede313 (diff) | |
download | ethtool-e2b0502a2fc578d2d3ffa41d0ac4ca6080d66386.tar.gz |
ethtool: Can't parse ints with stroul()
A recent change to how int's were being parsed from the command
line had them being read in with an unsigned int string operator.
This didn't allow signed numbers from being read in correctly.
This patch adds a get_uint() routine, and fixes the get_int()
routine to read in signed values.
Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
Diffstat (limited to 'ethtool.c')
-rw-r--r-- | ethtool.c | 23 |
1 files changed, 21 insertions, 2 deletions
@@ -324,6 +324,7 @@ typedef enum { CMDL_NONE, CMDL_BOOL, CMDL_INT, + CMDL_UINT, CMDL_STR, } cmdline_type_t; @@ -404,18 +405,32 @@ static struct cmdline_info cmdline_coalesce[] = { static int get_int(char *str, int base) { - unsigned long v; + long v; char *endp; if (!str) show_usage(1); errno = 0; - v = strtoul(str, &endp, base); + v = strtol(str, &endp, base); if ( errno || *endp || v > INT_MAX) show_usage(1); return (int)v; } +static int get_uint(char *str, int base) +{ + unsigned long v; + char *endp; + + if (!str) + show_usage(1); + errno = 0; + v = strtoul(str, &endp, base); + if ( errno || *endp || v > UINT_MAX) + show_usage(1); + return v; +} + static void parse_generic_cmdline(int argc, char **argp, int first_arg, int *changed, struct cmdline_info *info, @@ -447,6 +462,10 @@ static void parse_generic_cmdline(int argc, char **argp, *p = get_int(argp[i],0); break; } + case CMDL_UINT: { + *p = get_uint(argp[i],0); + break; + } case CMDL_STR: { char **s = info[idx].wanted_val; *s = strdup(argp[i]); |