On Fri, Aug 3, 2012 at 12:34 PM, Pravin Shelar <pshe...@nicira.com> wrote:
> On Thu, Aug 2, 2012 at 7:39 PM, Jesse Gross <je...@nicira.com> wrote:
>> When installing a flow with an action to set a particular field we
>> need to validate that the packets that are part of the flow actually
>> contain that header.  With IP we use zeroed addresses and with TCP/UDP
>> the check is for zeroed ports.  This check is overly broad and can catch
>> packets like DHCP requests that have a zero source address in a
>> legitimate header.  This changes the check to look for a zeroed protocol
>> number for IP or for both ports be zero for TCP/UDP before considering
>> the header to not exist.
>>
>> Bug #12769
> Looks good.
> Acked-by: Pravin B Shelar <pshe...@nicira.com>

Thanks, pushed to master and branches back to 1.4.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to