From ad61c98d7a41f706eda54731080aef95f7d31fb4 Mon Sep 17 00:00:00 2001 From: Alin Serdean Date: Mon, 4 Jan 2016 23:04:11 +0000 Subject: [PATCH 1/1] datapath-windows: Fix subscribe/unsubscribe packets 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 Signed-off-by: Ben Pfaff --- datapath-windows/ovsext/User.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/datapath-windows/ovsext/User.c b/datapath-windows/ovsext/User.c index 42af7f32d..04d22949e 100644 --- a/datapath-windows/ovsext/User.c +++ b/datapath-windows/ovsext/User.c @@ -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*/ -- 2.20.1