On Tue, May 08, 2012 at 10:42:11AM -0700, Justin Pettit wrote: > Not all ports may fit in a Features Reply, so if that's the case, then > use the new port description stat message for looking up ports. > > Signed-off-by: Justin Pettit <[email protected]>
All I noticed is that in fetch_port_by_stats() we should check that the reply is actually a port_desc stats reply (perhaps with ofputil_decode_msg_type()), since right now AFAICT it will try to interpret an error reply as a stats reply, which probably will work out badly. Thanks, Ben. _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
