datapath-windows: Remove setting of replyLen to zero.
authorAnkur Sharma <ankursharma@vmware.com>
Sat, 11 Oct 2014 22:07:39 +0000 (15:07 -0700)
committerBen Pfaff <blp@nicira.com>
Mon, 13 Oct 2014 21:23:06 +0000 (14:23 -0700)
This was one of the review comment which i forgot to address in
FLOW_DUMP checkin. We do not need to explicitly set replyLen to zero
 as caller would have already set it.

Signed-off-by: Ankur Sharma <ankursharma@vmware.com>
Acked-by: Nithin Raju <nithin@vmware.com>
Signed-off-by: Ben Pfaff <blp@nicira.com>
datapath-windows/ovsext/Flow.c

index 73ade92..848f6bb 100644 (file)
@@ -269,8 +269,6 @@ OvsFlowNlCmdHandler(POVS_USER_PARAMS_CONTEXT usrParamsCtx,
     RtlZeroMemory(&stats, sizeof(stats));
     RtlZeroMemory(&replyStats, sizeof(replyStats));
 
-    *replyLen = 0;
-
     if (!(usrParamsCtx->outputBuffer)) {
         /* No output buffer */
         rc = STATUS_INVALID_BUFFER_SIZE;
@@ -387,8 +385,6 @@ OvsFlowNlGetCmdHandler(POVS_USER_PARAMS_CONTEXT usrParamsCtx,
 
     NL_BUFFER nlBuf;
 
-    *replyLen = 0;
-
     if (!(usrParamsCtx->outputBuffer)) {
         /* No output buffer */
         rc = STATUS_INVALID_BUFFER_SIZE;
@@ -463,8 +459,6 @@ _FlowNlGetCmdHandler(POVS_USER_PARAMS_CONTEXT usrParamsCtx,
     UINT32 keyAttrOffset = 0;
     UINT32 tunnelKeyAttrOffset = 0;
 
-    *replyLen = 0;
-
     if (usrParamsCtx->inputLength > usrParamsCtx->outputLength) {
         /* Should not be the case.
          * We'll be copying the flow keys back from