flow: Fix misaligned access.
authorBen Pfaff <blp@nicira.com>
Fri, 7 May 2010 18:43:18 +0000 (11:43 -0700)
committerBen Pfaff <blp@nicira.com>
Fri, 7 May 2010 21:36:08 +0000 (14:36 -0700)
The testsuite for the flow extractor triggered this.

With this commit, "make check" passes on SPARC.

lib/flow.c

index fb7cf32a5bed2fc8de94db586aeb49de4251c6aa..38ad72abcb5906f20383fa23af4bf17a06df5f70 100644 (file)
@@ -27,6 +27,7 @@
 #include "openflow/openflow.h"
 #include "openvswitch/datapath-protocol.h"
 #include "packets.h"
+#include "unaligned.h"
 #include "xtoxll.h"
 
 #include "vlog.h"
@@ -154,8 +155,8 @@ flow_extract(struct ofpbuf *packet, uint32_t tun_id, uint16_t in_port,
         if (flow->dl_type == htons(ETH_TYPE_IP)) {
             const struct ip_header *nh = pull_ip(&b);
             if (nh) {
-                flow->nw_src = nh->ip_src;
-                flow->nw_dst = nh->ip_dst;
+                flow->nw_src = get_unaligned_u32(&nh->ip_src);
+                flow->nw_dst = get_unaligned_u32(&nh->ip_dst);
                 flow->nw_tos = nh->ip_tos & IP_DSCP_MASK;
                 flow->nw_proto = nh->ip_proto;
                 packet->l4 = b.data;