diff options
| author | Purna Chandra Mandal <purna.mandal@microchip.com> | 2016-05-17 10:35:54 +0530 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2016-06-25 13:50:53 -0700 |
| commit | 24a6a3037e0b47168f9b3bb9db7ad60c89dad7ba (patch) | |
| tree | a2e09aa866698d1184bc73b10ce4fe24a66b082e /drivers/tty/serial/pic32_uart.c | |
| parent | 210417ce836aa3f0ededcc8f541f5e4a05d2e880 (diff) | |
serial: pic32_uart: Fix double free of 'sport->irq_fault_name'.
Allocated memory for 'sport->irq_fault_name' is freed twice, first
in error check of 'if(!sport->irq_rx_name)' and other in fallback
handler.
Signed-off-by: Purna Chandra Mandal <purna.mandal@microchip.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/tty/serial/pic32_uart.c')
| -rw-r--r-- | drivers/tty/serial/pic32_uart.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/tty/serial/pic32_uart.c b/drivers/tty/serial/pic32_uart.c index 62a43bf5698e..7f8e99bbcb73 100644 --- a/drivers/tty/serial/pic32_uart.c +++ b/drivers/tty/serial/pic32_uart.c @@ -445,7 +445,6 @@ static int pic32_uart_startup(struct uart_port *port) sport->idx); if (!sport->irq_rx_name) { dev_err(port->dev, "%s: kasprintf err!", __func__); - kfree(sport->irq_fault_name); ret = -ENOMEM; goto out_f; } |