[ovs-git] Open vSwitch: datapath: Relax set header validation. (branch-1.7)

dev at openvswitch.org dev at openvswitch.org
Sat Aug 4 02:29:53 UTC 2012

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Open vSwitch".

The branch, branch-1.7 has been updated
       via  e353e467f08fdcad56cabf6ace93101e7174394d (commit)
      from  c4610f3c889f838f051cab4c7fb0c3b5fe715a41 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit e353e467f08fdcad56cabf6ace93101e7174394d
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=e353e467f08fdcad56cabf6ace93101e7174394d
Author: Jesse Gross <jesse at nicira.com>
datapath: Relax set header validation.
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

Reported-by: Ethan Jackson <ethan at nicira.com>
Signed-off-by: Jesse Gross <jesse at nicira.com>
Acked-by: Pravin B Shelar <pshelar at nicira.com>


Summary of changes:
 datapath/datapath.c |    6 +++---
 lib/odp-util.c      |    2 +-
 2 files changed, 4 insertions(+), 4 deletions(-)

Open vSwitch

More information about the git mailing list