Re: [PATCH 3/3] fbdev/efifb: Cleanup fb_info in .fb_destroy rather than .remove

2022-05-05 Thread Thomas Zimmermann
Am 04.05.22 um 23:58 schrieb Javier Martinez Canillas: The driver is calling framebuffer_release() in its .remove callback, but this will cause the struct fb_info to be freed too early. Since it could be that a reference is still hold to it if user-space opened the fbdev. This would lead to a

[PATCH 3/3] fbdev/efifb: Cleanup fb_info in .fb_destroy rather than .remove

2022-05-04 Thread Javier Martinez Canillas
The driver is calling framebuffer_release() in its .remove callback, but this will cause the struct fb_info to be freed too early. Since it could be that a reference is still hold to it if user-space opened the fbdev. This would lead to a use-after-free error if the framebuffer device was unregist