Received: by 2002:a25:4158:0:0:0:0:0 with SMTP id o85csp4113589yba; Tue, 7 May 2019 12:19:07 -0700 (PDT) X-Google-Smtp-Source: APXvYqxxRO4LFzyvO9vHZwg2JgFazXUNoWE9JHav2Bjwv2jXKXp5N7o/yuu97p/N11sSEqG2/BoN X-Received: by 2002:aa7:928b:: with SMTP id j11mr43395298pfa.200.1557256747263; Tue, 07 May 2019 12:19:07 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1557256747; cv=none; d=google.com; s=arc-20160816; b=BLTga3ovy53wQfR6EQIUefHw87UhPMd+e9AtlAQeXnDMWQWEamkwlBzKYcEuHYsVK9 MmLh9K8H4xr4U8jJY/L5U27uRP4xFhY+fMPvLQ/l+p5OIiPoJsDFLtz8l2lFwlWQRGfQ zv+cfL9ddyvyE0VmGJu+6+KybyuC8yvHftDM52y8pzims/hjQBrftqBYW1FvyBkRobVO VNVOMxztCuF2XART7UtPstOqfNZbsiKX8O3MxaXiOL9lwTBkYPB2Q2Opl6a1737fld75 3w1mvne/0CBJ1O4aUzNMU5/qGc5hjymYWBnCZ43EnAHQDYR6gcFnp153qqCRO2ooJcE3 B9Wg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :references:in-reply-to:from:subject:cc:to:message-id:date; bh=8gM5F8bAiaPHxR5oyE9/BST0ueeNhze4cDRD98i1f6Y=; b=kG4XGRlJjsPyBP/AYZGk18jYWt0OuZdiF7BwaTr+KILz+IbXsOUFNP+ZDHhe4TzQAf I+3dsdLow5fuSBsA1xd3w+U5ALPGjcY15UKx11j7b6e5T9L9zRlRzm709TOFYod45iKe KqBSVYpBhlVau2Uj2vBRoqItApHvRHYFC8LM/HmnfJQcUWvlDDzFSQTChZwnGEpL6g9A M2TqmgUVLkSID0NAQE4U8WWXXMG14hzFe1z6fOLt4kgQWfhhcz99fR00XG2XZ96FaCPt Ut7fOKPWIDZqTEYFncltI8mbkc+mkYMDuDPb4UruzVAYg4g4SPtBVR0sx9s397q3GDkb KwEw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id s16si20423217plr.369.2019.05.07.12.18.51; Tue, 07 May 2019 12:19:07 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726420AbfEGTSB (ORCPT + 99 others); Tue, 7 May 2019 15:18:01 -0400 Received: from shards.monkeyblade.net ([23.128.96.9]:33084 "EHLO shards.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725843AbfEGTSA (ORCPT ); Tue, 7 May 2019 15:18:00 -0400 Received: from localhost (unknown [IPv6:2601:601:9f80:35cd::3d8]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (Client did not present a certificate) (Authenticated sender: davem-davemloft) by shards.monkeyblade.net (Postfix) with ESMTPSA id 6C1A714B56316; Tue, 7 May 2019 12:17:59 -0700 (PDT) Date: Tue, 07 May 2019 12:17:58 -0700 (PDT) Message-Id: <20190507.121758.1374598572835036372.davem@davemloft.net> To: yuehaibing@huawei.com Cc: g.nault@alphalink.fr, jian.w.wen@oracle.com, edumazet@google.com, kafai@fb.com, xiyou.wangcong@gmail.com, linux-kernel@vger.kernel.org, netdev@vger.kernel.org Subject: Re: [PATCH] l2tp: Fix possible NULL pointer dereference From: David Miller In-Reply-To: <20190506144404.25220-1-yuehaibing@huawei.com> References: <20190506144404.25220-1-yuehaibing@huawei.com> X-Mailer: Mew version 6.8 on Emacs 26.1 Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit X-Greylist: Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.5.12 (shards.monkeyblade.net [149.20.54.216]); Tue, 07 May 2019 12:17:59 -0700 (PDT) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: YueHaibing Date: Mon, 6 May 2019 22:44:04 +0800 > BUG: unable to handle kernel NULL pointer dereference at 0000000000000128 . .. > ---[ end trace 8322b2b8bf83f8e1 > > If alloc_workqueue fails in l2tp_init, l2tp_net_ops > is unregistered on failure path. Then l2tp_exit_net > is called which will flush NULL workqueue, this patch > add a NULL check to fix it. > > Fixes: 67e04c29ec0d ("l2tp: unregister l2tp_net_ops on failure path") > Signed-off-by: YueHaibing Applied and queued up for -stable, thanks.