Skip to content

Commit

Permalink
datapath-windows: Fix subscribe/unsubscribe packets
Browse files Browse the repository at this point in the history
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 <[email protected]>
Signed-off-by: Ben Pfaff <[email protected]>
  • Loading branch information
Alin Serdean authored and blp committed Jan 5, 2016
1 parent d8d1ef2 commit ad61c98
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions datapath-windows/ovsext/User.c
Original file line number Diff line number Diff line change
Expand Up @@ -1109,7 +1109,7 @@ OvsCreateQueueNlPacket(PVOID userData,
*/
NTSTATUS
OvsSubscribePacketCmdHandler(POVS_USER_PARAMS_CONTEXT usrParamsCtx,
UINT32 *replyLen)
UINT32 *replyLen)
{
NDIS_STATUS status;
BOOLEAN rc;
Expand All @@ -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*/
Expand Down

0 comments on commit ad61c98

Please sign in to comment.