iproute: fix handling of "dev IFACE" selector

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
This commit is contained in:
Denys Vlasenko 2011-02-09 04:39:09 +01:00
parent e7212a4ce5
commit f133471302
2 changed files with 25 additions and 14 deletions

View File

@ -31,8 +31,8 @@ struct filter_t {
//int type; - read-only //int type; - read-only
//int typemask; - unused //int typemask; - unused
//int tos, tosmask; - unused //int tos, tosmask; - unused
int iif, iifmask; int iif;
int oif, oifmask; int oif;
//int realm, realmmask; - unused //int realm, realmmask; - unused
//inet_prefix rprefsrc; - read-only //inet_prefix rprefsrc; - read-only
inet_prefix rvia; inet_prefix rvia;
@ -182,17 +182,25 @@ static int FAST_FUNC print_route(const struct sockaddr_nl *who UNUSED_PARAM,
) { ) {
return 0; return 0;
} }
if (G_filter.flushb if (G_filter.oif != 0) {
&& r->rtm_family == AF_INET6 if (!tb[RTA_OIF])
&& r->rtm_dst_len == 0 return 0;
&& r->rtm_type == RTN_UNREACHABLE if (G_filter.oif != *(int*)RTA_DATA(tb[RTA_OIF]))
&& tb[RTA_PRIORITY] return 0;
&& *(int*)RTA_DATA(tb[RTA_PRIORITY]) == -1
) {
return 0;
} }
if (G_filter.flushb) { if (G_filter.flushb) {
/* We are creating route flush commands */
if (r->rtm_family == AF_INET6
&& r->rtm_dst_len == 0
&& r->rtm_type == RTN_UNREACHABLE
&& tb[RTA_PRIORITY]
&& *(int*)RTA_DATA(tb[RTA_PRIORITY]) == -1
) {
return 0;
}
struct nlmsghdr *fn; struct nlmsghdr *fn;
if (NLMSG_ALIGN(G_filter.flushp) + n->nlmsg_len > G_filter.flushe) { if (NLMSG_ALIGN(G_filter.flushp) + n->nlmsg_len > G_filter.flushe) {
if (flush_update()) if (flush_update())
@ -208,6 +216,8 @@ static int FAST_FUNC print_route(const struct sockaddr_nl *who UNUSED_PARAM,
return 0; return 0;
} }
/* We are printing routes */
if (n->nlmsg_type == RTM_DELROUTE) { if (n->nlmsg_type == RTM_DELROUTE) {
printf("Deleted "); printf("Deleted ");
} }
@ -257,10 +267,12 @@ static int FAST_FUNC print_route(const struct sockaddr_nl *who UNUSED_PARAM,
RTA_DATA(tb[RTA_GATEWAY]), RTA_DATA(tb[RTA_GATEWAY]),
abuf, sizeof(abuf))); abuf, sizeof(abuf)));
} }
if (tb[RTA_OIF] && G_filter.oifmask != -1) { if (tb[RTA_OIF]) {
printf("dev %s ", ll_index_to_name(*(int*)RTA_DATA(tb[RTA_OIF]))); printf("dev %s ", ll_index_to_name(*(int*)RTA_DATA(tb[RTA_OIF])));
} }
/* Todo: parse & show "proto kernel", "scope link" here */
if (tb[RTA_PREFSRC] && /*G_filter.rprefsrc.bitlen - always 0*/ 0 != host_len) { if (tb[RTA_PREFSRC] && /*G_filter.rprefsrc.bitlen - always 0*/ 0 != host_len) {
/* Do not use format_host(). It is our local addr /* Do not use format_host(). It is our local addr
and symbolic name will not be useful. and symbolic name will not be useful.
@ -292,7 +304,7 @@ static int FAST_FUNC print_route(const struct sockaddr_nl *who UNUSED_PARAM,
printf(" error %d", ci->rta_error); printf(" error %d", ci->rta_error);
} }
} }
if (tb[RTA_IIF] && G_filter.iifmask != -1) { if (tb[RTA_IIF] && G_filter.iif == 0) {
printf(" iif %s", ll_index_to_name(*(int*)RTA_DATA(tb[RTA_IIF]))); printf(" iif %s", ll_index_to_name(*(int*)RTA_DATA(tb[RTA_IIF])));
} }
bb_putchar('\n'); bb_putchar('\n');
@ -662,12 +674,10 @@ static int iproute_list_or_flush(char **argv, int flush)
if (id) { if (id) {
idx = xll_name_to_index(id); idx = xll_name_to_index(id);
G_filter.iif = idx; G_filter.iif = idx;
G_filter.iifmask = -1;
} }
if (od) { if (od) {
idx = xll_name_to_index(od); idx = xll_name_to_index(od);
G_filter.oif = idx; G_filter.oif = idx;
G_filter.oifmask = -1;
} }
} }

View File

@ -55,6 +55,7 @@ int FAST_FUNC xrtnl_wilddump_request(struct rtnl_handle *rth, int family, int ty
return rtnl_send(rth, (void*)&req, sizeof(req)); return rtnl_send(rth, (void*)&req, sizeof(req));
} }
//TODO: pass rth->fd instead of full rth?
int FAST_FUNC rtnl_send(struct rtnl_handle *rth, char *buf, int len) int FAST_FUNC rtnl_send(struct rtnl_handle *rth, char *buf, int len)
{ {
struct sockaddr_nl nladdr; struct sockaddr_nl nladdr;