datapath-windows: Fix subscribe/unsubscribe packets
authorAlin Serdean <aserdean@cloudbasesolutions.com>
Mon, 4 Jan 2016 23:04:11 +0000 (23:04 +0000)
committerBen Pfaff <blp@ovn.org>
Tue, 5 Jan 2016 00:04:32 +0000 (16:04 -0800)
The policy of the subscribe packets is defined by the following:
    const NL_POLICY policy[] =  {
        [OVS_NL_ATTR_PACKET_PID] = {.type = NL_A_U32 },
        [OVS_NL_ATTR_PACKET_SUBSCRIBE] = {.type = NL_A_U8 }
        };
Switch the value of the join operation with the one from the policy.

Signed-off-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com>
Signed-off-by: Ben Pfaff <blp@ovn.org>
datapath-windows/ovsext/User.c

index 42af7f3..04d2294 100644 (file)
@@ -1109,7 +1109,7 @@ fail:
  */
 NTSTATUS
 OvsSubscribePacketCmdHandler(POVS_USER_PARAMS_CONTEXT usrParamsCtx,
-                            UINT32 *replyLen)
+                             UINT32 *replyLen)
 {
     NDIS_STATUS status;
     BOOLEAN rc;
@@ -1135,7 +1135,7 @@ OvsSubscribePacketCmdHandler(POVS_USER_PARAMS_CONTEXT usrParamsCtx,
         goto done;
     }
 
-    join = NlAttrGetU8(attrs[OVS_NL_ATTR_PACKET_PID]);
+    join = NlAttrGetU8(attrs[OVS_NL_ATTR_PACKET_SUBSCRIBE]);
     pid = NlAttrGetU32(attrs[OVS_NL_ATTR_PACKET_PID]);
 
     /* The socket subscribed with must be the same socket we perform receive*/