diff options
| author | Rafael Zalamena <rzalamena@cvs.openbsd.org> | 2016-10-23 11:56:52 +0000 |
|---|---|---|
| committer | Rafael Zalamena <rzalamena@cvs.openbsd.org> | 2016-10-23 11:56:52 +0000 |
| commit | f4f7538feb83a756171d2c2d3ec47431287f52c4 (patch) | |
| tree | 22a9996d9d1722d87c1264178979b96af74b1044 /sys/net/switchofp.c | |
| parent | 6181e963abb562516cdb399fc3af2384c738db67 (diff) | |
Don't free mbuf on error during swofp_flow_entry_put_instructions() parse,
otherwise we won't have the request ofp_header to send in the error message.
ok jca@
Diffstat (limited to 'sys/net/switchofp.c')
| -rw-r--r-- | sys/net/switchofp.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/sys/net/switchofp.c b/sys/net/switchofp.c index 9f20e84d4c4..4ecb3388575 100644 --- a/sys/net/switchofp.c +++ b/sys/net/switchofp.c @@ -1,4 +1,4 @@ -/* $OpenBSD: switchofp.c,v 1.15 2016/10/21 22:12:38 jsg Exp $ */ +/* $OpenBSD: switchofp.c,v 1.16 2016/10/23 11:56:51 rzalamena Exp $ */ /* * Copyright (c) 2016 Kazuya GODA <goda@openbsd.org> @@ -4633,9 +4633,6 @@ swofp_flow_entry_put_instructions(struct mbuf **m, return (0); failed: - m_freem(*m); - (*m) = NULL; - return (error); } |