Re: [PATCH 11/20] ethernet: ucc_geth: fix use-after-free in ucc_geth_remove()

From: Rasmus Villemoes
Date: Sat Dec 05 2020 - 16:51:35 EST

Next message: Martin Blumenstingl: "Re: [PATCH v2 1/3] phy: amlogic: phy-meson-gxl-usb2: fix shared reset controller use"
Previous message: Nicola Di Lieto: "[PATCH 2/2] pwm: pwm-gpio: Add DT bindings"
In reply to: Rasmus Villemoes: "Re: [PATCH 11/20] ethernet: ucc_geth: fix use-after-free in ucc_geth_remove()"
Next in thread: Rasmus Villemoes: "[PATCH 10/20] ethernet: ucc_geth: remove {rx,tx}_glbl_pram_offset from struct ucc_geth_private"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

> I only noticed because I needed to add a free of the ug_info in a later
> patch.

Where, ironically, I add a use-after-free bug by freeing ug_info before
the ucc_geth_memclean() call.

:facepalm:

Next message: Martin Blumenstingl: "Re: [PATCH v2 1/3] phy: amlogic: phy-meson-gxl-usb2: fix shared reset controller use"
Previous message: Nicola Di Lieto: "[PATCH 2/2] pwm: pwm-gpio: Add DT bindings"
In reply to: Rasmus Villemoes: "Re: [PATCH 11/20] ethernet: ucc_geth: fix use-after-free in ucc_geth_remove()"
Next in thread: Rasmus Villemoes: "[PATCH 10/20] ethernet: ucc_geth: remove {rx,tx}_glbl_pram_offset from struct ucc_geth_private"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]