Received: by 2002:a05:6902:102b:0:0:0:0 with SMTP id x11csp1083142ybt; Wed, 24 Jun 2020 19:42:44 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxZQ4cSu/zjMOObDHckM1pViD/krSWM28rejfkZh/SwyPEA8+k7z4FIXG4gpXTJTvYEnKKL X-Received: by 2002:a50:f1d9:: with SMTP id y25mr28644444edl.292.1593052964466; Wed, 24 Jun 2020 19:42:44 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1593052964; cv=none; d=google.com; s=arc-20160816; b=eRnCaqUccltpS+rT7N0bLFX4UtW8iCDsQUHuxyC78BllleVbXlNcmursE2jq0hrY6B 7iM5VKXzJoxQPsYXbvwQvR7VNGr1hEBlFDMDX0Ha0GJmr+DTaWvzVQ17NcjGWhU+VWFo m3vyvOdW2UllZvXEnWyL0z7J0S9m0UG+jVEprt4c3GWzHOBh2dMLmCRUixIch1HwCfqe tnkYh9poXNmmHsykQ7M89FCGytAPGDkdROYanawsPyubEeAH5yzFGnlpRTyL8EH8gNFj vQMFa5H4kXougjDs5126WTjR60iFQXTTDVIutDOzTA6/4Z1PAIWQudAtr0hJCKglsX2W HSmg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:message-id:date:subject:to:from :dkim-signature; bh=fbnRB4YoedUi6LoLD8JlUhcVfnMSczit3rkXQlpuCkk=; b=CJA5OdbDLaTosf93PNpv6djqY0m8C9wSzh51EGMumnmGwcrmC5CbNJM7NB3AWGQGen Vp+HLF8HIkg0gh07/luC2fGBDHxPPdZvKy4uZq46d4AHYQUkGm1jHKyGsnJ+LxyeEwcl 07wdp9gbFm4e45nk0oYVCLbDAoT/9bgfB/xPZ1ec9jKhrKsJ1MMPL4wl3pn+IzZHH27w b8p6YPvDsxjvd+tGxD4jDRr8yYlRgOhMC14AAxDKjRrDIRwSG/0VGhy+qcCiPs5KH+sN r/tF3JSQQSQ/1lvl+NhCpc8vJdDrkhxcTNc43MOH/uU76SjFajTP17ldxFLNDFUXIaZG Zdhg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=kEXvyUMc; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id e17si4227652ejk.740.2020.06.24.19.42.19; Wed, 24 Jun 2020 19:42:44 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=kEXvyUMc; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2389456AbgFYCmP (ORCPT + 99 others); Wed, 24 Jun 2020 22:42:15 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:51030 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2388930AbgFYCmO (ORCPT ); Wed, 24 Jun 2020 22:42:14 -0400 Received: from mail-qt1-x835.google.com (mail-qt1-x835.google.com [IPv6:2607:f8b0:4864:20::835]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 1ACE4C061573; Wed, 24 Jun 2020 19:42:14 -0700 (PDT) Received: by mail-qt1-x835.google.com with SMTP id g13so3521133qtv.8; Wed, 24 Jun 2020 19:42:14 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:subject:date:message-id; bh=fbnRB4YoedUi6LoLD8JlUhcVfnMSczit3rkXQlpuCkk=; b=kEXvyUMcDZ9wfp+GNFnO7Kdy0+8rFA8576vLxYjOiNP0e2JVWJ+dy80UGug5pUWJYV Zh4UG/Suq3ekYULrhscG84+fpLM5gICOlmQRCpkCLyObFPu2rrsGSf92EXXD8GocGX/c +UzOWtEs1PoLcU+tSBNnMTl87FUYJaqcuIDSrWBeRKYqSofD4J8lgALECQrRrBxoBR68 8E8xep34JgXlCrcCbE2IiaNElDnvpe6gIe29lwOP4A9ZWR0qAfq+huVnU/ftYyE2i1lf cxHN1qvjB273LYnDQMY6ntmqVU9GYFA1cbuR9izWL8NeA1HcUVTfKbRns30z7vaAYX2z 4ayw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:subject:date:message-id; bh=fbnRB4YoedUi6LoLD8JlUhcVfnMSczit3rkXQlpuCkk=; b=JEtAbcbH19ZlICmkbQgeBlY01Qojlsu7+twvMODyLvZY7CS6+Gpj6R9ONBxf20EOvn l6BRAJepYpl/FkPj59lpWDZdO0Yy/9KXrLMnBmzCMoA6lwc4e/n/BeWs00vvi0iiM6Ux 4SNHfgzUPHsDFnWmctNsJFhPMCWyutPYiizxCFnLaWv/e9hdmfVcMmcIDoLWTFkWgF6Z G/uavR263ieySMB3Jgx/FyEjK90KXF+A1A218N4qY4yaDmFygG+ClQl6VZF4c5UHb+pv a9xsP2yfl3JXJyCzT5cZXr1Mle2kOQhyut6c1E95d/ZmXYRjyoCmKc8AyFs4u5Vuv60B IVOA== X-Gm-Message-State: AOAM5322eEirxp8wsWFWayKJC2FztKaefYU2ji7aYbeVEj9eqGmADtzG KWXFhfFlO5r6ucQG4xzfRv8= X-Received: by 2002:ac8:2aa9:: with SMTP id b38mr11608952qta.49.1593052933353; Wed, 24 Jun 2020 19:42:13 -0700 (PDT) Received: from linux.home ([2604:2000:1344:41d:b4f5:b2c7:27bb:8a39]) by smtp.googlemail.com with ESMTPSA id n63sm4890048qkn.104.2020.06.24.19.42.12 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 24 Jun 2020 19:42:12 -0700 (PDT) From: Gaurav Singh To: gaurav1086@gmail.com, Pablo Neira Ayuso , Jozsef Kadlecsik , Florian Westphal , "David S. Miller" , Alexey Kuznetsov , Hideaki YOSHIFUJI , Jakub Kicinski , netfilter-devel@vger.kernel.org (open list:NETFILTER), coreteam@netfilter.org (open list:NETFILTER), netdev@vger.kernel.org (open list:NETWORKING [IPv4/IPv6]), linux-kernel@vger.kernel.org (open list) Subject: [PATCH] [net/ipv6] remove redundant null check in frag_mt6 Date: Wed, 24 Jun 2020 22:42:05 -0400 Message-Id: <20200625024207.1625-1-gaurav1086@gmail.com> X-Mailer: git-send-email 2.17.1 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org fh cannot be NULL since its already checked above after assignment and is being dereferenced before. Remove the redundant null check. Signed-off-by: Gaurav Singh --- net/ipv6/netfilter/ip6t_frag.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/net/ipv6/netfilter/ip6t_frag.c b/net/ipv6/netfilter/ip6t_frag.c index fb91eeee4a1e..3aad6439386b 100644 --- a/net/ipv6/netfilter/ip6t_frag.c +++ b/net/ipv6/netfilter/ip6t_frag.c @@ -85,8 +85,7 @@ frag_mt6(const struct sk_buff *skb, struct xt_action_param *par) !((fraginfo->flags & IP6T_FRAG_NMF) && (ntohs(fh->frag_off) & IP6_MF))); - return (fh != NULL) && - id_match(fraginfo->ids[0], fraginfo->ids[1], + return id_match(fraginfo->ids[0], fraginfo->ids[1], ntohl(fh->identification), !!(fraginfo->invflags & IP6T_FRAG_INV_IDS)) && !((fraginfo->flags & IP6T_FRAG_RES) && -- 2.17.1