Received: by 2002:a05:6358:4e97:b0:b3:742d:4702 with SMTP id ce23csp3337349rwb; Tue, 16 Aug 2022 00:36:10 -0700 (PDT) X-Google-Smtp-Source: AA6agR7LpqSQdB9Qe6ghtBbRdXgS8WnyQ8lUc+ujYikdi14UI25F35gskZuH02k7hMSKQS2SATaP X-Received: by 2002:a17:907:9694:b0:731:8022:94f6 with SMTP id hd20-20020a170907969400b00731802294f6mr12640578ejc.398.1660635369918; Tue, 16 Aug 2022 00:36:09 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1660635369; cv=none; d=google.com; s=arc-20160816; b=BStFGUdw/PPH6NGIiPA3SZBixg5nd18p/wXe7WjVXf9uaoBElFt+Aj+twnqIdjtivF WAmEhncSpfjik8XOGkzaopVVvLFl080VhHFb+LmmDsV4MlFrDMhkS0NNQzBxAaH1b+Og B0dRpRo22rimw1PpYFCmUyN+IvA/Yfn6+rHiBjGX0nGkKBsVMta8++a4/EWf8g+sDxb2 +pgyP+n9AokTK4kaMw7W9u7sswLrgBzWNf+va1a3EyxoKT1bsJBR2kIaofJJxWDA8D4E f0LJpybKArMZbV5yyN1pfPvZ/jvpfdyOtkDP35aXb/Mh/l4LQTEkVGZSnj8/9P3z7B9F r3zw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:cc:to:subject :message-id:date:from:in-reply-to:references:mime-version :dkim-signature; bh=sXgeOW1LGi8MM4yVGp5zgGERRljNyGhH22BW5mob+Ls=; b=kcTJiAAVzPnLgP1rtUpqreZ0XXl/APke2axGbuxRg+64aM/u0XCfTV6LVAjPAbKfah jgmng/sXxjQluuiwXaQy1pZHcHztDaDMqV4Gs/NbVExf2KxiN/MGCCxeYec0hJ1vYJR7 +sCwolU8Zzi5z7lBm68to4vgfNFhSmP9zaYVwqDUwhRKqhzvmOIFl1Jmgjrxm/cerOvX /NMnRihlOh0uPHKkfQPKQE4DT6Q4E/llRofojnnH8NFbFMhg4RiHtRAh1wfWBJ74GrC8 n/ufF3nYmdceDk0quL8ZoqLH/8sfUvV9PNk8DfaTUNS7He2C17DwgruOyzM3z+hw1Dxf fZ9w== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@paul-moore-com.20210112.gappssmtp.com header.s=20210112 header.b=JjHhOaND; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id w11-20020a056402268b00b0043d82d2a5e9si11153461edd.29.2022.08.16.00.35.44; Tue, 16 Aug 2022 00:36:09 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@paul-moore-com.20210112.gappssmtp.com header.s=20210112 header.b=JjHhOaND; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231501AbiHPHCM (ORCPT + 99 others); Tue, 16 Aug 2022 03:02:12 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:50088 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231789AbiHPHBA (ORCPT ); Tue, 16 Aug 2022 03:01:00 -0400 Received: from mail-ot1-x332.google.com (mail-ot1-x332.google.com [IPv6:2607:f8b0:4864:20::332]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id B27451022A7 for ; Mon, 15 Aug 2022 19:27:17 -0700 (PDT) Received: by mail-ot1-x332.google.com with SMTP id a14-20020a0568300b8e00b0061c4e3eb52aso6628460otv.3 for ; Mon, 15 Aug 2022 19:27:17 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=paul-moore-com.20210112.gappssmtp.com; s=20210112; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:from:to:cc; bh=sXgeOW1LGi8MM4yVGp5zgGERRljNyGhH22BW5mob+Ls=; b=JjHhOaNDTBVJYBoaLj81WqLdQyGVp5S8sYO1N6gRxRWu9jMc6MatSBEeTcdhO5Q8Oj QCnS+iibiRCKrXHSsre9viWVJSdSTZjrYBB9vbWgywMczgWCI9HpHvv6AWyWr4KaxS5U TVNLpqqB/XvgfoPv96x1mJ35UOIEQx0jZhTvl9Km8DVPopdG1foUeFKCs0vldwM+OBXL IP0lFNv1hWA3zH1keduPNBVRZcGSQgmtbmmMEV+aIDSMRGT3mftZMlvHrT1s9YicyqsO X51vy30swk8Wxm+JZ0/5AzvdOTJiJoRUxQxsEpZVlIPT42uakjTNuf+St3S1k67XPDZI KEAg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc; bh=sXgeOW1LGi8MM4yVGp5zgGERRljNyGhH22BW5mob+Ls=; b=jbehlPhBxbEE0HIuVmwrmlzm+6nrxYZUwyDyWXFSP7ZPDiYpl4fU/gj3if0DTAAFkY FGmVRN9FVJCw5PTN8xC7gNh1FUdDEIWZuYuMExvZhBZQsYbN7inVuQmt7f9QSU4Lnt6I 1z/GV6j6BHjqszYX7/PLhFsGikTADVbqgQjpIhyPp4UYixqXBciYvkNFuOd3kV2a5CJk wBdtyiQluFR2PQ3tWKtDDRUPwJJ95RKXHrVW8Ni114PSw1J1vosUGPJaco5X9vnl3G6H BRAaNtYKd3BxKADlvfNKGtHjNpKoogriPXdd4+5v3kjoPaHr4YWwkhYy/jcj+XOC0vcG 6Muw== X-Gm-Message-State: ACgBeo0iqn55TE+tn2KF0xtVkCU4ff0J9xnF1u0RDJxrMMjaJHa2mq9k Ui4OYcCgUGBcshAr0PzD0L77xva6klFRcbXABR8M X-Received: by 2002:a05:6830:33e4:b0:636:732d:5a48 with SMTP id i4-20020a05683033e400b00636732d5a48mr6937750otu.69.1660616837000; Mon, 15 Aug 2022 19:27:17 -0700 (PDT) MIME-Version: 1.0 References: <20220714012421.55627-1-xiujianfeng@huawei.com> In-Reply-To: From: Paul Moore Date: Mon, 15 Aug 2022 22:27:06 -0400 Message-ID: Subject: Re: [PATCH RESEND -next] lsm_audit: Clean up redundant NULL pointer check To: xiujianfeng Cc: jmorris@namei.org, serge@hallyn.com, linux-security-module@vger.kernel.org, linux-kernel@vger.kernel.org Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_NONE, T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Jul 13, 2022 at 10:49 PM xiujianfeng wrote= : > =E5=9C=A8 2022/7/14 10:33, Paul Moore =E5=86=99=E9=81=93: > > On Wed, Jul 13, 2022 at 9:27 PM Xiu Jianfeng w= rote: > >> The implements of {ip,tcp,udp,dccp,sctp,ipv6}_hdr(skb) guarantee that > >> they will never return NULL, and elsewhere users don't do the check > >> as well, so remove the check here. > >> > >> Signed-off-by: Xiu Jianfeng > >> --- > >> security/lsm_audit.c | 14 +------------- > >> 1 file changed, 1 insertion(+), 13 deletions(-) > > Hi Xiu Jianfeng, > > > > We just changed LSM maintainers earlier this week, and while I haven't > > forgotten about your patch (I reviewed it previously), it is too late > > in this current release cycle (-rc6) to merge non-critical fixes. > > This would go into the LSM tree after the upcoming merge window. > > > > Thank you for your patience and understanding. > > Hi, paul, I get it, thanks very much :) I just merged this into the lsm/next branch, thanks for your patience. --=20 paul-moore.com