diff options
| author |   rzalamena <rzalamena@openbsd.org> | 2016-11-10 14:10:48 +0000 |
|---|---|---|
| committer | rzalamena <rzalamena@openbsd.org> | 2016-11-10 14:10:48 +0000 |
| commit | 4a6aa43a396a3fe3fe8bf80816d6c7cfd840af1f (patch) | |
| tree | 639dd2424bc05577d1e09b31a7770f090e4f676a | |
| parent | Simplify checkfs() and fix read-only/remote fs detection. (diff) | |
| download | wireguard-openbsd-4a6aa43a396a3fe3fe8bf80816d6c7cfd840af1f.tar.xz wireguard-openbsd-4a6aa43a396a3fe3fe8bf80816d6c7cfd840af1f.zip | |
Fix swofp_send_error mbuf handling so it doesn't leak mbufs and set the
proper mbuf header length.
ok mikeb@
| -rw-r--r-- | sys/net/switchofp.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/sys/net/switchofp.c b/sys/net/switchofp.c index c1b071fb90a..2dd37f4b6f2 100644 --- a/sys/net/switchofp.c +++ b/sys/net/switchofp.c @@ -1,4 +1,4 @@ -/* $OpenBSD: switchofp.c,v 1.28 2016/11/07 17:36:09 rzalamena Exp $ */ +/* $OpenBSD: switchofp.c,v 1.29 2016/11/10 14:10:48 rzalamena Exp $ */ /* * Copyright (c) 2016 Kazuya GODA <goda@openbsd.org> @@ -4575,10 +4575,12 @@ swofp_send_error(struct switch_softc *sc, struct mbuf *m, oe->err_type = htons(type); oe->err_code = htons(code); oe->err_oh.oh_length = htons(len + sizeof(struct ofp_error)); - m->m_len = m->m_pkthdr.len = len + sizeof(struct ofp_error); + m->m_len = m->m_pkthdr.len = sizeof(struct ofp_error); - if (m_copyback(m, sizeof(struct ofp_error), len, data, M_DONTWAIT)) + if (m_copyback(m, sizeof(struct ofp_error), len, data, M_DONTWAIT)) { + m_freem(m); return; + } (void)swofp_output(sc, m); } |