ofp-msgs: ofpmp_reserve() must add "more" flag when splitting stats.
authorBen Pfaff <blp@nicira.com>
Fri, 3 Aug 2012 22:56:02 +0000 (15:56 -0700)
committerBen Pfaff <blp@nicira.com>
Mon, 6 Aug 2012 16:26:57 +0000 (09:26 -0700)
commit79b8c36c58f979296ce2cec1e6d899fa8321bc21
tree842e8c12ab68c330d5e10a2480a7984a11d855d6
parent1448105187550e12fe806aa4863ffa5dc397afb8
ofp-msgs: ofpmp_reserve() must add "more" flag when splitting stats.

Commit 982697a4d2 "ofp-msgs: New approach to encoding and decoding OpenFlow
headers." introduced a bug in multipart message composition such that, when
a multipart message actually requires multiple parts, the OFPSF_REPLY_MORE
bit failed to be set on any of the parts, which in turn caused the receiver
to see a truncated reply.  This commit fixes the problem.

This commit also adds a test case that would have detected the problem and
fixes a minor bug in that test case (if an error actually occurred, then
the test case would hang).

Bug #12766.
Reported-by: James Schmidt <jschmidt@nicira.com>
Signed-off-by: Ben Pfaff <blp@nicira.com>
lib/ofp-msgs.c
tests/ofproto.at