ofp-util: Make ofputil_encode_hello() return a message with correct length.
authorBen Pfaff <blp@nicira.com>
Fri, 9 Nov 2012 17:59:12 +0000 (09:59 -0800)
committerBen Pfaff <blp@nicira.com>
Fri, 9 Nov 2012 18:01:43 +0000 (10:01 -0800)
This doesn't fix a visible bug, because code to send OpenFlow messages
updates the length itself, but it still seems like the right thing to do.

Signed-off-by: Ben Pfaff <blp@nicira.com>
lib/ofp-util.c

index 14ac7c13d1ad5076afaf911c611459b6895f2935..dea4aeafce0ce006e6ec56b69fb9ecba3fd4eee6 100644 (file)
@@ -1188,6 +1188,8 @@ ofputil_encode_hello(uint32_t allowed_versions)
         oheh->type = htons(OFPHET_VERSIONBITMAP);
         oheh->length = htons(map_len + sizeof *oheh);
         *(ovs_be32 *)(oheh + 1) = htonl(allowed_versions);
+
+        ofpmsg_update_length(msg);
     }
 
     return msg;