On Thu, Jun 15, 2017 at 02:07:59PM +0530, nusid...@redhat.com wrote: > From: Numan Siddique <nusid...@redhat.com> > > 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 <nusid...@redhat.com>
Thank you for the report and the fix. I modified the patch so that xlate_ctx's paused_flow member is a 'struct flow *' instead of a 'struct flow', so that xlate_actions() doesn't always have to zero-initialize an extra struct flow (which is large), and I applied it to master. Thanks a lot! _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev