From mboxrd@z Thu Jan 1 00:00:00 1970 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 5653767E8F; Tue, 23 Jan 2024 02:01:57 +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=1705975317; cv=none; b=GzhqLNPbWzz64N8jAb4TXlqzLyg+WTgFX10DgCGIqC1F+BJTA04RbP4hlyGj8gMxV6TECPWn/Fvc9Yz0vxaNIp105OXaBe57FJyM2VXX/IZWdvQo/4BmfYa3Q8VBiQfmynM8vd2ubNFBIllBsRiHTKOx2cIhr/CN2sc0Zh6zmvI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1705975317; c=relaxed/simple; bh=C1lhYc5abG21wYSaLcMo38v1S9BG4JKgN830508G/L8=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=Aig7bw0t//YAm60u+blFGRiHTQ1Flot3qYTeQ7Aklv0yoZGK5Cwjbq8r/cQyhOQPv/PoV+qamLGQPARPFp+FBKM1sNhD8Pevteo5ZVTDlEr+QODqZLt4MDn2rsKIU0H/prOEDwOCgP0AT7XyIsmyGfsQzxUUlR7Hxnp6SyI3SO0= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b=yhbQKQrN; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b="yhbQKQrN" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 19A08C433C7; Tue, 23 Jan 2024 02:01:57 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1705975317; bh=C1lhYc5abG21wYSaLcMo38v1S9BG4JKgN830508G/L8=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=yhbQKQrNp8U/WqAoXjvRq1t3gMWKSj++Q+M0o3ij60JdoApt6fMNCW48zc4vmlp6k l+fyYxLNXa39CUCallL+8z1GejJSrCD4cHMXaD1YhpElk0SbX+iGsWZFlFrMXNvlxa 6r/m8C4TnukwVj7+jZSCXLdSPQfN/qxSlRw3Rrxg= From: Greg Kroah-Hartman To: stable@vger.kernel.org Cc: Greg Kroah-Hartman , patches@lists.linux.dev, =?UTF-8?q?Uwe=20Kleine-K=C3=B6nig?= , Philipp Zabel , Philipp Zabel , Sasha Levin Subject: [PATCH 6.6 248/583] drm/imx/lcdc: Fix double-free of driver data Date: Mon, 22 Jan 2024 15:54:59 -0800 Message-ID: <20240122235819.588025332@linuxfoundation.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240122235812.238724226@linuxfoundation.org> References: <20240122235812.238724226@linuxfoundation.org> User-Agent: quilt/0.67 X-stable: review X-Patchwork-Hint: ignore Precedence: bulk X-Mailing-List: patches@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 6.6-stable review patch. If anyone has any objections, please let me know. ------------------ From: Uwe Kleine-König [ Upstream commit ff3670877e7c73d06c2a835d9abb62efcae0145c ] The struct imx_lcdc driver data is allocated using devm_drm_dev_alloc() so it must not be explicitly kfree()d. Also drm_kms_helper_poll_fini() should not be called as there is no matching drm_kms_helper_poll_init(). So drop the release function completely. Fixes: c87e859cdeb5 ("drm/imx/lcdc: Implement DRM driver for imx25") Signed-off-by: Uwe Kleine-König Reviewed-by: Philipp Zabel Signed-off-by: Philipp Zabel Link: https://patchwork.freedesktop.org/patch/msgid/20230706092731.2630232-1-u.kleine-koenig@pengutronix.de Signed-off-by: Sasha Levin --- drivers/gpu/drm/imx/lcdc/imx-lcdc.c | 9 --------- 1 file changed, 9 deletions(-) diff --git a/drivers/gpu/drm/imx/lcdc/imx-lcdc.c b/drivers/gpu/drm/imx/lcdc/imx-lcdc.c index 22b65f4a0e30..4beb3b4bd694 100644 --- a/drivers/gpu/drm/imx/lcdc/imx-lcdc.c +++ b/drivers/gpu/drm/imx/lcdc/imx-lcdc.c @@ -342,21 +342,12 @@ static const struct drm_mode_config_helper_funcs imx_lcdc_mode_config_helpers = .atomic_commit_tail = drm_atomic_helper_commit_tail_rpm, }; -static void imx_lcdc_release(struct drm_device *drm) -{ - struct imx_lcdc *lcdc = imx_lcdc_from_drmdev(drm); - - drm_kms_helper_poll_fini(drm); - kfree(lcdc); -} - DEFINE_DRM_GEM_DMA_FOPS(imx_lcdc_drm_fops); static struct drm_driver imx_lcdc_drm_driver = { .driver_features = DRIVER_GEM | DRIVER_MODESET | DRIVER_ATOMIC, .fops = &imx_lcdc_drm_fops, DRM_GEM_DMA_DRIVER_OPS_VMAP, - .release = imx_lcdc_release, .name = "imx-lcdc", .desc = "i.MX LCDC driver", .date = "20200716", -- 2.43.0