From: Ben Pfaff <blp@nicira.com>
Date: Tue, 12 Feb 2013 23:49:12 +0000 (-0800)
Subject: ofp-msgs: ensure that l2 is set in ofpmp_reserve()
X-Git-Tag: v1.9.0~2
X-Git-Url: http://git.cascardo.eti.br/?p=cascardo%2Fovs.git;a=commitdiff_plain;h=48efd47df8028c42afbb2bc38b7fbb7434d57cc3

ofp-msgs: ensure that l2 is set in ofpmp_reserve()

Ensure that the buffer returned by ofpmp_reserve() has buf->l2 set
as this may be required by nxm_reg_load_to_nxast() when generating
the reply to an stats request

This problem was observed when dumping a large number of flows
with set_field actions using ovs-ofctl dump-flows.

Signed-off-by: Ben Pfaff <blp@nicira.com>
Co-authored-by: Simon Horman <horms@verge.net.au>
Signed-off-by: Simon Horman <horms@verge.net.au>
---

diff --git a/lib/ofp-msgs.c b/lib/ofp-msgs.c
index 00e1a8448..702f68f96 100644
--- a/lib/ofp-msgs.c
+++ b/lib/ofp-msgs.c
@@ -832,6 +832,8 @@ ofpmp_reserve(struct list *replies, size_t len)
 
         next = ofpbuf_new(MAX(1024, hdrs_len + len));
         ofpbuf_put(next, msg->data, hdrs_len);
+        next->l2 = next->data;
+        next->l3 = ofpbuf_tail(next);
         list_push_back(replies, &next->list_node);
 
         *ofpmp_flags__(msg->data) |= htons(OFPSF_REPLY_MORE);