Commit bd389b90 authored by Frank Pavlic's avatar Frank Pavlic Committed by Jeff Garzik

[PATCH] s390: synthax checking for VIPA addresses fixed

[patch 1/7] s390: synthax checking for VIPA addresses fixed

From: Peter Tiedemann <ptiedem@de.ibm.com>
	- synthax checking for VIPA addresses fixed
Signed-off-by: default avatarFrank Pavlic <fpavlic@de.ibm.com>

diffstat:
 qeth.h     |   65 ++++++++++++++++++++++++++++++++++++++++++++++++-------------
 qeth_sys.c |    6 ++---
 2 files changed, 55 insertions(+), 16 deletions(-)
Signed-off-by: default avatarJeff Garzik <jgarzik@pobox.com>
parent 2ecc26b8
...@@ -8,6 +8,7 @@ ...@@ -8,6 +8,7 @@
#include <linux/trdevice.h> #include <linux/trdevice.h>
#include <linux/etherdevice.h> #include <linux/etherdevice.h>
#include <linux/if_vlan.h> #include <linux/if_vlan.h>
#include <linux/ctype.h>
#include <net/ipv6.h> #include <net/ipv6.h>
#include <linux/in6.h> #include <linux/in6.h>
...@@ -24,7 +25,7 @@ ...@@ -24,7 +25,7 @@
#include "qeth_mpc.h" #include "qeth_mpc.h"
#define VERSION_QETH_H "$Revision: 1.142 $" #define VERSION_QETH_H "$Revision: 1.151 $"
#ifdef CONFIG_QETH_IPV6 #ifdef CONFIG_QETH_IPV6
#define QETH_VERSION_IPV6 ":IPv6" #define QETH_VERSION_IPV6 ":IPv6"
...@@ -1074,6 +1075,26 @@ qeth_get_qdio_q_format(struct qeth_card *card) ...@@ -1074,6 +1075,26 @@ qeth_get_qdio_q_format(struct qeth_card *card)
} }
} }
static inline int
qeth_isdigit(char * buf)
{
while (*buf) {
if (!isdigit(*buf++))
return 0;
}
return 1;
}
static inline int
qeth_isxdigit(char * buf)
{
while (*buf) {
if (!isxdigit(*buf++))
return 0;
}
return 1;
}
static inline void static inline void
qeth_ipaddr4_to_string(const __u8 *addr, char *buf) qeth_ipaddr4_to_string(const __u8 *addr, char *buf)
{ {
...@@ -1090,18 +1111,27 @@ qeth_string_to_ipaddr4(const char *buf, __u8 *addr) ...@@ -1090,18 +1111,27 @@ qeth_string_to_ipaddr4(const char *buf, __u8 *addr)
int i; int i;
start = buf; start = buf;
for (i = 0; i < 3; i++) { for (i = 0; i < 4; i++) {
if (!(end = strchr(start, '.'))) if (i == 3) {
end = strchr(start,0xa);
if (end)
len = end - start;
else
len = strlen(start);
}
else {
end = strchr(start, '.');
len = end - start;
}
if ((len <= 0) || (len > 3))
return -EINVAL; return -EINVAL;
len = end - start;
memset(abuf, 0, 4); memset(abuf, 0, 4);
strncpy(abuf, start, len); strncpy(abuf, start, len);
if (!qeth_isdigit(abuf))
return -EINVAL;
addr[i] = simple_strtoul(abuf, &tmp, 10); addr[i] = simple_strtoul(abuf, &tmp, 10);
start = end + 1; start = end + 1;
} }
memset(abuf, 0, 4);
strcpy(abuf, start);
addr[3] = simple_strtoul(abuf, &tmp, 10);
return 0; return 0;
} }
...@@ -1128,18 +1158,27 @@ qeth_string_to_ipaddr6(const char *buf, __u8 *addr) ...@@ -1128,18 +1158,27 @@ qeth_string_to_ipaddr6(const char *buf, __u8 *addr)
tmp_addr = (u16 *)addr; tmp_addr = (u16 *)addr;
start = buf; start = buf;
for (i = 0; i < 7; i++) { for (i = 0; i < 8; i++) {
if (!(end = strchr(start, ':'))) if (i == 7) {
end = strchr(start,0xa);
if (end)
len = end - start;
else
len = strlen(start);
}
else {
end = strchr(start, ':');
len = end - start;
}
if ((len <= 0) || (len > 4))
return -EINVAL; return -EINVAL;
len = end - start;
memset(abuf, 0, 5); memset(abuf, 0, 5);
strncpy(abuf, start, len); strncpy(abuf, start, len);
if (!qeth_isxdigit(abuf))
return -EINVAL;
tmp_addr[i] = simple_strtoul(abuf, &tmp, 16); tmp_addr[i] = simple_strtoul(abuf, &tmp, 16);
start = end + 1; start = end + 1;
} }
memset(abuf, 0, 5);
strcpy(abuf, start);
tmp_addr[7] = simple_strtoul(abuf, &tmp, 16);
return 0; return 0;
} }
......
/* /*
* *
* linux/drivers/s390/net/qeth_sys.c ($Revision: 1.55 $) * linux/drivers/s390/net/qeth_sys.c ($Revision: 1.58 $)
* *
* Linux on zSeries OSA Express and HiperSockets support * Linux on zSeries OSA Express and HiperSockets support
* This file contains code related to sysfs. * This file contains code related to sysfs.
...@@ -20,7 +20,7 @@ ...@@ -20,7 +20,7 @@
#include "qeth_mpc.h" #include "qeth_mpc.h"
#include "qeth_fs.h" #include "qeth_fs.h"
const char *VERSION_QETH_SYS_C = "$Revision: 1.55 $"; const char *VERSION_QETH_SYS_C = "$Revision: 1.58 $";
/*****************************************************************************/ /*****************************************************************************/
/* */ /* */
...@@ -1117,7 +1117,7 @@ qeth_parse_ipatoe(const char* buf, enum qeth_prot_versions proto, ...@@ -1117,7 +1117,7 @@ qeth_parse_ipatoe(const char* buf, enum qeth_prot_versions proto,
start = buf; start = buf;
/* get address string */ /* get address string */
end = strchr(start, '/'); end = strchr(start, '/');
if (!end){ if (!end || (end-start >= 49)){
PRINT_WARN("Invalid format for ipato_addx/delx. " PRINT_WARN("Invalid format for ipato_addx/delx. "
"Use <ip addr>/<mask bits>\n"); "Use <ip addr>/<mask bits>\n");
return -EINVAL; return -EINVAL;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment