linux-kernel - Re: [PATCH v2 9/9] drm/imx: parallel-display: put the bridge returned by drm_bridge_get_next

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <078a95b5a85cbf73bcbeb9e110b2bac2@kernel.org>
Date: Tue, 19 Aug 2025 13:58:49 +0000
From: "Maxime Ripard" <mripard@...nel.org>
To: "Luca Ceresoli" <luca.ceresoli@...tlin.com>
Cc: dri-devel@...ts.freedesktop.org, imx@...ts.linux.dev, linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org, "Andrzej
 Hajda" <andrzej.hajda@...el.com>, "David Airlie" <airlied@...il.com>, "Fabio
 Estevam" <festevam@...il.com>, "Hui Pu" <Hui.Pu@...ealthcare.com>, "Jernej
 Skrabec" <jernej.skrabec@...il.com>, "Jonas Karlman" <jonas@...boo.se>, "Laurent
 Pinchart" <Laurent.pinchart@...asonboard.com>, "Liu Ying" <victor.liu@....com>, "Maarten
 Lankhorst" <maarten.lankhorst@...ux.intel.com>, "Maxime Ripard" <mripard@...nel.org>, "Neil
 Armstrong" <neil.armstrong@...aro.org>, "Pengutronix Kernel Team" <kernel@...gutronix.de>, "Philipp
 Zabel" <p.zabel@...gutronix.de>, "Robert Foss" <rfoss@...nel.org>, "Sascha
 Hauer" <s.hauer@...gutronix.de>, "Shawn Guo" <shawnguo@...nel.org>, "Simona
 Vetter" <simona@...ll.ch>, "Thomas Petazzoni" <thomas.petazzoni@...tlin.com>, "Thomas
 Zimmermann" <tzimmermann@...e.de>, "Tomi Valkeinen" <tomi.valkeinen@...asonboard.com>
Subject: Re: [PATCH v2 9/9] drm/imx: parallel-display: put the bridge
 returned by drm_bridge_get_next_bridge()

On Fri, 1 Aug 2025 19:05:31 +0200, Luca Ceresoli wrote:
> The bridge returned by drm_bridge_get_next_bridge() is refcounted. Put it
> when done. We need to ensure it is not put before either next_bridge or
> next_bridge_state is in use, thus for simplicity use a cleanup action.
> 
> Signed-off-by: Luca Ceresoli <luca.ceresoli@...tlin.com>
> 
> [ ... ]

Reviewed-by: Maxime Ripard <mripard@...nel.org>

Thanks!
Maxime