diff mbox

[ovs-dev] ovs-vswitchd: call flow_extract for the resumed packet in nxt_resume

Message ID 56FD294A.8070103@redhat.com
State Accepted
Headers show

Commit Message

Numan Siddique March 31, 2016, 1:42 p.m. UTC
When the packet-in is resumed by vswitchd, it is not setting the
'dp_packet' offsets, because of which it is crashing while executing
the actions.

Signed-off-by: Numan Siddique <nusiddiq@redhat.com>
---
 ofproto/ofproto-dpif.c | 3 +++
 1 file changed, 3 insertions(+)

Comments

Ben Pfaff March 31, 2016, 3:46 p.m. UTC | #1
On Thu, Mar 31, 2016 at 07:12:34PM +0530, Numan Siddique wrote:
> When the packet-in is resumed by vswitchd, it is not setting the
> 'dp_packet' offsets, because of which it is crashing while executing
> the actions.
> 
> Signed-off-by: Numan Siddique <nusiddiq@redhat.com>

Thanks!  Applied.
diff mbox

Patch

diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c
index 99c96de..530c49a 100644
--- a/ofproto/ofproto-dpif.c
+++ b/ofproto/ofproto-dpif.c
@@ -4443,6 +4443,9 @@  nxt_resume(struct ofproto *ofproto_,
     dp_packet_init(&packet, pin->public.packet_len);
     dp_packet_put(&packet, pin->public.packet, pin->public.packet_len);
 
+    struct flow headers;
+    flow_extract(&packet, &headers);
+
     /* Execute the datapath actions on the packet. */
     struct dpif_execute execute = {
         .actions = odp_actions.data,