diff options
| author | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2013-10-18 16:47:43 +0200 |
|---|---|---|
| committer | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2013-10-31 13:18:49 +0100 |
| commit | 8ff186f81fc7ab979a21c03ef58d5bf61cde44dd (patch) | |
| tree | 1b2f6a370e2742cfc242207aa03e4ce84f911d8a /freebsd/sbin/ifconfig | |
| parent | IFCONFIG(8): Add const qualifier (diff) | |
| download | rtems-libbsd-8ff186f81fc7ab979a21c03ef58d5bf61cde44dd.tar.bz2 | |
IFCONFIG(8): Fix isdigit() and isprint() usage
Diffstat (limited to 'freebsd/sbin/ifconfig')
| -rw-r--r-- | freebsd/sbin/ifconfig/af_inet.c | 2 | ||||
| -rw-r--r-- | freebsd/sbin/ifconfig/ifbridge.c | 2 | ||||
| -rw-r--r-- | freebsd/sbin/ifconfig/ifgroup.c | 4 |
3 files changed, 4 insertions, 4 deletions
diff --git a/freebsd/sbin/ifconfig/af_inet.c b/freebsd/sbin/ifconfig/af_inet.c index 3ef6c1a1..7a4acfc9 100644 --- a/freebsd/sbin/ifconfig/af_inet.c +++ b/freebsd/sbin/ifconfig/af_inet.c @@ -116,7 +116,7 @@ in_getaddr(const char *s, int which) int masklen; struct sockaddr_in *min = sintab[MASK]; *p = '\0'; - if (!isdigit(*(p + 1))) + if (!isdigit((unsigned char)*(p + 1))) errstr = "invalid"; else masklen = (int)strtonum(p + 1, 0, 32, &errstr); diff --git a/freebsd/sbin/ifconfig/ifbridge.c b/freebsd/sbin/ifconfig/ifbridge.c index eebeebd4..05ebfe31 100644 --- a/freebsd/sbin/ifconfig/ifbridge.c +++ b/freebsd/sbin/ifconfig/ifbridge.c @@ -155,7 +155,7 @@ bridge_interfaces(int s, const char *prefix) err(1, "strdup"); /* replace the prefix with whitespace */ for (p = pad; *p != '\0'; p++) { - if(isprint(*p)) + if(isprint((unsigned char)*p)) *p = ' '; } diff --git a/freebsd/sbin/ifconfig/ifgroup.c b/freebsd/sbin/ifconfig/ifgroup.c index 619c8f74..596a5220 100644 --- a/freebsd/sbin/ifconfig/ifgroup.c +++ b/freebsd/sbin/ifconfig/ifgroup.c @@ -52,7 +52,7 @@ setifgroup(const char *group_name, int d, int s, const struct afswtch *rafp) memset(&ifgr, 0, sizeof(ifgr)); strlcpy(ifgr.ifgr_name, name, IFNAMSIZ); - if (group_name[0] && isdigit(group_name[strlen(group_name) - 1])) + if (group_name[0] && isdigit((unsigned char)group_name[strlen(group_name) - 1])) errx(1, "setifgroup: group names may not end in a digit"); if (strlcpy(ifgr.ifgr_group, group_name, IFNAMSIZ) >= IFNAMSIZ) @@ -70,7 +70,7 @@ unsetifgroup(const char *group_name, int d, int s, const struct afswtch *rafp) memset(&ifgr, 0, sizeof(ifgr)); strlcpy(ifgr.ifgr_name, name, IFNAMSIZ); - if (group_name[0] && isdigit(group_name[strlen(group_name) - 1])) + if (group_name[0] && isdigit((unsigned char)group_name[strlen(group_name) - 1])) errx(1, "unsetifgroup: group names may not end in a digit"); if (strlcpy(ifgr.ifgr_group, group_name, IFNAMSIZ) >= IFNAMSIZ) |