]> git.proxmox.com Git - ovs.git/commit
datapath: Fix wrong push/pop ethernet validation
authorJaime Caamaño Ruiz <jcaamano@suse.com>
Fri, 2 Nov 2018 11:45:14 +0000 (12:45 +0100)
committerBen Pfaff <blp@ovn.org>
Fri, 2 Nov 2018 20:44:53 +0000 (13:44 -0700)
commit9a98793c0932033b0a9b7616a2e95b3f3fd2a7d1
treed78fef8d9bbd0d5353bda113e1b4665e96c25b6f
parent0fb02e8e50f61aca2eaed0e84de71d97a483cdf2
datapath: Fix wrong push/pop ethernet validation

Upstream commit:
    commit 46ebe2834ba5b541f28ee72e556a3fed42c47570
    Author: Jaime Caamaño Ruiz <jcaamano@suse.com>
    Date:   Wed Oct 31 18:52:03 2018 +0100

    openvswitch: Fix push/pop ethernet validation

    When there are both pop and push ethernet header actions among the
    actions to be applied to a packet, an unexpected EINVAL (Invalid
    argument) error is obtained. This is due to mac_proto not being reset
    correctly when those actions are validated.

    Reported-at:
    https://mail.openvswitch.org/pipermail/ovs-discuss/2018-October/047554.html
Fixes: 91820da6ae85 ("openvswitch: add Ethernet push and pop actions")
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com>
Tested-by: Greg Rose <gvrose8192@gmail.com>
Reviewed-by: Greg Rose <gvrose8192@gmail.com>
Reported-at: https://mail.openvswitch.org/pipermail/ovs-discuss/2018-October/047554.html
Fixes: 6fcecb85ab ("datapath: add Ethernet push and pop actions")
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com>
Signed-off-by: Ben Pfaff <blp@ovn.org>
datapath/flow_netlink.c