* [PATCH] net: spi: fix a potential NULL pointer dereference
@ 2019-03-10 7:57 Kangjie Lu
2019-03-23 3:10 ` Kangjie Lu
0 siblings, 1 reply; 2+ messages in thread
From: Kangjie Lu @ 2019-03-10 7:57 UTC (permalink / raw)
To: kjlu
Cc: pakki001, Wolfgang Grandegger, Marc Kleine-Budde,
David S. Miller, Wei Liu, Joe Perches, linux-can, netdev,
linux-kernel
In case alloc_workqueue fails to allocate the work queue and
returns NULL, the fix releases the resources and returns
-ENOMEM.
Signed-off-by: Kangjie Lu <kjlu@umn.edu>
---
drivers/net/can/spi/mcp251x.c | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/drivers/net/can/spi/mcp251x.c b/drivers/net/can/spi/mcp251x.c
index e90817608645..2737b9a20dfe 100644
--- a/drivers/net/can/spi/mcp251x.c
+++ b/drivers/net/can/spi/mcp251x.c
@@ -962,6 +962,14 @@ static int mcp251x_open(struct net_device *net)
priv->wq = alloc_workqueue("mcp251x_wq", WQ_FREEZABLE | WQ_MEM_RECLAIM,
0);
+ if (unlikely(!priv->wq)) {
+ dev_err(&spi->dev, "failed to allocate work queue\n");
+ mcp251x_power_enable(priv->transceiver, 0);
+ close_candev(net);
+ ret = -ENOMEM;
+ goto open_unlock;
+ }
+
INIT_WORK(&priv->tx_work, mcp251x_tx_work_handler);
INIT_WORK(&priv->restart_work, mcp251x_restart_work_handler);
--
2.17.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] net: spi: fix a potential NULL pointer dereference
2019-03-10 7:57 [PATCH] net: spi: fix a potential NULL pointer dereference Kangjie Lu
@ 2019-03-23 3:10 ` Kangjie Lu
0 siblings, 0 replies; 2+ messages in thread
From: Kangjie Lu @ 2019-03-23 3:10 UTC (permalink / raw)
To: kjlu
Cc: pakki001, Wolfgang Grandegger, Marc Kleine-Budde,
David S. Miller, Wei Liu, Joe Perches, linux-can, netdev,
linux-kernel
> On Mar 10, 2019, at 1:57 AM, Kangjie Lu <kjlu@umn.edu> wrote:
>
> In case alloc_workqueue fails to allocate the work queue and
> returns NULL, the fix releases the resources and returns
> -ENOMEM.
>
> Signed-off-by: Kangjie Lu <kjlu@umn.edu>
> ---
> drivers/net/can/spi/mcp251x.c | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/drivers/net/can/spi/mcp251x.c b/drivers/net/can/spi/mcp251x.c
> index e90817608645..2737b9a20dfe 100644
> --- a/drivers/net/can/spi/mcp251x.c
> +++ b/drivers/net/can/spi/mcp251x.c
> @@ -962,6 +962,14 @@ static int mcp251x_open(struct net_device *net)
>
> priv->wq = alloc_workqueue("mcp251x_wq", WQ_FREEZABLE | WQ_MEM_RECLAIM,
> 0);
> + if (unlikely(!priv->wq)) {
> + dev_err(&spi->dev, "failed to allocate work queue\n");
> + mcp251x_power_enable(priv->transceiver, 0);
> + close_candev(net);
> + ret = -ENOMEM;
> + goto open_unlock;
> + }
> +
Can someone review this patch?
> INIT_WORK(&priv->tx_work, mcp251x_tx_work_handler);
> INIT_WORK(&priv->restart_work, mcp251x_restart_work_handler);
>
> --
> 2.17.1
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2019-03-23 3:10 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-03-10 7:57 [PATCH] net: spi: fix a potential NULL pointer dereference Kangjie Lu
2019-03-23 3:10 ` Kangjie Lu
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).