diff options
| author |   damien <damien@openbsd.org> | 2006-10-11 19:20:46 +0000 |
|---|---|---|
| committer | damien <damien@openbsd.org> | 2006-10-11 19:20:46 +0000 |
| commit | e4fad577e899be7f3cdb50285040d25ccd63cb0b (patch) | |
| tree | 9d9c4bb412285dc7fc8d68687e09a9279511673d | |
| parent | hmm, slightly reword the previous to sound less like a susv3 cut/paste, (diff) | |
| download | wireguard-openbsd-e4fad577e899be7f3cdb50285040d25ccd63cb0b.tar.xz wireguard-openbsd-e4fad577e899be7f3cdb50285040d25ccd63cb0b.zip | |
fix a potential null deref and a memleak (both in error path).
ok mglocker@
| -rw-r--r-- | sys/dev/ic/pgt.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/sys/dev/ic/pgt.c b/sys/dev/ic/pgt.c index 0369d13a7d1..53970f70fff 100644 --- a/sys/dev/ic/pgt.c +++ b/sys/dev/ic/pgt.c @@ -1,4 +1,4 @@ -/* $OpenBSD: pgt.c,v 1.34 2006/10/11 10:20:31 claudio Exp $ */ +/* $OpenBSD: pgt.c,v 1.35 2006/10/11 19:20:46 damien Exp $ */ /* * Copyright (c) 2006 Claudio Jeker <claudio@openbsd.org> @@ -1814,10 +1814,10 @@ pgt_ieee80211_node_alloc(struct ieee80211com *ic) struct pgt_ieee80211_node *pin; pin = malloc(sizeof(*pin), M_DEVBUF, M_NOWAIT); - bzero(pin, sizeof *pin); - if (pin != NULL) + if (pin != NULL) { + bzero(pin, sizeof *pin); pin->pin_dot1x_auth = PIN_DOT1X_UNAUTHORIZED; - + } return (struct ieee80211_node *)pin; } @@ -3201,6 +3201,7 @@ pgt_dma_alloc_queue(struct pgt_softc *sc, enum pgt_queue pq) if (error != 0) { printf("%s: can not create DMA tag for fragment\n", sc->sc_dev.dv_xname); + free(pd, M_DEVBUF); break; } |