On Fri, Mar 15, 2019 at 04:04:19PM -0700, Ashish Varma wrote: > 'usable_protocols' is now getting set to OFPUTIL_P_OF10_ANY on return from > 'parse_flow_monitor_request' function. The calling function now checks for the > value in this variable against the 'allowed_protocols' variable. > Also a check is added for a match field which is not supported in OpenFlow 1.0 > and return an error. > Modified the man page of ovs-ofctl to reflect Flow Monitor support as > OpenFlow 1.0 Nicira extension only. > > Signed-off-by: Ashish Varma <ashishvarma....@gmail.com>
Thanks for the patch! (And sorry it took so long to review.) Would you mind adding tests that trigger the new error conditions? Thanks, Ben. _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev