[ovs-git] Open vSwitch: ofproto-dpif: Correct endian problem in recv_upcalls() (master)

dev at openvswitch.org dev at openvswitch.org
Tue Oct 1 15:21:25 UTC 2013


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  be58eabb281bafffc2eb60c2ee31446f312a2a83 (commit)
      from  10e576406c7444ef198cc2d4b6c4adfe242c085c (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 be58eabb281bafffc2eb60c2ee31446f312a2a83
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=be58eabb281bafffc2eb60c2ee31446f312a2a83
Author: Simon Horman <horms at verge.net.au>
		
ofproto-dpif: Correct endian problem in recv_upcalls()
		
Use nl_attr_get_u32() instead of nl_attr_get_be32() to parse nla
so that the decoded value which is passed to mhash_add()
is host byte order as mhash_add() expects.

This resolves a minor regression introduced by
10e576406c7444ef ("ofproto-dpif: Move special upcall handling into
ofproto-dpif-upcall.").

I do not expect this change has any runtime implications.

Detected using sparse.

Cc: Ethan Jackson <ethan at nicira.com>
Cc: Ben Pfaff <blp at nicira.com>
Signed-off-by: Simon Horman <horms at verge.net.au>
Signed-off-by: Ben Pfaff <blp at nicira.com>


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

Summary of changes:
 ofproto/ofproto-dpif-upcall.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


hooks/post-receive
-- 
Open vSwitch



More information about the git mailing list