[ovs-git] Open vSwitch: datapath: Avoid assigning a NULL pointer to flow actions. (master)

[dev at openvswitch.org]

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, master has been updated
       via  90b8c2f7c08eab4aea41601e57d4eee021ef5561 (commit)
       via  87968f01fc8404b98f0bb98aac0f73b743b79b5c (commit)
       via  5382a3cf9be0632c83cff17f08c9b46036a8ad88 (commit)
      from  fc3431c6a09e625343dd39b57b86839af90a08fd (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 90b8c2f7c08eab4aea41601e57d4eee021ef5561
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=90b8c2f7c08eab4aea41601e57d4eee021ef5561
Author: Jarno Rajahalme <jrajahalme at nicira.com>
		
datapath: Avoid assigning a NULL pointer to flow actions.
		
Flow SET can accept an empty set of actions, with the intended
semantics of leaving existing actions unmodified.  This seems to have
been brokin after OVS 1.7, as we have assigned the flow's actions
pointer to NULL in this case, but we never check for the NULL pointer
later on.  This patch restores the intended behavior and documents it
in the include/linux/openvswitch.h.

Signed-off-by: Jarno Rajahalme <jrajahalme at nicira.com>
Signed-off-by: Pravin B Shelar <pshelar at nicira.com>


commit 87968f01fc8404b98f0bb98aac0f73b743b79b5c
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=87968f01fc8404b98f0bb98aac0f73b743b79b5c
Author: Jarno Rajahalme <jrajahalme at nicira.com>
		
datapath: Remove a debugging message.
		
This was left in accidentally.

Signed-off-by: Jarno Rajahalme <jrajahalme at nicira.com>


commit 5382a3cf9be0632c83cff17f08c9b46036a8ad88
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=5382a3cf9be0632c83cff17f08c9b46036a8ad88
Author: Jarno Rajahalme <jrajahalme at nicira.com>
		
lib/flow: Fix flow_hash_5tuple().
		
First part of the hash was discarded as basis was used too late.

Also be explicit about the input type expected by mhash_add().

Signed-off-by: Jarno Rajahalme <jrajahalme at nicira.com>
Signed-off-by: Alex Wang <alexw at nicira.com>


-----------------------------------------------------------------------

Summary of changes:
 datapath/datapath.c         |   20 ++++++++------------
 include/linux/openvswitch.h |    4 +++-
 lib/flow.c                  |    8 ++++----
 3 files changed, 15 insertions(+), 17 deletions(-)


hooks/post-receive
-- 
Open vSwitch