matthias
/
iproute2
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

tc: fix bugs for tcp_flags and ip_attr hex output 

Fix hex output for both the ip_attr and tcp_flags print functions.

Sample usage:

$ $TC qdisc add dev lo ingress
$ $TC filter add dev lo parent ffff: prio 3 proto ip flower ip_tos 0x8/32
$ $TC fitler add dev lo parent ffff: prio 5 proto ip flower ip_proto tcp \
	tcp_flags 0x909/f00

$ $TC filter show dev lo parent ffff:

filter protocol ip pref 3 flower chain 0
filter protocol ip pref 3 flower chain 0 handle 0x1
  eth_type ipv4
  ip_tos 0x8/32
  not_in_hw
filter protocol ip pref 5 flower chain 0
filter protocol ip pref 5 flower chain 0 handle 0x1
  eth_type ipv4
  ip_proto tcp
  tcp_flags 0x909/f00
  not_in_hw

$ $TC -j filter show dev lo parent ffff:

[{
    "protocol":"ip",
    "pref":3,
    "kind":"flower",
    "chain":0
},{
    "protocol":"ip",
    "pref":3,
    "kind":"flower",
    "chain":0,
    "options": {
	"handle":1,
	"keys": {
	    "eth_type":"ipv4",
	    "ip_tos":"0x8/32"
    },
    "not_in_hw":true
    }
},{
    "protocol":"ip",
    "pref":5,
    "kind":"flower",
    "chain":0
},{
    "protocol":"ip",
    "pref":5,
    "kind":"flower",
    "chain":0,
    "options": {
	"handle":1,
	"keys": {
	    "eth_type":"ipv4",
	    "ip_proto":"tcp",
	    "tcp_flags":"0x909/f00"
	},
	"not_in_hw":true
    }
}]

Signed-off-by: Keara Leibovitz <kleib@mojatatu.com>
Signed-off-by: David Ahern <dsahern@gmail.com>
This commit is contained in:
Keara Leibovitz 2018-07-26 09:45:30 -04:00 committed by David Ahern
parent a0bc57e1ef
commit e8bd395508
1 changed files with 9 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
tc/f_flower.c
View File

@ -1189,7 +1189,7 @@ static void flower_print_ip_proto(__u8 *p_ip_proto,
*p_ip_proto = ip_proto; *p_ip_proto = ip_proto;
} }
static void flower_print_ip_attr(char *name, struct rtattr *key_attr, static void flower_print_ip_attr(const char *name, struct rtattr *key_attr,
struct rtattr *mask_attr) struct rtattr *mask_attr)
{ {
SPRINT_BUF(namefrm); SPRINT_BUF(namefrm);
@ -1199,11 +1199,12 @@ static void flower_print_ip_attr(char *name, struct rtattr *key_attr,
if (!key_attr) if (!key_attr)
return; return;
done = sprintf(out, "%x", rta_getattr_u8(key_attr)); done = sprintf(out, "0x%x", rta_getattr_u8(key_attr));
if (mask_attr) if (mask_attr)
sprintf(out + done, "/%x", rta_getattr_u8(mask_attr)); sprintf(out + done, "/%x", rta_getattr_u8(mask_attr));
sprintf(namefrm, "\n %s %%s", name); print_string(PRINT_FP, NULL, "%s ", _SL_);
sprintf(namefrm, "%s %%s", name);
print_string(PRINT_ANY, name, namefrm, out); print_string(PRINT_ANY, name, namefrm, out);
} }
@ -1308,7 +1309,7 @@ static void flower_print_port(char *name, struct rtattr *attr)
print_hu(PRINT_ANY, name, namefrm, rta_getattr_be16(attr)); print_hu(PRINT_ANY, name, namefrm, rta_getattr_be16(attr));
} }
static void flower_print_tcp_flags(char *name, struct rtattr *flags_attr, static void flower_print_tcp_flags(const char *name, struct rtattr *flags_attr,
struct rtattr *mask_attr) struct rtattr *mask_attr)
{ {
SPRINT_BUF(namefrm); SPRINT_BUF(namefrm);
@ -1318,11 +1319,12 @@ static void flower_print_tcp_flags(char *name, struct rtattr *flags_attr,
if (!flags_attr) if (!flags_attr)
return; return;
done = sprintf(out, "%x", rta_getattr_be16(flags_attr)); done = sprintf(out, "0x%x", rta_getattr_be16(flags_attr));
if (mask_attr) if (mask_attr)
sprintf(out + done, "%x", rta_getattr_be16(flags_attr)); sprintf(out + done, "/%x", rta_getattr_be16(mask_attr));
sprintf(namefrm, "\n %s %%s", name); print_string(PRINT_FP, NULL, "%s ", _SL_);
sprintf(namefrm, "%s %%s", name);
print_string(PRINT_ANY, name, namefrm, out); print_string(PRINT_ANY, name, namefrm, out);
} }