Subject: [merged] backlight-lcd-call-put_device-if-device_register-fails.patch removed from -mm tree To: levex@xxxxxxxxx,jg1.han@xxxxxxxxxxx,mm-commits@xxxxxxxxxxxxxxx From: akpm@xxxxxxxxxxxxxxxxxxxx Date: Fri, 31 Jan 2014 11:15:59 -0800 The patch titled Subject: drivers/video/backlight/lcd.c: call put_device if device_register fails has been removed from the -mm tree. Its filename was backlight-lcd-call-put_device-if-device_register-fails.patch This patch was dropped because it was merged into mainline or a subsystem tree ------------------------------------------------------ From: Levente Kurusa <levex@xxxxxxxxx> Subject: drivers/video/backlight/lcd.c: call put_device if device_register fails Currently we kfree the container of the device which failed to register. This is wrong as the last reference is not given up with a put_device call. Also, now that we have put_device() callen, we no longer need the kfree as the new_ld->dev.release function will take care of kfreeing the associated memory. Signed-off-by: Levente Kurusa <levex@xxxxxxxxx> Acked-by: Jingoo Han <jg1.han@xxxxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- drivers/video/backlight/lcd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff -puN drivers/video/backlight/lcd.c~backlight-lcd-call-put_device-if-device_register-fails drivers/video/backlight/lcd.c --- a/drivers/video/backlight/lcd.c~backlight-lcd-call-put_device-if-device_register-fails +++ a/drivers/video/backlight/lcd.c @@ -228,7 +228,7 @@ struct lcd_device *lcd_device_register(c rc = device_register(&new_ld->dev); if (rc) { - kfree(new_ld); + put_device(&new_ld->dev); return ERR_PTR(rc); } _ Patches currently in -mm which might be from levex@xxxxxxxxx are origin.patch w1-call-put_device-if-device_register-fails.patch -- To unsubscribe from this list: send the line "unsubscribe mm-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html