diff options
author | Sairam Venugopal <vsairam@vmware.com> | 2015-08-26 14:37:16 -0700 |
---|---|---|
committer | Ben Pfaff <blp@nicira.com> | 2015-08-26 16:37:39 -0700 |
commit | b8b00f0ce4044fd6cb535f3514f5f6b205bf50a2 (patch) | |
tree | 62c785e73d1a65183e6d21de3905935dee1ab56d /datapath-windows/ovsext/User.c | |
parent | 5e2e998a5ed76e7aab2cd5cdc277b59c160dd199 (diff) | |
download | openvswitch-b8b00f0ce4044fd6cb535f3514f5f6b205bf50a2.tar.gz |
datapath-windows: Fix iterator in NlAttrParse()
Signed-off-by: Sairam Venugopal <vsairam@vmware.com>
Acked-by: Nithin Raju <nithin@vmware.com>
Signed-off-by: Ben Pfaff <blp@nicira.com>
Diffstat (limited to 'datapath-windows/ovsext/User.c')
-rw-r--r-- | datapath-windows/ovsext/User.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/datapath-windows/ovsext/User.c b/datapath-windows/ovsext/User.c index 213e9c752..8045e9a63 100644 --- a/datapath-windows/ovsext/User.c +++ b/datapath-windows/ovsext/User.c @@ -48,6 +48,7 @@ OVS_USER_STATS ovsUserStats; static VOID _MapNlAttrToOvsPktExec(PNL_ATTR *nlAttrs, PNL_ATTR *keyAttrs, OvsPacketExecute *execute); extern NL_POLICY nlFlowKeyPolicy[]; +extern UINT32 nlFlowKeyPolicyLen; static __inline VOID OvsAcquirePidHashLock() @@ -339,7 +340,8 @@ OvsNlExecuteCmdHandler(POVS_USER_PARAMS_CONTEXT usrParamsCtx, /* Get all the top level Flow attributes */ if ((NlAttrParse(nlMsgHdr, attrOffset, NlMsgAttrsLen(nlMsgHdr), - nlPktExecPolicy, nlAttrs, ARRAY_SIZE(nlAttrs))) + nlPktExecPolicy, ARRAY_SIZE(nlPktExecPolicy), + nlAttrs, ARRAY_SIZE(nlAttrs))) != TRUE) { OVS_LOG_ERROR("Attr Parsing failed for msg: %p", nlMsgHdr); @@ -353,8 +355,8 @@ OvsNlExecuteCmdHandler(POVS_USER_PARAMS_CONTEXT usrParamsCtx, /* Get flow keys attributes */ if ((NlAttrParseNested(nlMsgHdr, keyAttrOffset, NlAttrLen(nlAttrs[OVS_PACKET_ATTR_KEY]), - nlFlowKeyPolicy, keyAttrs, - ARRAY_SIZE(keyAttrs))) != TRUE) { + nlFlowKeyPolicy, nlFlowKeyPolicyLen, + keyAttrs, ARRAY_SIZE(keyAttrs))) != TRUE) { OVS_LOG_ERROR("Key Attr Parsing failed for msg: %p", nlMsgHdr); status = STATUS_UNSUCCESSFUL; goto done; |