* [PATCH] netpoll: don't dereference NULL dev from np
@ 2009-05-11 10:36 Pavel Emelyanov
2009-05-18 3:38 ` David Miller
0 siblings, 1 reply; 2+ messages in thread
From: Pavel Emelyanov @ 2009-05-11 10:36 UTC (permalink / raw)
To: David Miller; +Cc: Stephen Hemminger, Linux Netdev List
It looks like the dev in netpoll_poll can be NULL - at lease it's
checked at the function beginning. Thus the dev->netde_ops dereference
looks dangerous.
Signed-off-by: Pavel Emelyanov <xemul@openvz.org>
---
diff --git a/net/core/netpoll.c b/net/core/netpoll.c
index b5873bd..64f51ee 100644
--- a/net/core/netpoll.c
+++ b/net/core/netpoll.c
@@ -175,9 +175,13 @@ static void service_arp_queue(struct netpoll_info *npi)
void netpoll_poll(struct netpoll *np)
{
struct net_device *dev = np->dev;
- const struct net_device_ops *ops = dev->netdev_ops;
+ const struct net_device_ops *ops;
+
+ if (!dev || !netif_running(dev))
+ return;
- if (!dev || !netif_running(dev) || !ops->ndo_poll_controller)
+ ops = dev->netdev_ops;
+ if (!ops->ndo_poll_controller)
return;
/* Process pending work on NIC */
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] netpoll: don't dereference NULL dev from np
2009-05-11 10:36 [PATCH] netpoll: don't dereference NULL dev from np Pavel Emelyanov
@ 2009-05-18 3:38 ` David Miller
0 siblings, 0 replies; 2+ messages in thread
From: David Miller @ 2009-05-18 3:38 UTC (permalink / raw)
To: xemul; +Cc: shemminger, netdev
From: Pavel Emelyanov <xemul@openvz.org>
Date: Mon, 11 May 2009 14:36:35 +0400
> It looks like the dev in netpoll_poll can be NULL - at lease it's
> checked at the function beginning. Thus the dev->netde_ops dereference
> looks dangerous.
>
> Signed-off-by: Pavel Emelyanov <xemul@openvz.org>
Good catch, probably added by the netdev_ops changes.
Applied, thanks!
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2009-05-18 3:38 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-05-11 10:36 [PATCH] netpoll: don't dereference NULL dev from np Pavel Emelyanov
2009-05-18 3:38 ` David Miller
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).