On Fri, Aug 01, 2014 at 06:39:14PM -0700, Ethan Jackson wrote: > This problem is uncovered by a future patch. > > Signed-off-by: Ethan Jackson <[email protected]>
I think that the actual bug here is that the code uses MAX instead of MIN. The goal is to push as many MPLS labels as are present, but not more than 'struct flow' has space for. > @@ -210,7 +210,7 @@ parse_mpls(void **datap, size_t *sizep) > break; > } > } > - return MAX(count, FLOW_MAX_MPLS_LABELS); > + return count; > } > > static inline ovs_be16 > -- > 1.8.1.2 > > _______________________________________________ > dev mailing list > [email protected] > http://openvswitch.org/mailman/listinfo/dev _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
