On Wed, Nov 04, 2020 at 05:33:27PM +0100, Ilya Maximets wrote:
> On 11/3/20 5:46 PM, Ben Pfaff wrote:
> > On Tue, Nov 03, 2020 at 03:53:51AM +0100, Ilya Maximets wrote:
> >> On 11/3/20 12:28 AM, Ben Pfaff wrote:
> >>> match_format() prints out "igmp" for IGMP flows, but
> >>> ofp_parse_protocol()
On 11/3/20 5:46 PM, Ben Pfaff wrote:
> On Tue, Nov 03, 2020 at 03:53:51AM +0100, Ilya Maximets wrote:
>> On 11/3/20 12:28 AM, Ben Pfaff wrote:
>>> match_format() prints out "igmp" for IGMP flows, but
>>> ofp_parse_protocol() didn't accept it, which meant that OVS would print
>>> out a flow that it
On Tue, Nov 03, 2020 at 03:53:51AM +0100, Ilya Maximets wrote:
> On 11/3/20 12:28 AM, Ben Pfaff wrote:
> > match_format() prints out "igmp" for IGMP flows, but
> > ofp_parse_protocol() didn't accept it, which meant that OVS would print
> > out a flow that it wouldn't re-parse. This fixes the
On 11/3/20 12:28 AM, Ben Pfaff wrote:
> match_format() prints out "igmp" for IGMP flows, but
> ofp_parse_protocol() didn't accept it, which meant that OVS would print
> out a flow that it wouldn't re-parse. This fixes the problem and adds
> a test.
I'm a bit confused. IIUC, matching on igmp is
match_format() prints out "igmp" for IGMP flows, but
ofp_parse_protocol() didn't accept it, which meant that OVS would print
out a flow that it wouldn't re-parse. This fixes the problem and adds
a test.
Signed-off-by: Ben Pfaff
---
lib/ofp-parse.c| 1 +
tests/ovs-ofctl.at | 2 ++
2 files