diff options
author | Florian Westphal <fw@strlen.de> | 2016-08-01 17:11:41 +0200 |
---|---|---|
committer | Florian Westphal <fw@strlen.de> | 2016-08-01 17:11:41 +0200 |
commit | ca6fd0a94fc2d2c6d662b76c22ffaa4f3ffc7a1f (patch) | |
tree | 9f781495b0c96599ac4cb3c886013f1ed607d066 /src/payload.c | |
parent | a0ed637e76da3fe44eb34eb2ab91d72be2365411 (diff) |
payload: print base and raw values for unknown payloads
We currently print 'unknown' rather than the raw offset values
for unrecognized header values.
If its unknown, prefer to print
payload @nh,0,16 set payload @nh,0,16
rather than 'unknown'.
Also add a helper to check if payload expression has a description
assigned to it.
Signed-off-by: Florian Westphal <fw@strlen.de>
Acked-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'src/payload.c')
-rw-r--r-- | src/payload.c | 14 |
1 files changed, 13 insertions, 1 deletions
diff --git a/src/payload.c b/src/payload.c index 9ba980a5..af533b2a 100644 --- a/src/payload.c +++ b/src/payload.c @@ -26,6 +26,18 @@ #include <gmputil.h> #include <utils.h> +bool payload_is_known(const struct expr *expr) +{ + const struct proto_hdr_template *tmpl; + const struct proto_desc *desc; + + desc = expr->payload.desc; + tmpl = expr->payload.tmpl; + + return desc && tmpl && desc != &proto_unknown && + tmpl != &proto_unknown_template; +} + static void payload_expr_print(const struct expr *expr) { const struct proto_desc *desc; @@ -33,7 +45,7 @@ static void payload_expr_print(const struct expr *expr) desc = expr->payload.desc; tmpl = expr->payload.tmpl; - if (desc != NULL && tmpl != NULL) + if (payload_is_known(expr)) printf("%s %s", desc->name, tmpl->token); else printf("payload @%s,%u,%u", |