On Fri, Aug 3, 2012 at 12:34 PM, Pravin Shelar <[email protected]> wrote: > On Thu, Aug 2, 2012 at 7:39 PM, Jesse Gross <[email protected]> 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 <[email protected]>
Thanks, pushed to master and branches back to 1.4. _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
