evie/android_kernel_oneplus_msm8998
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix a potential NULL pointer dereference in write_bulk_callback() in drivers/net/usb/pegasus.c

Browse source 
This patch fixes a potential null dereference bug where we dereference
pegasus before a null check. This patch simply moves the dereferencing
after the null check.

Signed-off-by: Micah Gruber <micah.gruber@gmail.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
This commit is contained in:
Micah Gruber 2007-07-23 16:05:52 +08:00 • committed by Jeff Garzik
parent 80ba80a9bf
commit 9351982b25
1 changed files with 3 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
drivers/net/usb/pegasus.c
View file

@ -768,11 +768,13 @@ done:
static void write_bulk_callback(struct urb *urb) static void write_bulk_callback(struct urb *urb)
{ {
pegasus_t *pegasus = urb->context; pegasus_t *pegasus = urb->context;
struct net_device *net = pegasus->net; struct net_device *net;
if (!pegasus) if (!pegasus)
return; return;
net = pegasus->net;
if (!netif_device_present(net) || !netif_running(net)) if (!netif_device_present(net) || !netif_running(net))
return; return;