From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 3B5B0C7EE23 for ; Mon, 27 Feb 2023 02:10:46 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id C5A0810E349; Mon, 27 Feb 2023 02:10:45 +0000 (UTC) Received: from ams.source.kernel.org (ams.source.kernel.org [145.40.68.75]) by gabe.freedesktop.org (Postfix) with ESMTPS id 84FFE10E348; Mon, 27 Feb 2023 02:10:44 +0000 (UTC) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id 3017FB80D24; Mon, 27 Feb 2023 02:10:43 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id BF857C433D2; Mon, 27 Feb 2023 02:10:39 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1677463841; bh=X8H0NbanBc47rOaL4RneRfp8UDNfcN20LwCv04MwCdI=; h=From:To:Cc:Subject:Date:From; b=M1DzKGWZndgU6Y+jSzNUQWFgjHlluKP3L/9ds5Msyo7nuPxNE8tHcrnDEJvwZOdv8 IDSrmz4AF7d+qlWvmJTOs8Pjs1K/XVM5Z6aoXkb2B2HhqTArBBVl1jrM4vIM6IsYgW HiO01yMAqUclCTQU2nSUn3iOu0H2eHcxfJw4QPt1ajEPkhOBGkUsUaL8L+v1cdxyn8 TTA6E219p7bDE1/6EgbQJjJFmoc25xNwfVaTBQlASD4qkGhI5xCKr+doc/7fkuXa/1 PfSCCMKw47cLoopFRHGwzeqDvQSw7iDhbzkVtKCYdcmLpDD78yhWRdLqc8mpin1lDa qUcdrx9TIBFRg== From: Sasha Levin To: linux-kernel@vger.kernel.org, stable@vger.kernel.org Subject: [PATCH AUTOSEL 5.4 01/15] drm/amd/display: Fix potential null-deref in dm_resume Date: Sun, 26 Feb 2023 21:10:20 -0500 Message-Id: <20230227021038.1052958-1-sashal@kernel.org> X-Mailer: git-send-email 2.39.0 MIME-Version: 1.0 X-stable: review X-Patchwork-Hint: Ignore Content-Transfer-Encoding: 8bit X-BeenThere: amd-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Discussion list for AMD gfx List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Sasha Levin , stylon.wang@amd.com, Xinhui.Pan@amd.com, kernel test robot , sunpeng.li@amd.com, airlied@gmail.com, Wayne Lin , Rodrigo.Siqueira@amd.com, Roman Li , amd-gfx@lists.freedesktop.org, aurabindo.pillai@amd.com, hersenxs.wu@amd.com, dri-devel@lists.freedesktop.org, daniel@ffwll.ch, Jasdeep Dhillon , Alex Deucher , harry.wentland@amd.com, christian.koenig@amd.com, Dan Carpenter Errors-To: amd-gfx-bounces@lists.freedesktop.org Sender: "amd-gfx" From: Roman Li [ Upstream commit 7a7175a2cd84b7874bebbf8e59f134557a34161b ] [Why] Fixing smatch error: dm_resume() error: we previously assumed 'aconnector->dc_link' could be null [How] Check if dc_link null at the beginning of the loop, so further checks can be dropped. Reported-by: kernel test robot Reported-by: Dan Carpenter Reviewed-by: Wayne Lin Acked-by: Jasdeep Dhillon Signed-off-by: Roman Li Signed-off-by: Alex Deucher Signed-off-by: Sasha Levin --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c index 9fd711005c1f5..1e7083bc8a527 100644 --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c @@ -1206,12 +1206,14 @@ static int dm_resume(void *handle) list_for_each_entry(connector, &ddev->mode_config.connector_list, head) { aconnector = to_amdgpu_dm_connector(connector); + if (!aconnector->dc_link) + continue; + /* * this is the case when traversing through already created * MST connectors, should be skipped */ - if (aconnector->dc_link && - aconnector->dc_link->type == dc_connection_mst_branch) + if (aconnector->dc_link->type == dc_connection_mst_branch) continue; mutex_lock(&aconnector->hpd_lock); -- 2.39.0