Re: [ovs-dev] [PATCH 1/2] ofctrl: Fix version check in ofctrl_inject_packet().

2017-01-06 Thread Ben Pfaff
On Fri, Jan 06, 2017 at 10:55:11AM -0800, Joe Stringer wrote: > On 6 January 2017 at 10:50, Joe Stringer wrote: > > On 5 January 2017 at 17:03, Ben Pfaff wrote: > >> "enum ofp_version" is unsigned in the System V ABI used by Linux, so > >> it will never be less than

Re: [ovs-dev] [PATCH 1/2] ofctrl: Fix version check in ofctrl_inject_packet().

2017-01-06 Thread Joe Stringer
On 6 January 2017 at 10:50, Joe Stringer wrote: > On 5 January 2017 at 17:03, Ben Pfaff wrote: >> "enum ofp_version" is unsigned in the System V ABI used by Linux, so >> it will never be less than 0, so an rconn with an unnegotiated version will >> never be found