diff options
| author |   blambert <blambert@openbsd.org> | 2011-11-28 10:25:22 +0000 |
|---|---|---|
| committer | blambert <blambert@openbsd.org> | 2011-11-28 10:25:22 +0000 |
| commit | 8fa01fb9746da710226eda31c97013a52a88e9c0 (patch) | |
| tree | 7fd0f33493a1f7209afb37344661c5f36518418d /sys | |
| parent | fix formula (diff) | |
| download | wireguard-openbsd-8fa01fb9746da710226eda31c97013a52a88e9c0.tar.xz wireguard-openbsd-8fa01fb9746da710226eda31c97013a52a88e9c0.zip | |
Fix reversed error-handling gotos in myx_buf_fill(), which would lead to
either an mbuf leak or a NULL pointer dereference.
ok sthen@ claudio@ dlg@
testing claudio@ dlg@
Diffstat (limited to 'sys')
| -rw-r--r-- | sys/dev/pci/if_myx.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/sys/dev/pci/if_myx.c b/sys/dev/pci/if_myx.c index f15fe82af59..da0062c6d6f 100644 --- a/sys/dev/pci/if_myx.c +++ b/sys/dev/pci/if_myx.c @@ -1,4 +1,4 @@ -/* $OpenBSD: if_myx.c,v 1.29 2011/08/08 01:30:25 dlg Exp $ */ +/* $OpenBSD: if_myx.c,v 1.30 2011/11/28 10:25:22 blambert Exp $ */ /* * Copyright (c) 2007 Reyk Floeter <reyk@openbsd.org> @@ -1752,10 +1752,10 @@ myx_buf_fill(struct myx_softc *sc, int ring) return (mb); -mfree: - m_freem(m); put: myx_buf_put(&sc->sc_rx_buf_free[ring], mb); +mfree: + m_freem(m); return (NULL); } |