summaryrefslogtreecommitdiff
path: root/sys/net
diff options
context:
space:
mode:
authorMike Belopuhov <mikeb@cvs.openbsd.org>2017-03-07 16:28:38 +0000
committerMike Belopuhov <mikeb@cvs.openbsd.org>2017-03-07 16:28:38 +0000
commit4c94160c23a324643ab96b361e50479017c90786 (patch)
treeac4e2c6aceb294b95760dc33805f022320b43e51 /sys/net
parentec56c410b88d0c8821f611698ea16c216dcc3ed2 (diff)
Don't overwrite the flow ID once it's set
Output processing may split, encapsulate or obfuscate a single stream which makes the changed flow ID less useful for purposes of flow control, for instance fair sharing of bandwidth. OK dlg
Diffstat (limited to 'sys/net')
-rw-r--r--sys/net/pf.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/sys/net/pf.c b/sys/net/pf.c
index f75ef912d0d..711485754dc 100644
--- a/sys/net/pf.c
+++ b/sys/net/pf.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: pf.c,v 1.1016 2017/03/07 09:29:40 mpi Exp $ */
+/* $OpenBSD: pf.c,v 1.1017 2017/03/07 16:28:37 mikeb Exp $ */
/*
* Copyright (c) 2001 Daniel Hartmeier
@@ -6782,7 +6782,7 @@ done:
s->key[PF_SK_STACK]->inp = pd.m->m_pkthdr.pf.inp;
}
- if (s) {
+ if (s && (pd.m->m_pkthdr.ph_flowid & M_FLOWID_VALID) == 0) {
pd.m->m_pkthdr.ph_flowid = M_FLOWID_VALID |
(M_FLOWID_MASK & bemtoh64(&s->id));
}