diff options
author | Bernhard Reutner-Fischer <rep.dot.nop@gmail.com> | 2008-05-16 16:55:17 +0000 |
---|---|---|
committer | Bernhard Reutner-Fischer <rep.dot.nop@gmail.com> | 2008-05-16 16:55:17 +0000 |
commit | b8635e28b521ce0f2c016fd16a716c4b43bd13e7 (patch) | |
tree | 6f6c8eb72b72619bc2f77355611c4fc1fb7207b1 | |
parent | dff6190112e078d9c524f178ca7769ab9888229a (diff) | |
download | busybox-w32-b8635e28b521ce0f2c016fd16a716c4b43bd13e7.tar.gz busybox-w32-b8635e28b521ce0f2c016fd16a716c4b43bd13e7.tar.bz2 busybox-w32-b8635e28b521ce0f2c016fd16a716c4b43bd13e7.zip |
- very minor shrinkage
odd that this saved 1byte. And there could be something wrong with
ifcombine or the like since omitting the braces produces bigger code?
-rw-r--r-- | networking/libiproute/ll_addr.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/networking/libiproute/ll_addr.c b/networking/libiproute/ll_addr.c index ab5a2c522..e732efdb2 100644 --- a/networking/libiproute/ll_addr.c +++ b/networking/libiproute/ll_addr.c | |||
@@ -64,11 +64,7 @@ int ll_addr_a2n(unsigned char *lladdr, int len, char *arg) | |||
64 | *cp = 0; | 64 | *cp = 0; |
65 | cp++; | 65 | cp++; |
66 | } | 66 | } |
67 | if (sscanf(arg, "%x", &temp) != 1) { | 67 | if (sscanf(arg, "%x", &temp) != 1 || (temp < 0 || temp > 255)) { |
68 | bb_error_msg("\"%s\" is invalid lladdr", arg); | ||
69 | return -1; | ||
70 | } | ||
71 | if (temp < 0 || temp > 255) { | ||
72 | bb_error_msg("\"%s\" is invalid lladdr", arg); | 68 | bb_error_msg("\"%s\" is invalid lladdr", arg); |
73 | return -1; | 69 | return -1; |
74 | } | 70 | } |