[KERNEL32]
authorPierre Schweitzer <pierre@reactos.org>
Sat, 24 Oct 2015 08:43:54 +0000 (08:43 +0000)
committerPierre Schweitzer <pierre@reactos.org>
Sat, 24 Oct 2015 08:43:54 +0000 (08:43 +0000)
Don't pass uninit pointer address to CsrFreeCaptureBuffer()

CID 1223181

svn path=/trunk/; revision=69664

reactos/dll/win32/kernel32/client/vdm.c

index 712b232..a3cb7cd 100644 (file)
@@ -101,7 +101,7 @@ BaseCheckVDM(IN ULONG BinaryType,
 {
     NTSTATUS Status;
     PBASE_CHECK_VDM CheckVdm = &ApiMessage->Data.CheckVDMRequest;
-    PCSR_CAPTURE_BUFFER CaptureBuffer;
+    PCSR_CAPTURE_BUFFER CaptureBuffer = NULL;
     PWCHAR CurrentDir = NULL;
     PWCHAR ShortAppName = NULL;
     PWCHAR ShortCurrentDir = NULL;
@@ -517,7 +517,7 @@ Cleanup:
     if (AnsiReserved) RtlFreeHeap(RtlGetProcessHeap(), 0, AnsiReserved);
 
     /* Free the capture buffer */
-    CsrFreeCaptureBuffer(CaptureBuffer);
+    if (CaptureBuffer) CsrFreeCaptureBuffer(CaptureBuffer);
 
     /* Free the current directory, if it was allocated here, and its short path */
     if (ShortCurrentDir) RtlFreeHeap(RtlGetProcessHeap(), 0, ShortCurrentDir);