All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/3] drm/meson: fix use-after-free driver unload issues
@ 2022-09-19  1:09 ` Adrián Larumbe
  0 siblings, 0 replies; 24+ messages in thread
From: Adrián Larumbe @ 2022-09-19  1:09 UTC (permalink / raw)
  To: narmstrong, khilman, linux-amlogic, dri-devel; +Cc: adrian.larumbe

This patch series tries to fix some use-after-free bugs I've observed with
the help of KASAN in Amlogic's KMS DRM driver.

The first patch in the series reorders the driver deinitialisation sequence
so that devres won't deallocate things that are still expected to be around
by a later call to drm_dev_put.

The second patch adds a missing call to component_master_del inside a new
driver's remove callback.

The third patch makes sure some drm bridges added during driver
initialisation are removed at module unload time, to make sure the global
bridge list doesn't keep nodes to freed memory.

All three patches have been tested on an Odroid N2+ plus SBC.

Adrián Larumbe (3):
  drm/meson: reorder driver deinit sequence to fix use-after-free bug
  drm/meson: explicitly remove aggregate driver at module unload time
  drm/meson: remove drm bridges at aggregate driver unbind time

 drivers/gpu/drm/meson/meson_drv.c          | 14 +++++++++++++-
 drivers/gpu/drm/meson/meson_drv.h          |  7 +++++++
 drivers/gpu/drm/meson/meson_encoder_cvbs.c |  7 +++++++
 drivers/gpu/drm/meson/meson_encoder_cvbs.h |  1 +
 drivers/gpu/drm/meson/meson_encoder_hdmi.c |  7 +++++++
 drivers/gpu/drm/meson/meson_encoder_hdmi.h |  1 +
 drivers/gpu/drm/meson/meson_venc.h         | 15 +++++++++++++++
 7 files changed, 51 insertions(+), 1 deletion(-)

-- 
2.37.0


^ permalink raw reply	[flat|nested] 24+ messages in thread

end of thread, other threads:[~2022-09-23  9:48 UTC | newest]

Thread overview: 24+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-09-19  1:09 [PATCH 0/3] drm/meson: fix use-after-free driver unload issues Adrián Larumbe
2022-09-19  1:09 ` Adrián Larumbe
2022-09-19  1:09 ` [PATCH 1/3] drm/meson: reorder driver deinit sequence to fix use-after-free bug Adrián Larumbe
2022-09-19  1:09   ` Adrián Larumbe
2022-09-19 12:57   ` Neil Armstrong
2022-09-19 12:57     ` Neil Armstrong
2022-09-19  1:09 ` [PATCH 2/3] drm/meson: explicitly remove aggregate driver at module unload time Adrián Larumbe
2022-09-19  1:09   ` Adrián Larumbe
2022-09-19 12:57   ` Neil Armstrong
2022-09-19 12:57     ` Neil Armstrong
2022-09-19  1:09 ` [PATCH 3/3] drm/meson: remove drm bridges at aggregate driver unbind time Adrián Larumbe
2022-09-19  1:09   ` Adrián Larumbe
2022-09-19 13:03   ` Neil Armstrong
2022-09-19 13:03     ` Neil Armstrong
2022-09-20 11:49     ` Adrián Larumbe
2022-09-20 11:49       ` Adrián Larumbe
2022-09-20 12:56       ` Neil Armstrong
2022-09-20 12:56         ` Neil Armstrong
2022-09-20 22:28         ` [PATCH v2 " Adrián Larumbe
2022-09-20 22:28           ` Adrián Larumbe
2022-09-23  9:28           ` Neil Armstrong
2022-09-23  9:28             ` Neil Armstrong
2022-09-23  9:48 ` (subset) [PATCH 0/3] drm/meson: fix use-after-free driver unload issues Neil Armstrong
2022-09-23  9:48   ` Neil Armstrong

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.