Received: by 2002:ab2:710b:0:b0:1ef:a325:1205 with SMTP id z11csp1147045lql; Tue, 12 Mar 2024 08:31:37 -0700 (PDT) X-Forwarded-Encrypted: i=3; AJvYcCXzbQCb8El+/7zk/Rn7v38mVHm167VZTw6jUKvL2i2jzHt7zn+0b7p23v1lbwLPbwhUg3Dcu/SDQSgLTcrDiMoe7Jg5SafGXaQCn9tgbQ== X-Google-Smtp-Source: AGHT+IG1k19P6CfI2Xf4atIrZutCOLojRdVtmZiMEKFIvDZJZqEQBUKFwasa20NSSJbLxZb8lEI+ X-Received: by 2002:a17:902:8207:b0:1dc:73c9:4853 with SMTP id x7-20020a170902820700b001dc73c94853mr637034pln.41.1710257497131; Tue, 12 Mar 2024 08:31:37 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1710257497; cv=pass; d=google.com; s=arc-20160816; b=G2YR7wVfiGF8txSxFIg23PfO6I6+EY9F905MAzH+0gxCfOXNChNgnnxRqBFRVnKifo yhpD8VeV4NnvDI7/1VOIAjYvzNgARb8bi53OkvaqdiUfvBBda6w/30HAYdRe+gN/R51i LHel9mRIcHgZQU/EQ7OkmC3weHGBhrKN3jcWVXrZ9zXR3K3+N2w5JEDSLuvvZL0Q0TYm Dj4YKPVoGILVZ6I0a6uPYQ8zWefN+LRmyh8WCNaZbOVbD8iphh6qAVoY2OElmCw+Hhcb 7K46gXCvpBIdAwC/FBcM46dWHO9VL8b2hCB74X9d3I1crKpxBVHnHcN9Mi7ZfWadv4pB DnIA== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=date:message-id:user-agent:cc:to:references:in-reply-to:from :subject:content-transfer-encoding:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:dkim-signature; bh=oy2wJ9gSDRFdacEXVvrYZXVQ4/Axm+75JzwVBqKx0a4=; fh=TAzr4MzZPR63tkkPQJMRfIuqiC9z+f44LtKXv0TbZNs=; b=YNSxf5MOAvTncz+di30mSVb73tAhfyRIh/G+YNdj8+EJsdls4YeB9pgx1CJblZQuS1 Ns/6H3/SSxvb4ExjH8pYNjkbuOz57F/Lh/ObyFXeDI+jcDERsU4OuiETtT0zGSarIsHs xw7dus6F6LXSQYj6yY1tMG5d0xvCfdmSpSgH0M59WeY5B7m87qqRdrKacp8BV431v09d gFE9RASzrBAQUViI79JQUfu6y6kCbNwKONIbVxZ8lbiI3GpiRuJPqwrlFvvrTIAA+ZXA sGWNHGzOU8orVgJwFuiHYPfI+QfMr3STelANTpIfgH+avw59AR5ICHg0Qh7dd4XGP5pT EXOA==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=s7rNhgdZ; arc=pass (i=1 dkim=pass dkdomain=kernel.org); spf=pass (google.com: domain of linux-wireless+bounces-4625-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) smtp.mailfrom="linux-wireless+bounces-4625-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from sv.mirrors.kernel.org (sv.mirrors.kernel.org. [139.178.88.99]) by mx.google.com with ESMTPS id k16-20020a170902ce1000b001ddc90af6c6si177303plg.319.2024.03.12.08.31.36 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 12 Mar 2024 08:31:37 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-wireless+bounces-4625-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=s7rNhgdZ; arc=pass (i=1 dkim=pass dkdomain=kernel.org); spf=pass (google.com: domain of linux-wireless+bounces-4625-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) smtp.mailfrom="linux-wireless+bounces-4625-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sv.mirrors.kernel.org (Postfix) with ESMTPS id C6EF6286937 for ; Tue, 12 Mar 2024 15:31:36 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 3606A7C6C3; Tue, 12 Mar 2024 15:31:32 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="s7rNhgdZ" X-Original-To: linux-wireless@vger.kernel.org Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 0BC877C0B1; Tue, 12 Mar 2024 15:31:31 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1710257492; cv=none; b=PDcmbigm6gLf/0KzztvCMIQVSQFt+lUW9VHT0U0JhtNKi2VqNXJnT1JkyGuEAhS/yFucp1u1HSLa+/MZCJCadiakvb7VgKmqVXtUo3pTD/n7uRv+LRRprfzhuGoZ3+0un2pgjD/umT0WrYvb/rlsYkzjraSwTnV0NN/aRdTuLTg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1710257492; c=relaxed/simple; bh=AB3f0oe5VQwbDBA3sbXZIh/T3+n5F4eIfPvO3h0claU=; h=Content-Type:MIME-Version:Subject:From:In-Reply-To:References:To: Cc:Message-ID:Date; b=n1hvjMwVjtZuMk4rTNtCm4TNXMl6qd7zZFx5nLOfMBKkQg6rkHq786jhwamHtOAKnyO4a3rbHYaCH2Xt6KyEXZIM5sXM6y15+3bZP/+7uObYornx7e28i0uaLgSsrcXLa10R0IIULbyRx46zagJkkBhL4F9LlWI3aR/IypTLmN4= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=s7rNhgdZ; arc=none smtp.client-ip=10.30.226.201 Received: by smtp.kernel.org (Postfix) with ESMTPSA id AC41BC43394; Tue, 12 Mar 2024 15:31:29 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1710257491; bh=AB3f0oe5VQwbDBA3sbXZIh/T3+n5F4eIfPvO3h0claU=; h=Subject:From:In-Reply-To:References:To:Cc:Date:From; b=s7rNhgdZJeuIPZPdWpCxscx0yw/uvXTi2/htWFXHogJA1V1Yu2orZwspE+XQJoSc3 o0UBmjo+iPmz5TVxBs2Cs8JhkiOk5rwX/ixi+AOn5ng9+czTiDVqFBhHNhTKs5lzlN B2mgr88HhOcTJUU+H0JdWPYr5OcfrvUwSL6QoxZKSk24djs4lPonhFLmlgt3izGrLe T3mpPxqK1Sp5X2bAq9aYa93duF5xUqkgFJi7xmZ2aCnrDD754Ohr9mib0hQBGnXMO1 b+VpbKxNx24JRf+IFBc0KfsTufDIAO3Rj9M/H5aK5O2Lj4K0kXoQC2Wjj8hjkbz9vE tyF/bmUakCKTw== Content-Type: text/plain; charset="utf-8" Precedence: bulk X-Mailing-List: linux-wireless@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: Re: [v3] ssb: Fix potential NULL pointer dereference in ssb_device_uevent() From: Kalle Valo In-Reply-To: <20240306123028.164155-1-rand.sec96@gmail.com> References: <20240306123028.164155-1-rand.sec96@gmail.com> To: Rand Deeb Cc: Michael Buesch , linux-wireless@vger.kernel.org, linux-kernel@vger.kernel.org, deeb.rand@confident.ru, lvc-project@linuxtesting.org, voskresenski.stanislav@confident.ru, khoroshilov@ispras.ru, Rand Deeb User-Agent: pwcli/0.1.1-git (https://github.com/kvalo/pwcli/) Python/3.11.2 Message-ID: <171025748752.1969294.6301568129349383051.kvalo@kernel.org> Date: Tue, 12 Mar 2024 15:31:29 +0000 (UTC) Rand Deeb wrote: > The ssb_device_uevent() function first attempts to convert the 'dev' pointer > to 'struct ssb_device *'. However, it mistakenly dereferences 'dev' before > performing the NULL check, potentially leading to a NULL pointer > dereference if 'dev' is NULL. > > To fix this issue, move the NULL check before dereferencing the 'dev' pointer, > ensuring that the pointer is valid before attempting to use it. > > Found by Linux Verification Center (linuxtesting.org) with SVACE. > > Signed-off-by: Rand Deeb Patch applied to wireless-next.git, thanks. 789c17185fb0 ssb: Fix potential NULL pointer dereference in ssb_device_uevent() -- https://patchwork.kernel.org/project/linux-wireless/patch/20240306123028.164155-1-rand.sec96@gmail.com/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches