tty: serial: 8250: 8250_gsc:- Handle return NULL error from ioremap_nocache
authorArvind Yadav <arvind.yadav.cs@gmail.com>
Thu, 19 Jan 2017 08:00:44 +0000 (13:30 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 19 Jan 2017 13:20:23 +0000 (14:20 +0100)
Here, If ioremap_nocache will fail. It will return NULL.
Kernel can run into a NULL-pointer dereference.
This error check will avoid NULL pointer dereference.

Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/tty/serial/8250/8250_gsc.c

index b1e6ae9..63306de 100644 (file)
@@ -60,6 +60,10 @@ static int __init serial_init_chip(struct parisc_device *dev)
                                        7272727 : 1843200;
        uart.port.mapbase       = address;
        uart.port.membase       = ioremap_nocache(address, 16);
+       if (!uart.port.membase) {
+               dev_warn(&dev->dev, "Failed to map memory\n");
+               return -ENOMEM;
+       }
        uart.port.irq   = dev->irq;
        uart.port.flags = UPF_BOOT_AUTOCONF;
        uart.port.dev   = &dev->dev;