[ovs-git] [openvswitch/ovs] 0d3239: ofproto-dpif: Fix the clone issue with continuatio...

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: 0d3239e85606bedcc1d9b852de82c735947c2e27
      https://github.com/openvswitch/ovs/commit/0d3239e85606bedcc1d9b852de82c735947c2e27
  Author: Numan Siddique <nusiddiq at redhat.com>
  Date:   2017-07-10 (Mon, 10 Jul 2017)

  Changed paths:
    M ofproto/ofproto-dpif-xlate.c
    M tests/ofproto-dpif.at

  Log Message:
  -----------
  ofproto-dpif: Fix the clone issue with continuation

When the clone action is composed and if the inner clone actions
modifies any of the metadata fields, the updated values are not
preserved when xlate_clone() returns. This causes the controller
to receive invalid metadata if the cloned packet has a 'controller'
action with the 'pause' flag set. When the controller resumes the
packet the resumed packet will not continue properly.

This patch addresses this issue.

The issue can be reproduced running the test case added in this
patch without the fix.

Signed-off-by: Numan Siddique <nusiddiq at redhat.com>
Signed-off-by: Ben Pfaff <blp at ovn.org>