leandrof/linux
1
0
Fork 0
forked from Minki/linux
Code Issues Pull Requests Packages Projects Releases Wiki Activity

staging: rts_pstor: potential NULL dereference

Browse Source 
pci_get_bus_and_slot() may return NULL, but the caller checks
wrong variable.

Signed-off-by: Vasiliy Kulikov <segoon@openwall.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This commit is contained in:
Vasiliy Kulikov 2011-02-02 21:29:31 +03:00 committed by Greg Kroah-Hartman
parent 4b906e58a1
commit 37af07d19a
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
drivers/staging/rts_pstor/rtsx.c
View File

@ -334,7 +334,7 @@ int rtsx_read_pci_cfg_byte(u8 bus, u8 dev, u8 func, u8 offset, u8 *val)
u8 devfn = (dev << 3) | func; u8 devfn = (dev << 3) | func;
pdev = pci_get_bus_and_slot(bus, devfn); pdev = pci_get_bus_and_slot(bus, devfn);
if (!dev) if (!pdev)
return -1; return -1;
pci_read_config_byte(pdev, offset, &data); pci_read_config_byte(pdev, offset, &data);