Received: by 2002:a05:6902:102b:0:0:0:0 with SMTP id x11csp948687ybt; Wed, 17 Jun 2020 18:46:15 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwBZrr8w/TZVDmT/VE42ze5yEygJpbVzgzG41mtF8vbEWRxAdrHRheQc28dt9CdXOZVT5zT X-Received: by 2002:a17:906:3d41:: with SMTP id q1mr1946210ejf.12.1592444774834; Wed, 17 Jun 2020 18:46:14 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1592444774; cv=none; d=google.com; s=arc-20160816; b=FyF8tcurjG2iwNQ4Rf/Yxi+zHdx5LmokoAMn5eoHSQ8oQtLpgoucKuwgiPgLE/bKaG ex19+8FCnI/NjHgJZ1+dPZYMudUJNCJvT9pe4T7Vi0G50mnTJQds/XJkRuzXe17S8DZj OJ2V+8b6DCkB8OlOvvZ/oHGMxBw+liOg39SKCGaZXdbD37uv68HoQuVwiFp2bPwCQpgc p+MkMh/uDjutRjHNg3QlNMkSQvun8aF0Q2cwoi4wcWTz43mDgVWuofUF9cj4KF2weXoY nxvS7QmK8ni6A+Pzoj34hXFCEq8csEay2FannaD6V5todkSN3s3WS3aiWQjODfdTIS5h rphg== 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=uLeNQByXmfCLbkfFUJI38G+xX0LS1+m6Dp/pbkK2cqE=; b=nSsr7gi/6UzKnTgZKZpp1xjbgiQLMksM/MZZfE9bFMPO9rgeDRbgy89EqEVQNsRIoI I6M/cQjGkJUM+bWpcQ8MxEDnIjUg2vbBsHh2g1lXzEvG44rjwhK/zoojoLiv/4DX3TfG 2SuXfBuA3L8uJKVDaefJ65Y+yDOv5PLCsOM7PmQQSNVkDSbtzJ52mlIhuMTQWDM6XV9i l+xYGLXxmA960KyQ9HTKereLS7mGjYLHGcaQuKKIHDjERTggz3M5aS3oh1KWniJUnMEz cdMuLAvHN8ZKg8vmsjXLPBXGQAZJIP90tNlQiMe1nhRT821ELwKBxB0oQKIVKXdneJPW 8ELg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=uMmVVjHN; 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 wp10si956192ejb.56.2020.06.17.18.45.52; Wed, 17 Jun 2020 18:46:14 -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=uMmVVjHN; 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 S1733009AbgFRBnk (ORCPT + 99 others); Wed, 17 Jun 2020 21:43:40 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:50424 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1730793AbgFRBng (ORCPT ); Wed, 17 Jun 2020 21:43:36 -0400 Received: from mail-qt1-x841.google.com (mail-qt1-x841.google.com [IPv6:2607:f8b0:4864:20::841]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 409AAC06174E; Wed, 17 Jun 2020 18:43:36 -0700 (PDT) Received: by mail-qt1-x841.google.com with SMTP id v19so780275qtq.10; Wed, 17 Jun 2020 18:43:36 -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=uLeNQByXmfCLbkfFUJI38G+xX0LS1+m6Dp/pbkK2cqE=; b=uMmVVjHNrJUxxR3TZEt/J4FtQ0DWcqd00+UbmOgvkHtJI6tgEtBZygEio9Xllu0K60 jtINJGFfcO2Ao8vr0tN4delYcKWzT/AipR4TuhTwb3I6WGaUgW3qxvXi77tI56WENIk3 QVZUe9sjmxLD5/ZFwB+8QsH7zLEyj1jCN0NXZa37+8PSdIrvy32thJ/Thox91q21A5mj 8wx2+jjr+iJrBzv+zi7q3IHLfgMa61tEavfz61cJTvyNGL7thAuILTOC91IpDZzFnLF7 5EhS7kiKMm94RgYPCu0ZUkowU2/VOBwr6GyyxE5ipKoLu6CAhidii+fEG8Zz9Q5Y2TDr 03vw== 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=uLeNQByXmfCLbkfFUJI38G+xX0LS1+m6Dp/pbkK2cqE=; b=Sex3fKjB/9SPdOf1MiuDPcwCoFvlahcISDL1LFYH3jLdjPgzDdjUYkDNMYcopbyqrX CEh8O4qzVTDCP79kr1Seh31yhnU/5FujCuCeg8fnJRodab9EwnEuGQUXZg6ApA45X9By FfK2h7Oc0Np3QEr3ibGAYh+I09v7xqTqPqM9HoeHoFSfbIc55NKjcqnNSkwOB+X05AE8 jp3z//tfv9GtOvv4EOfsDVmFHxneOTUXfs2KOMV5vArwNxh18yNlh6kOL/H0tou3mRD8 xiVehYebEzjp0YY21NxvPMW0y3HNF/1SctMHUBXsqukOq5PTRtz4XM4cKQgybZmIq2Sf H5Lw== X-Gm-Message-State: AOAM532Pj9AeMSePl42c/X4JaA+jZybt2DXk46pW4GVob6QlXHmFnLNW w/KU9tlZEv+qhq7imXza0Mk= X-Received: by 2002:ac8:4281:: with SMTP id o1mr2102273qtl.322.1592444614758; Wed, 17 Jun 2020 18:43:34 -0700 (PDT) Received: from linux.home ([2604:2000:1344:41d:9c3:b47c:c995:4853]) by smtp.googlemail.com with ESMTPSA id x54sm1821885qta.42.2020.06.17.18.43.33 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 17 Jun 2020 18:43:34 -0700 (PDT) From: Gaurav Singh To: gaurav1086@gmail.com, Jamal Hadi Salim , Cong Wang , Jiri Pirko , "David S. Miller" , Jakub Kicinski , netdev@vger.kernel.org (open list:TC subsystem), linux-kernel@vger.kernel.org (open list) Subject: [PATCH] [net/sched] Fix null pointer deref skb in tc_ctl_action Date: Wed, 17 Jun 2020 21:43:28 -0400 Message-Id: <20200618014328.28668-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 Add null check for skb Signed-off-by: Gaurav Singh --- net/sched/act_api.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/net/sched/act_api.c b/net/sched/act_api.c index 8ac7eb0a8309..fd584821d75a 100644 --- a/net/sched/act_api.c +++ b/net/sched/act_api.c @@ -1473,9 +1473,12 @@ static const struct nla_policy tcaa_policy[TCA_ROOT_MAX + 1] = { static int tc_ctl_action(struct sk_buff *skb, struct nlmsghdr *n, struct netlink_ext_ack *extack) { + if (!skb) + return 0; + struct net *net = sock_net(skb->sk); struct nlattr *tca[TCA_ROOT_MAX + 1]; - u32 portid = skb ? NETLINK_CB(skb).portid : 0; + u32 portid = NETLINK_CB(skb).portid; int ret = 0, ovr = 0; if ((n->nlmsg_type != RTM_GETACTION) && -- 2.17.1