diff options
| author | Florian Westphal <fw@strlen.de> | 2021-02-08 15:54:44 +0100 |
|---|---|---|
| committer | Florian Westphal <fw@strlen.de> | 2021-02-08 16:06:09 +0100 |
| commit | 5b1fb140a0db0e14d036face850dad3a4e209901 (patch) | |
| tree | adb0924def1d553d7e08580af5826fa379a55862 | |
| parent | e6c32b2fa0b820bc81cbb99e8ed601eabbbfac69 (diff) | |
trace: do not remove icmp type from packet dump
As of 0.9.8 the icmp type is marked as a protocol field, so its
elided in 'nft monitor trace' output:
icmp code 0 icmp id 44380 ..
Restore it. Unlike tcp, where 'tcp sport' et. al in the dump
will make the 'ip protocol tcp' redundant this case isn't obvious
in the icmp case:
icmp type 8 code 0 id ...
Reported-by: Martin Gignac <martin.gignac@gmail.com>
Fixes: 98b871512c4677 ("src: add auto-dependencies for ipv4 icmp")
Signed-off-by: Florian Westphal <fw@strlen.de>
| -rw-r--r-- | src/netlink.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/netlink.c b/src/netlink.c index ec2dad29..c3887d5b 100644 --- a/src/netlink.c +++ b/src/netlink.c @@ -1859,7 +1859,9 @@ next: pctx->pbase == PROTO_BASE_INVALID) { payload_dependency_store(pctx, stmt, base - stacked); } else { - payload_dependency_kill(pctx, lhs, ctx->family); + /* Don't strip 'icmp type' from payload dump. */ + if (pctx->icmp_type == 0) + payload_dependency_kill(pctx, lhs, ctx->family); if (lhs->flags & EXPR_F_PROTOCOL) payload_dependency_store(pctx, stmt, base - stacked); } |