On 31 October 2014 15:11, Ben Pfaff <b...@nicira.com> wrote: > On Fri, Oct 31, 2014 at 02:40:11PM -0700, Joe Stringer wrote: > > Just because the ethertype is MPLS, this doesn't mean that the datapath > > understands and provides OVS_KEY_ATTR_MPLS attributes for the flow. > > Previously we would check the size of the OVS_KEY_ATTR_MPLS attribute > > before checking whether the attribute is present. This would cause a > > segfault in nl_attr_get_size(), usually triggered from a handler thread. > > > > This patch brings the MPLS parsing code more in line with the rest of > > the parse_l2_5_onward() function, by only processing MPLS if the > > attribute is present. > > > > Reported-by: Pravin B Shelar <pshe...@nicira.com> > > Signed-off-by: Joe Stringer <joestrin...@nicira.com> > > You will backport this, right? > > Acked-by: Ben Pfaff <b...@nicira.com> >
Thanks, I pushed this to master and branch-2.3. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
