diff options
| author |   chl <chl@openbsd.org> | 2011-05-02 22:20:20 +0000 |
|---|---|---|
| committer | chl <chl@openbsd.org> | 2011-05-02 22:20:20 +0000 |
| commit | 409df8f9959d26d0ed1b3610c9aa4b774b98919d (patch) | |
| tree | f53691112bc531fb9171fc329e115a35e7faff86 /sys/dev | |
| parent | fix string containing state names used for debug printf()s, (diff) | |
| download | wireguard-openbsd-409df8f9959d26d0ed1b3610c9aa4b774b98919d.tar.xz wireguard-openbsd-409df8f9959d26d0ed1b3610c9aa4b774b98919d.zip | |
Fix a tyop which could potentially lead to null dereference.
While there fix a dead store.
Found by LLVM/Clang Static Analyzer.
ok deraadt@
Diffstat (limited to 'sys/dev')
| -rw-r--r-- | sys/dev/pcmcia/if_ray.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/sys/dev/pcmcia/if_ray.c b/sys/dev/pcmcia/if_ray.c index d3fde4b6003..88c59fd0285 100644 --- a/sys/dev/pcmcia/if_ray.c +++ b/sys/dev/pcmcia/if_ray.c @@ -1,4 +1,4 @@ -/* $OpenBSD: if_ray.c,v 1.45 2010/09/07 16:21:46 deraadt Exp $ */ +/* $OpenBSD: if_ray.c,v 1.46 2011/05/02 22:20:20 chl Exp $ */ /* $NetBSD: if_ray.c,v 1.21 2000/07/05 02:35:54 onoe Exp $ */ /* @@ -837,7 +837,7 @@ ray_stop(struct ray_softc *sc) wakeup(ray_report_params); } if (sc->sc_updreq) { - sc->sc_repreq->r_failcause = RAY_FAILCAUSE_EDEVSTOP; + sc->sc_updreq->r_failcause = RAY_FAILCAUSE_EDEVSTOP; wakeup(ray_update_params); } @@ -1295,7 +1295,7 @@ ray_intr_start(struct ray_softc *sc) if (firsti == RAY_CCS_LINK_NULL) return; - i = 0; + if (!RAY_ECF_READY(sc)) { /* * if this can really happen perhaps we need to save |