The xlate_actions() call in handle_flow_miss_without_facet() didn't
implement fin_timeout properly because tcp_flags wasn't getting set.

I have not tested that this fixes the problem, but it seems "obviously
correct".

Bug #16506.
Reported-by: Ying Chen <[email protected]>
Signed-off-by: Ben Pfaff <[email protected]>
---
 AUTHORS                |    1 +
 ofproto/ofproto-dpif.c |    1 +
 2 files changed, 2 insertions(+)

diff --git a/AUTHORS b/AUTHORS
index c113f16..498e9f0 100644
--- a/AUTHORS
+++ b/AUTHORS
@@ -206,6 +206,7 @@ Vjekoslav Brajkovic     [email protected]
 Voravit T.              [email protected]
 YAMAMOTO Takashi        [email protected]
 Yeming Zhao             [email protected]
+Ying Chen               [email protected]
 Yongqiang Liu           [email protected]
 kk yap                  [email protected]
 likunyun                [email protected]
diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c
index 1da8ffb..ecb50d5 100644
--- a/ofproto/ofproto-dpif.c
+++ b/ofproto/ofproto-dpif.c
@@ -3689,6 +3689,7 @@ handle_flow_miss_without_facet(struct flow_miss *miss,
         action_xlate_ctx_init(&ctx, ofproto, &miss->flow,
                               &miss->initial_vals, rule, 0, packet);
         ctx.resubmit_stats = &stats;
+        ctx.tcp_flags = stats.tcp_flags;
         xlate_actions(&ctx, rule->up.ofpacts, rule->up.ofpacts_len,
                       &odp_actions);
 
-- 
1.7.10.4

_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to