]> git.proxmox.com Git - mirror_ovs.git/commit
ofp-util: Fix indirect group delete message with no buckets.
authorLászló Sürü <laszlo.suru@ericsson.com>
Thu, 24 Mar 2016 16:14:01 +0000 (16:14 +0000)
committerBen Pfaff <blp@ovn.org>
Wed, 30 Mar 2016 23:53:22 +0000 (16:53 -0700)
commitbeaa2b8702a20fa1f55e3b4c072dfe6fa394d6d1
tree78439b48c6fa90246eaf1a48f57335bb4a0b3dfd
parentd1a997a29b2128b5424403cae49b87d02492a117
ofp-util: Fix indirect group delete message with no buckets.

Using OpenFlow v1.3.5 and onwards OVS 2.5.0 returns OFPGMFC_INVALID_GROUP
error when an INDIRECT type of group deletion requested, although the
delete message is according to OpenFlow v1.3.5 standard.

The reason is the conflicting protocol check in Open  vSwitch's OpenFlow
termination, that is the indirect group delete command is once checked not
to include any bucket - as described in standard, and secondly checked to
include exactly one bucket - as also mandatory for indirect groups.

This error is not seen in the OVS internal make time verification (make check),
as ovs-ofctl CLI tool does not accept group type as command argument.
Therefore indirect group delete works, although internally it is converted into
ALL group type.

The fix is simply to ignore the mandatory single bucket check in case of
indirect group delete.  On the other hand the check is still executed in case
of group addition or modification.

Moreover to this it is planned to extend 'ovs-ofctl del-groups' arguments with
a group 'type' as well to for test purposes.

Signed-off-by: László Sűrű <laszlo.suru@ericsson.com>
Signed-off-by: Ben Pfaff <blp@ovn.org>
lib/ofp-util.c