On Thu, Dec 13, 2012 at 04:28:31PM -0800, Justin Pettit wrote:
> If a negative number is supplied, the parsing code used to convert it
> into a signed one.  We ran into an incident where a third-party script
> was attempting to get the OpenFlow port number for an interface, but got
> -1 from the database, since the number had not yet been assigned.  This
> was converted to 65535, which maps to OFPP_NONE and all flows with
> ingress port OFPP_NONE were modified.  This commit disallows negative
> port numbers to help prevent broken integration scripts from disturbing
> the flow table.
> 
> Issue #14036
> 
> Signed-off-by: Justin Pettit <jpet...@nicira.com>

Looks good, thanks.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to