diff options
| author |   Florin Malita <fmalita@gmail.com> | 2006-05-24 21:21:31 -0400 |
|---|---|---|
| committer |   Dominik Brodowski <linux@dominikbrodowski.net> | 2006-06-02 00:56:17 +0200 |
| commit | 1617406a763870a84ffe6bba3659f30f96ac4a61 (patch) | |
| tree | 8cb1b005e2e0769f8fdefe05eda13c44aad61c36 /drivers/pcmcia | |
| parent | Merge branch 'upstream' of git://ftp.linux-mips.org/pub/scm/upstream-linus (diff) | |
| download | linux-dev-1617406a763870a84ffe6bba3659f30f96ac4a61.tar.xz linux-dev-1617406a763870a84ffe6bba3659f30f96ac4a61.zip | |
[PATCH] pcmcia: missing pcmcia_get_socket() result check
The result of pcmcia_get_socket() may be NULL but ds_event() uses it
without checking.
Coverity CID: 436.
Signed-off-by: Florin Malita <fmalita@gmail.com>
Signed-off-by: Dominik Brodowski <linux@dominikbrodowski.net>
Diffstat (limited to 'drivers/pcmcia')
| -rw-r--r-- | drivers/pcmcia/ds.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/drivers/pcmcia/ds.c b/drivers/pcmcia/ds.c index 48d3b3d30c21..74b3124e8247 100644 --- a/drivers/pcmcia/ds.c +++ b/drivers/pcmcia/ds.c @@ -1143,6 +1143,12 @@ static int ds_event(struct pcmcia_socket *skt, event_t event, int priority) { struct pcmcia_socket *s = pcmcia_get_socket(skt); + if (!s) { + printk(KERN_ERR "PCMCIA obtaining reference to socket %p " \ + "failed, event 0x%x lost!\n", skt, event); + return -ENODEV; + } + ds_dbg(1, "ds_event(0x%06x, %d, 0x%p)\n", event, priority, skt); |