diff options
| author | cem <cem@FreeBSD.org> | 2016-04-26 19:06:28 +0000 |
|---|---|---|
| committer | cem <cem@FreeBSD.org> | 2016-04-26 19:06:28 +0000 |
| commit | a595d7c8525d0b84ae32ee7c66841e94efe1defb (patch) | |
| tree | a99f2b5428d1259019b26ca58a30c37186c7c1dc /sys/dev | |
| parent | 01099cbeb7eb81176edfe46de76eb2e2b4ded41b (diff) | |
| download | FreeBSD-src-a595d7c8525d0b84ae32ee7c66841e94efe1defb.zip FreeBSD-src-a595d7c8525d0b84ae32ee7c66841e94efe1defb.tar.gz | |
iwm(4): Don't dereference potentially NULL pointer before NULL check
Introduced in r298594. There is no path before the 'vap == NULL' check where
vap is not already dereferenced.
Reported by: Coverity
CID: 1354979
Sponsored by: EMC / Isilon Storage Division
Diffstat (limited to 'sys/dev')
| -rw-r--r-- | sys/dev/iwm/if_iwm.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sys/dev/iwm/if_iwm.c b/sys/dev/iwm/if_iwm.c index 5b9ea20..202dd98 100644 --- a/sys/dev/iwm/if_iwm.c +++ b/sys/dev/iwm/if_iwm.c @@ -4377,7 +4377,7 @@ iwm_intr(void *arg) device_printf(sc->sc_dev, " rx ring: cur=%d\n", sc->rxq.cur); device_printf(sc->sc_dev, - " 802.11 state %d\n", vap->iv_state); + " 802.11 state %d\n", (vap == NULL) ? -1 : vap->iv_state); /* Don't stop the device; just do a VAP restart */ IWM_UNLOCK(sc); |
