There is a potential NULL pointer dereference in case fb_create_modedb() fails and returns NULL.

Signed-off-by: YueHaibing yuehaibing@huawei.com --- drivers/video/fbdev/core/fbmon.c | 2 ++ 1 file changed, 2 insertions(+)

diff --git a/drivers/video/fbdev/core/fbmon.c b/drivers/video/fbdev/core/fbmon.c index dd31289..3558a70 100644 --- a/drivers/video/fbdev/core/fbmon.c +++ b/drivers/video/fbdev/core/fbmon.c @@ -978,6 +978,8 @@ void fb_edid_to_monspecs(unsigned char *edid, struct fb_monspecs *specs) get_monspecs(edid, specs);

specs->modedb = fb_create_modedb(edid, &specs->modedb_len, specs); + if (!specs->modedb) + return;

/* * Workaround for buggy EDIDs that sets that the first