From 48efd47df8028c42afbb2bc38b7fbb7434d57cc3 Mon Sep 17 00:00:00 2001 From: Ben Pfaff Date: Tue, 12 Feb 2013 15:49:12 -0800 Subject: [PATCH] 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 Co-authored-by: Simon Horman Signed-off-by: Simon Horman --- lib/ofp-msgs.c | 2 ++ 1 file changed, 2 insertions(+) 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); -- 2.20.1