diff options
author | Denys Vlasenko <vda.linux@googlemail.com> | 2010-07-04 05:55:43 +0400 |
---|---|---|
committer | Denys Vlasenko <vda.linux@googlemail.com> | 2010-07-04 05:55:43 +0400 |
commit | fd744519d1e848c4760c23805bf993396b19c8b2 (patch) | |
tree | eff0905b9d1cc1fdac330c88ee988c53efdb2db7 /networking/libiproute/ipaddress.c | |
parent | 21e8e8da6483c80a6054b06e48341968a7dccdd5 (diff) |
ip a: fix SEGV if "dev IFACE" is not specified
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'networking/libiproute/ipaddress.c')
-rw-r--r-- | networking/libiproute/ipaddress.c | 47 |
1 files changed, 23 insertions, 24 deletions
diff --git a/networking/libiproute/ipaddress.c b/networking/libiproute/ipaddress.c index a603053e1..381293412 100644 --- a/networking/libiproute/ipaddress.c +++ b/networking/libiproute/ipaddress.c @@ -444,28 +444,28 @@ int ipaddr_list_or_flush(char **argv, int flush) while (*argv) { const smalluint key = index_in_strings(option, *argv); if (key == 0) { /* to */ - NEXT_ARG(); - get_prefix(&G_filter.pfx, *argv, G_filter.family); - if (G_filter.family == AF_UNSPEC) { - G_filter.family = G_filter.pfx.family; - } + NEXT_ARG(); + get_prefix(&G_filter.pfx, *argv, G_filter.family); + if (G_filter.family == AF_UNSPEC) { + G_filter.family = G_filter.pfx.family; + } } else if (key == 1) { /* scope */ - uint32_t scope = 0; - NEXT_ARG(); - G_filter.scopemask = -1; - if (rtnl_rtscope_a2n(&scope, *argv)) { - if (strcmp(*argv, "all") != 0) { - invarg(*argv, "scope"); - } - scope = RT_SCOPE_NOWHERE; - G_filter.scopemask = 0; + uint32_t scope = 0; + NEXT_ARG(); + G_filter.scopemask = -1; + if (rtnl_rtscope_a2n(&scope, *argv)) { + if (strcmp(*argv, "all") != 0) { + invarg(*argv, "scope"); } - G_filter.scope = scope; + scope = RT_SCOPE_NOWHERE; + G_filter.scopemask = 0; + } + G_filter.scope = scope; } else if (key == 2) { /* up */ - G_filter.up = 1; + G_filter.up = 1; } else if (key == 3) { /* label */ - NEXT_ARG(); - G_filter.label = *argv; + NEXT_ARG(); + G_filter.label = *argv; } else { if (key == 4) /* dev */ NEXT_ARG(); @@ -681,7 +681,7 @@ static int ipaddr_modify(int cmd, char **argv) } else if (arg == 7) { /* label */ NEXT_ARG(); l = *argv; - addattr_l(&req.n, sizeof(req), IFA_LABEL, l, strlen(l)+1); + addattr_l(&req.n, sizeof(req), IFA_LABEL, l, strlen(l) + 1); } else { if (arg == 8) /* local */ NEXT_ARG(); @@ -698,11 +698,10 @@ static int ipaddr_modify(int cmd, char **argv) argv++; } - // d cannot be null here, NEXT_ARG() of "dev" ensures that - //if (d == NULL) { - // bb_error_msg(bb_msg_requires_arg, "\"dev\""); - // return -1; - //} + if (!d) { + /* There was no "dev IFACE", but we need that */ + bb_error_msg_and_die("need \"dev IFACE\""); + } if (l && strncmp(d, l, strlen(d)) != 0) { bb_error_msg_and_die("\"dev\" (%s) must match \"label\" (%s)", d, l); } |