diff options
author | Florian Westphal <fw@strlen.de> | 2018-04-13 19:19:57 +0200 |
---|---|---|
committer | Florian Westphal <fw@strlen.de> | 2018-04-13 20:05:51 +0200 |
commit | 394a40030332568c8654d6960aa6f851bac9149d (patch) | |
tree | 396aa84d044767a7d9117af87bc9a7b23381777a /iptables/nft-ipv6.c | |
parent | 6a1dbdfb8c6aa5cfe24db2e5b2314c11c3d8e2df (diff) |
nft: fix crash when getprotobynumber() returns 0
This needs to first check if pent is NULL.
Signed-off-by: Florian Westphal <fw@strlen.de>
Diffstat (limited to 'iptables/nft-ipv6.c')
-rw-r--r-- | iptables/nft-ipv6.c | 15 |
1 files changed, 9 insertions, 6 deletions
diff --git a/iptables/nft-ipv6.c b/iptables/nft-ipv6.c index 1be49dd3..af525422 100644 --- a/iptables/nft-ipv6.c +++ b/iptables/nft-ipv6.c @@ -417,16 +417,19 @@ static int nft_ipv6_xlate(const void *data, struct xt_xlate *xl) const struct protoent *pent = getprotobynumber(cs->fw6.ipv6.proto); char protonum[sizeof("65535")]; + const char *name = protonum; - if (!xlate_find_match(cs, pent->p_name)) { - snprintf(protonum, sizeof(protonum), "%u", - cs->fw6.ipv6.proto); - protonum[sizeof(protonum) - 1] = '\0'; + snprintf(protonum, sizeof(protonum), "%u", + cs->fw6.ipv6.proto); + + if (!pent || !xlate_find_match(cs, pent->p_name)) { + if (pent) + name = pent->p_name; xt_xlate_add(xl, "meta l4proto %s%s ", cs->fw6.ipv6.invflags & IP6T_INV_PROTO ? - "!= " : "", - pent ? pent->p_name : protonum); + "!= " : "", name); } + } xlate_ipv6_addr("ip6 saddr", &cs->fw6.ipv6.src, &cs->fw6.ipv6.smsk, |