Merge pull request #3724 from chipitsine/master

make cppcheck a bit happier

channels/printer/client/printer_cups.c

@@ -100,6 +100,7 @@ static UINT printer_cups_write_printjob(rdpPrintJob* printjob, BYTE* data, int s
 
 		if (fwrite(data, 1, size, fp) < size)
 		{
+			fclose(fp);
 			return ERROR_INTERNAL_ERROR;
 			// FIXME once this function doesn't return void anymore!
 		}

server/Mac/mf_event.c

@@ -192,7 +192,10 @@ mfEventQueue* mf_event_queue_new()
 		event_queue->events = (mfEvent**) malloc(sizeof(mfEvent*) * event_queue->size);
 		
 		if (pipe(event_queue->pipe_fd) < 0)
+		{
+			free(event_queue);
 			return NULL;
+		}
 		
 		pthread_mutex_init(&(event_queue->mutex), NULL);
 	}

server/shadow/shadow_capture.c

@@ -230,7 +230,10 @@ rdpShadowCapture* shadow_capture_new(rdpShadowServer* server)
 	capture->server = server;
 
 	if (!InitializeCriticalSectionAndSpinCount(&(capture->lock), 4000))
+	{
+		free(capture);
 		return NULL;
+	}
 
 	return capture;
 }

winpr/libwinpr/sspi/test/TestSchannel.c

@@ -437,6 +437,7 @@ static void* schannel_test_server_thread(void* arg)
 	if (!(lpTokenOut = (BYTE*) malloc(cbMaxToken)))
 	{
 		printf("Memory allocation failed\n");
+		free(lpTokenIn);
 		return NULL;
 	}
 	fContextReq = ASC_REQ_STREAM |

winpr/libwinpr/thread/argv.c

@@ -195,7 +195,10 @@ LPSTR* CommandLineToArgvA(LPCSTR lpCmdLine, int* pNumArgs)
 	buffer = (char*) HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, maxBufferSize);
 
 	if (!buffer)
+	{
+		free(lpEscapedChars);
 		return NULL;
+	}
 
 	pArgs = (LPSTR*) buffer;
 	pOutput = (char*) &buffer[maxNumArgs * (sizeof(char*))];

winpr/libwinpr/utils/sam.c

@@ -309,7 +309,10 @@ WINPR_SAM_ENTRY* SamLookupUserW(WINPR_SAM* sam, LPWSTR User, UINT32 UserLength,
 		return NULL;
 
 	if (!SamLookupStart(sam))
+	{
+		free(entry);
 		return NULL;
+	}
 
 	while (sam->line != NULL)
 	{