Fixed memory leak.

This commit is contained in:
Armin Novak 2015-12-10 13:57:05 +01:00
parent f0e3a2d845
commit 65062633c2

View File

@ -406,6 +406,7 @@ BOOL certificate_data_replace(rdpCertificateStore* certificate_store,
{ {
WLog_ERR(TAG, "SetFilePointer(%s) returned %s [%08X]", WLog_ERR(TAG, "SetFilePointer(%s) returned %s [%08X]",
certificate_store->file, strerror(errno), GetLastError()); certificate_store->file, strerror(errno), GetLastError());
free(data);
CloseHandle(fp); CloseHandle(fp);
return FALSE; return FALSE;
} }
@ -414,6 +415,7 @@ BOOL certificate_data_replace(rdpCertificateStore* certificate_store,
{ {
WLog_ERR(TAG, "SetEndOfFile(%s) returned %s [%08X]", WLog_ERR(TAG, "SetEndOfFile(%s) returned %s [%08X]",
certificate_store->file, strerror(errno), GetLastError()); certificate_store->file, strerror(errno), GetLastError());
free(data);
CloseHandle(fp); CloseHandle(fp);
return FALSE; return FALSE;
} }
@ -456,6 +458,7 @@ BOOL certificate_data_replace(rdpCertificateStore* certificate_store,
{ {
WLog_ERR(TAG, "malloc(%s) returned %s [%08X]", WLog_ERR(TAG, "malloc(%s) returned %s [%08X]",
certificate_store->file, strerror(errno), errno); certificate_store->file, strerror(errno), errno);
free(data);
CloseHandle(fp); CloseHandle(fp);
return FALSE; return FALSE;
} }
@ -465,6 +468,7 @@ BOOL certificate_data_replace(rdpCertificateStore* certificate_store,
WLog_ERR(TAG, "_snprintf(%s) returned %s [%08X]", WLog_ERR(TAG, "_snprintf(%s) returned %s [%08X]",
certificate_store->file, strerror(errno), errno); certificate_store->file, strerror(errno), errno);
free(tdata); free(tdata);
free(data);
CloseHandle(fp); CloseHandle(fp);
return FALSE; return FALSE;
} }
@ -473,6 +477,7 @@ BOOL certificate_data_replace(rdpCertificateStore* certificate_store,
WLog_ERR(TAG, "WriteFile(%s) returned %s [%08X]", WLog_ERR(TAG, "WriteFile(%s) returned %s [%08X]",
certificate_store->file, strerror(errno), errno); certificate_store->file, strerror(errno), errno);
free(tdata); free(tdata);
free(data);
CloseHandle(fp); CloseHandle(fp);
return FALSE; return FALSE;
} }