]> git.ipfire.org Git - thirdparty/kernel/stable.git/commit
video/logo: prevent use of logos after they have been freed
authorTomi Valkeinen <tomi.valkeinen@ti.com>
Thu, 18 Dec 2014 11:40:06 +0000 (13:40 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 27 Jan 2015 16:18:55 +0000 (08:18 -0800)
commit982d04c42d0cb7e70a575153b270191bf03506e0
tree3c1f5deb1c107c707e60438e2a88c63efeea08f9
parent92570438f512ad7ef883426aa0a784b0bb6bb382
video/logo: prevent use of logos after they have been freed

commit 92b004d1aa9f367c372511ca0330f58216b25703 upstream.

If the probe of an fb driver has been deferred due to missing
dependencies, and the probe is later ran when a module is loaded, the
fbdev framework will try to find a logo to use.

However, the logos are __initdata, and have already been freed. This
causes sometimes page faults, if the logo memory is not mapped,
sometimes other random crashes as the logo data is invalid, and
sometimes nothing, if the fbdev decides to reject the logo (e.g. the
random value depicting the logo's height is too big).

This patch adds a late_initcall function to mark the logos as freed. In
reality the logos are freed later, and fbdev probe may be ran between
this late_initcall and the freeing of the logos. In that case we will
miss drawing the logo, even if it would be possible.

Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/video/logo/logo.c