From 473744b5057d711c965a7dec459d687144490c23 Mon Sep 17 00:00:00 2001 From: maxv Date: Mon, 12 Mar 2018 12:45:26 +0000 Subject: [PATCH] Remove dead branches, 'npc' can't be NULL (and it is dereferenced earlier). --- sys/net/npf/npf_alg_icmp.c | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/sys/net/npf/npf_alg_icmp.c b/sys/net/npf/npf_alg_icmp.c index efd0f0eb4427..2256640401de 100644 --- a/sys/net/npf/npf_alg_icmp.c +++ b/sys/net/npf/npf_alg_icmp.c @@ -1,4 +1,4 @@ -/* $NetBSD: npf_alg_icmp.c,v 1.25 2017/12/10 00:07:36 rmind Exp $ */ +/* $NetBSD: npf_alg_icmp.c,v 1.26 2018/03/12 12:45:26 maxv Exp $ */ /*- * Copyright (c) 2010 The NetBSD Foundation, Inc. @@ -35,7 +35,7 @@ #ifdef _KERNEL #include -__KERNEL_RCSID(0, "$NetBSD: npf_alg_icmp.c,v 1.25 2017/12/10 00:07:36 rmind Exp $"); +__KERNEL_RCSID(0, "$NetBSD: npf_alg_icmp.c,v 1.26 2018/03/12 12:45:26 maxv Exp $"); #include #include @@ -135,9 +135,6 @@ npfa_icmp4_inspect(const int type, npf_cache_t *npc) case ICMP_REDIRECT: case ICMP_TIMXCEED: case ICMP_PARAMPROB: - if (npc == NULL) { - return false; - } /* Should contain original IP header. */ if (!nbuf_advance(nbuf, offsetof(struct icmp, icmp_ip), 0)) { return false; @@ -175,9 +172,6 @@ npfa_icmp6_inspect(const int type, npf_cache_t *npc) case ICMP6_PACKET_TOO_BIG: case ICMP6_TIME_EXCEEDED: case ICMP6_PARAM_PROB: - if (npc == NULL) { - return false; - } /* Should contain original IP header. */ if (!nbuf_advance(nbuf, sizeof(struct icmp6_hdr), 0)) { return false;