Re: [Patch v3 1/1] leds: ledtrig-tty: free allocated ttyname buffer on deactivate

On Mon, Nov 27, 2023 at 09:16:21AM +0100, Florian Eckert wrote:

The ttyname buffer for the ledtrig_tty_data struct is allocated in the sysfs ttyname_store() function. This buffer must be released on trigger deactivation. This was missing and is thus a memory leak.

While we are at it, the tty handler in the ledtrig_tty_data struct should also be returned in case of the trigger deactivation call.

Cc: stable@vger.kernel.org Fixes: fd4a641ac88f ("leds: trigger: implement a tty trigger") Signed-off-by: Florian Eckert fe@dev.tdt.de Reviewed-by: Uwe Kleine-König u.kleine-koenig@pengutronix.de

Reviewed-by: Greg Kroah-Hartman gregkh@linuxfoundation.org