alsa->thread = NULL;
}
- if (alsa->buffer)
- free(alsa->buffer);
+ free(alsa->buffer);
alsa->buffer = NULL;
alsa->receive = NULL;
void audin_set_subsystem(AUDIN_PLUGIN* audin, char* subsystem)
{
- if (audin->subsystem)
- free(audin->subsystem);
-
+ free(audin->subsystem);
audin->subsystem = _strdup(subsystem);
}
void audin_set_device_name(AUDIN_PLUGIN* audin, char* device_name)
{
- if (audin->device_name)
- free(audin->device_name);
-
+ free(audin->device_name);
audin->device_name = _strdup(device_name);
}
freerdp_dsp_context_free(opensles->dsp_context);
- if (opensles->device_name)
- free(opensles->device_name);
+ free(opensles->device_name);
free(opensles);
}
if (audin->dsp_context)
freerdp_dsp_context_free(audin->dsp_context);
- if (audin->context.client_formats)
- free(audin->context.client_formats);
-
+ free(audin->context.client_formats);
free(audin);
}
for (index = 0; index < formatList.numFormats; index++)
{
- if (formatList.formats[index].formatName)
- free(formatList.formats[index].formatName);
+ free(formatList.formats[index].formatName);
}
free(formatList.formats);
{
DISP_CHANNEL_CALLBACK* callback = (DISP_CHANNEL_CALLBACK*) pChannelCallback;
- if (callback)
- {
- free(callback);
- }
+ free(callback);
return 0;
}
{
DISP_PLUGIN* disp = (DISP_PLUGIN*) pPlugin;
- if (disp)
- {
- free(disp);
- }
+ free(disp);
return 0;
}
{
if (context)
{
- if (context->priv)
- {
- free(context->priv);
- }
-
+ free(context->priv);
free(context);
}
}
{
ECHO_PLUGIN* echo = (ECHO_PLUGIN*) pPlugin;
- if (echo)
- {
- free(echo);
- }
+ free(echo);
return 0;
}
{
if (context)
{
- if (context->priv)
- {
- free(context->priv);
- }
-
+ free(context->priv);
free(context);
}
}
WCHAR* buffer = NULL;
int length = 0;
- if (unicode_string->string != NULL)
- free(unicode_string->string);
+ free(unicode_string->string);
unicode_string->string = NULL;
unicode_string->length = 0;
}
}
- if (pKeys)
- free(pKeys);
+ free(pKeys);
pos = (int) Stream_GetPosition(s);
Stream_SetPosition(s, count_pos);
{
if (context)
{
- if (context->priv)
- {
- free(context->priv);
- }
-
+ free(context->priv);
free(context);
}
}
DeleteCriticalSection(&rdpei->lock);
- if (rdpei->listener_callback)
- free(rdpei->listener_callback);
-
+ free(rdpei->listener_callback);
free(rdpei->context);
-
free(rdpei);
return 0;
if (status < 0)
{
- if (h264.meta.regionRects)
- free(h264.meta.regionRects);
- if (h264.meta.quantQualityVals)
- free(h264.meta.quantQualityVals);
+ free(h264.meta.regionRects);
+ free(h264.meta.quantQualityVals);
return -1;
}
void rdpsnd_set_subsystem(rdpsndPlugin* rdpsnd, char* subsystem)
{
- if (rdpsnd->subsystem)
- free(rdpsnd->subsystem);
-
+ free(rdpsnd->subsystem);
rdpsnd->subsystem = _strdup(subsystem);
}
void rdpsnd_set_device_name(rdpsndPlugin* rdpsnd, char* device_name)
{
- if (rdpsnd->device_name)
- free(rdpsnd->device_name);
-
+ free(rdpsnd->device_name);
rdpsnd->device_name = _strdup(device_name);
}
winmm->device.WaveConfirm(&(winmm->device), wave);
- if (lpWaveHdr->lpData)
- free(lpWaveHdr->lpData);
+ free(lpWaveHdr->lpData);
free(wave);
}
if (context->priv->ChannelHandle)
WTSVirtualChannelClose(context->priv->ChannelHandle);
- if (context->priv->out_buffer)
- free(context->priv->out_buffer);
+ free(context->priv->out_buffer);
if (context->priv->dsp_context)
freerdp_dsp_context_free(context->priv->dsp_context);
- if (context->priv->input_stream)
- Stream_Free(context->priv->input_stream, TRUE);
+ if (context->priv->input_stream)
+ Stream_Free(context->priv->input_stream, TRUE);
- if (context->client_formats)
- free(context->client_formats);
+ free(context->client_formats);
free(context);
}
{
if (context)
{
- if (context->priv)
- {
- free(context->priv);
- }
-
+ free(context->priv);
free(context);
}
}
Stream_Write(irp->output, buffer, nbRead); /* ReadData */
}
- if (buffer)
- free(buffer);
+ free(buffer);
}
static void serial_process_irp_write(SERIAL_DEVICE* serial, IRP* irp)
/* Stream_Write_UINT8(irp->output, 0); /\* Padding (1 byte) *\/ */
/* } */
- if (InputBuffer != NULL)
- free(InputBuffer);
-
- if (OutputBuffer != NULL)
- free(OutputBuffer);
+ free(InputBuffer);
+ free(OutputBuffer);
}
static void serial_process_irp(SERIAL_DEVICE* serial, IRP* irp)
irp->IoStatus = STATUS_NO_MEMORY;
irp->Complete(irp);
- if (data)
- free(data);
+ free(data);
}
for (index = 0; index < call->cReaders; index++)
{
rgReaderState = &call->rgReaderStates[index];
-
- if (rgReaderState->szReader)
- free((void*) rgReaderState->szReader);
+ free(rgReaderState->szReader);
}
free(call->rgReaderStates);
for (index = 0; index < call->cReaders; index++)
{
rgReaderState = &call->rgReaderStates[index];
-
- if (rgReaderState->szReader)
- free((void*) rgReaderState->szReader);
+ free(rgReaderState->szReader);
}
free(call->rgReaderStates);
if (status)
return status;
- if (call->szReader)
- free(call->szReader);
+ free(call->szReader);
return ret.ReturnCode;
}
if (status)
return status;
- if (call->szReader)
- free(call->szReader);
+ free(call->szReader);
return ret.ReturnCode;
}
if (status != SCARD_S_SUCCESS)
return status;
- if (call->pbSendBuffer)
- free(call->pbSendBuffer);
-
- if (ret.pbRecvBuffer)
- free(ret.pbRecvBuffer);
-
- if (call->pioSendPci)
- free(call->pioSendPci);
-
- if (call->pioRecvPci)
- free(call->pioRecvPci);
+ free(call->pbSendBuffer);
+ free(ret.pbRecvBuffer);
+ free(call->pioSendPci);
+ free(call->pioRecvPci);
return ret.ReturnCode;
}
if (status != SCARD_S_SUCCESS)
return status;
- if (call->pvInBuffer)
- free(call->pvInBuffer);
-
- if (ret.pvOutBuffer)
- free(ret.pvOutBuffer);
+ free(call->pvInBuffer);
+ free(ret.pvOutBuffer);
return ret.ReturnCode;
}
TSMFFFmpegDecoder* mdecoder = (TSMFFFmpegDecoder*) decoder;
if (mdecoder->frame)
av_free(mdecoder->frame);
- if (mdecoder->decoded_data)
- free(mdecoder->decoded_data);
+
+ free(mdecoder->decoded_data);
+
if (mdecoder->codec_context)
{
if (mdecoder->prepared)
avcodec_close(mdecoder->codec_context);
- if (mdecoder->codec_context->extradata)
- free(mdecoder->codec_context->extradata);
+
+ free(mdecoder->codec_context->extradata);
av_free(mdecoder->codec_context);
}
free(decoder);
DEBUG_TSMF("");
- if (tsmf->listener_callback)
- free(tsmf->listener_callback);
-
+ free(tsmf->listener_callback);
free(tsmf);
return 0;
if (!sample)
return;
- if (sample->data)
- free(sample->data);
-
+ free(sample->data);
free(sample);
}
ArrayList_Clear(presentation->stream_list);
ArrayList_Free(presentation->stream_list);
- if (presentation->rects)
- free(presentation->rects);
+ free(presentation->rects);
ZeroMemory(presentation, sizeof(TSMF_PRESENTATION));
free(presentation);
void urbdrc_set_subsystem(URBDRC_PLUGIN* urbdrc, char* subsystem)
{
- if (urbdrc->subsystem)
- free(urbdrc->subsystem);
-
+ free(urbdrc->subsystem);
urbdrc->subsystem = _strdup(subsystem);
}
void android_event_key_free(ANDROID_EVENT_KEY* event)
{
- if (event != NULL)
- free(event);
+ free(event);
}
ANDROID_EVENT_KEY* android_event_unicodekey_new(UINT16 key)
void android_event_unicodekey_free(ANDROID_EVENT_KEY* event)
{
- if (event != NULL)
- free(event);
+ free(event);
}
ANDROID_EVENT_CURSOR* android_event_cursor_new(UINT16 flags, UINT16 x, UINT16 y)
void android_event_cursor_free(ANDROID_EVENT_CURSOR* event)
{
- if (event != NULL)
- free(event);
+ free(event);
}
ANDROID_EVENT* android_event_disconnect_new()
void android_event_disconnect_free(ANDROID_EVENT* event)
{
- if (event != NULL)
- free(event);
+ free(event);
}
ANDROID_EVENT_CLIPBOARD* android_event_clipboard_new(void* data, int data_length)
void android_event_clipboard_free(ANDROID_EVENT_CLIPBOARD* event)
{
- if (event != NULL)
+ if (event)
{
- if (event->data)
- {
- free(event->data);
- }
+ free(event->data);
free(event);
}
}
if (res == JNI_TRUE)
{
// read back string values
- if (*username != NULL)
- free(*username);
-
+ free(*username);
*username = get_string_from_string_builder(env, jstr1);
- if (*domain != NULL)
- free(*domain);
-
+ free(*domain);
*domain = get_string_from_string_builder(env, jstr2);
- if (*password == NULL)
- free(*password);
-
+ free(*password);
*password = get_string_from_string_builder(env, jstr3);
}
break;
}
}
-
- if (tIOHIDDeviceRefs)
- free(tIOHIDDeviceRefs);
-
+
+ free(tIOHIDDeviceRefs);
+
if (deviceCFSetRef)
{
CFRelease(deviceCFSetRef);
int i;
for (i = 0; i < argc; i++)
- {
- if (argv[i])
- free(argv[i]);
- }
-
+ free(argv[i]);
+
if (!is_connected)
return;
-
+
gdi_free(context->instance);
-
- if (pixel_data)
- free(pixel_data);
+
+ free(pixel_data);
}
- (void) drawRect:(NSRect)rect
{
if (instance)
{
- if (instance->iStream.lpVtbl)
- free(instance->iStream.lpVtbl);
-
+ free(instance->iStream.lpVtbl);
free(instance);
}
}
{
if (instance)
{
- if (instance->iDataObject.lpVtbl)
- free(instance->iDataObject.lpVtbl);
-
- if (instance->m_pFormatEtc)
- free(instance->m_pFormatEtc);
-
- if (instance->m_pStgMedium)
- free(instance->m_pStgMedium);
+ free(instance->iDataObject.lpVtbl);
+ free(instance->m_pFormatEtc);
+ free(instance->m_pStgMedium);
if (instance->m_pStream)
{
if (instance)
{
- if (instance->iEnumFORMATETC.lpVtbl)
- free(instance->iEnumFORMATETC.lpVtbl);
+ free(instance->iEnumFORMATETC.lpVtbl);
if (instance->m_pFormatEtc)
{
map->remote_format_id = 0;
map->local_format_id = 0;
- if (map->name)
- {
- free(map->name);
- map->name = NULL;
- }
+ free(map->name);
+ map->name = NULL;
}
}
{
for (i = 0; i < clipboard->nFiles; i++)
{
- if (clipboard->file_names[i])
- {
- free(clipboard->file_names[i]);
- clipboard->file_names[i] = NULL;
- }
+ free(clipboard->file_names[i]);
+ clipboard->file_names[i] = NULL;
}
}
{
for (i = 0; i < clipboard->nFiles; i++)
{
- if (clipboard->fileDescriptor[i])
- {
- free(clipboard->fileDescriptor[i]);
- clipboard->fileDescriptor[i] = NULL;
- }
+ free(clipboard->fileDescriptor[i]);
+ clipboard->fileDescriptor[i] = NULL;
}
}
clear_file_array(clipboard);
clear_format_map(clipboard);
- if (clipboard->file_names)
- free(clipboard->file_names);
- if (clipboard->fileDescriptor)
- free(clipboard->fileDescriptor);
- if (clipboard->format_mappings)
- free(clipboard->format_mappings);
+ free(clipboard->file_names);
+ free(clipboard->fileDescriptor);
+ free(clipboard->format_mappings);
free(clipboard);
}
XSetClassHint(xfc->display, appWindow->handle, class_hints);
XFree(class_hints);
- if (class)
- free(class);
+ free(class);
}
/* Set the input mode hint for the WM */
detect_status = 1;
}
- if (settings->ServerHostname)
- free(settings->ServerHostname);
-
+ free(settings->ServerHostname);
free(settings);
return detect_status;
status = fclose(fp);
}
- if (buffer)
- free(buffer);
+ free(buffer);
return (status == 0) ? TRUE : FALSE;
}
if (domain)
freerdp_set_param_string(settings, FreeRDP_Domain, domain);
- if (user)
- free(user);
-
- if (domain)
- free(domain);
+ free(user);
+ free(domain);
}
if (~((size_t) file->FullAddress))
if (index >= brushCache->maxMonoEntries)
{
WLog_ERR(TAG, "invalid brush (%d bpp) index: 0x%04X", bpp, index);
-
- if (entry)
- free(entry);
-
+ free(entry);
return;
}
- prevEntry = brushCache->monoEntries[index].entry;
-
- if (prevEntry != NULL)
- free(prevEntry);
+ free(brushCache->monoEntries[index].entry);
brushCache->monoEntries[index].bpp = bpp;
brushCache->monoEntries[index].entry = entry;
if (index >= brushCache->maxEntries)
{
WLog_ERR(TAG, "invalid brush (%d bpp) index: 0x%04X", bpp, index);
-
- if (entry)
- free(entry);
-
+ free(entry);
return;
}
- prevEntry = brushCache->entries[index].entry;
-
- if (prevEntry != NULL)
- free(prevEntry);
+ free(brushCache->entries[index].entry);
brushCache->entries[index].bpp = bpp;
brushCache->entries[index].entry = entry;
if (brushCache->entries)
{
for (i = 0; i < (int) brushCache->maxEntries; i++)
- {
- if (brushCache->entries[i].entry != NULL)
- free(brushCache->entries[i].entry);
- }
+ free(brushCache->entries[i].entry);
free(brushCache->entries);
}
if (brushCache->monoEntries)
{
for (i = 0; i < (int) brushCache->maxMonoEntries; i++)
- {
- if (brushCache->monoEntries[i].entry != NULL)
- free(brushCache->monoEntries[i].entry);
- }
+ free(brushCache->monoEntries[i].entry);
free(brushCache->monoEntries);
}
if (prevGlyph)
{
Glyph_Free(glyphCache->context, prevGlyph);
-
- if (prevGlyph->aj)
- free(prevGlyph->aj);
+ free(prevGlyph->aj);
free(prevGlyph);
}
glyphCache->fragCache.entries[index].fragment = fragment;
glyphCache->fragCache.entries[index].size = size;
- if (prevFragment)
- free(prevFragment);
+ free(prevFragment);
}
void glyph_cache_register_callbacks(rdpUpdate* update)
if (glyph)
{
Glyph_Free(glyphCache->context, glyph);
-
- if (glyph->aj)
- free(glyph->aj);
+ free(glyph->aj);
free(glyph);
-
glyphCache->glyphCache[i].entries[j] = NULL;
}
}
void nine_grid_cache_put(rdpNineGridCache* nine_grid, UINT32 index, void* entry)
{
- void* prevEntry;
-
if (index >= nine_grid->maxEntries)
{
WLog_ERR(TAG, "invalid NineGrid index: 0x%04X", index);
return;
}
- prevEntry = nine_grid->entries[index].entry;
-
- if (prevEntry != NULL)
- free(prevEntry);
-
+ free(nine_grid->entries[index].entry);
nine_grid->entries[index].entry = entry;
}
if (nine_grid->entries != NULL)
{
for (i = 0; i < (int) nine_grid->maxEntries; i++)
- {
- if (nine_grid->entries[i].entry != NULL)
- free(nine_grid->entries[i].entry);
- }
+ free(nine_grid->entries[i].entry);
free(nine_grid->entries);
}
if (index >= paletteCache->maxEntries)
{
WLog_ERR(TAG, "invalid color table index: 0x%04X", index);
-
- if (entry)
- free(entry);
-
+ free(entry);
return;
}
- if (paletteCache->entries[index].entry)
- free(paletteCache->entries[index].entry);
+ free(paletteCache->entries[index].entry);
paletteCache->entries[index].entry = entry;
}
UINT32 i;
for (i = 0; i< paletteCache->maxEntries; i++)
- {
- if (paletteCache->entries[i].entry)
- free(paletteCache->entries[i].entry);
- }
+ free(paletteCache->entries[i].entry);
free(paletteCache->entries);
free(paletteCache);
{
if (clrconv)
{
- if (clrconv->palette)
- free(clrconv->palette);
-
+ free(clrconv->palette);
free(clrconv);
}
}
{
if (context)
{
- if (context->resampled_buffer)
- free(context->resampled_buffer);
-
- if (context->adpcm_buffer)
- free(context->adpcm_buffer);
-
+ free(context->resampled_buffer);
+ free(context->adpcm_buffer);
free(context);
}
}
void ncrush_context_free(NCRUSH_CONTEXT* ncrush)
{
- if (ncrush)
- {
- free(ncrush);
- }
+ free(ncrush);
}
}
}
- if (context->BitmapData)
- free(context->BitmapData);
+ free(context->BitmapData);
BufferPool_Free(context->priv->PlanePool);
return progressive;
cleanup:
- if (progressive->rects)
- free(progressive->rects);
- if (progressive->tiles)
- free(progressive->tiles);
- if (progressive->quantVals)
- free(progressive->quantVals);
- if (progressive->quantProgVals)
- free(progressive->quantProgVals);
- if (progressive)
- free(progressive);
+ free(progressive->rects);
+ free(progressive->tiles);
+ free(progressive->quantVals);
+ free(progressive->quantProgVals);
+ free(progressive);
return NULL;
}
void zgfx_context_free(ZGFX_CONTEXT* zgfx)
{
- if (zgfx)
- {
- free(zgfx);
- }
+ free(zgfx);
}
break;
}
- if (tokens)
- free(tokens);
+
+ free(tokens);
+
return 1;
}
{
free(PasswordW);
free(PassStubW);
- if (pbIn)
- free (pbIn);
- if (pbOut)
- free (pbOut);
+ free(pbIn);
+ free(pbOut);
return NULL;
}
{
free(PasswordW);
free(PassStubW);
- free (pbIn);
- free (pbOut);
+ free(pbIn);
+ free(pbOut);
return NULL;
}
if (status < 0)
{
- if (PasswordW)
- free (PasswordW);
+ free(PasswordW);
return -1;
}
if (status != 1)
{
- if (PasswordW)
- free (PasswordW);
+ free(PasswordW);
return -1;
}
if (status != 1)
{
- if (PasswordW)
- free (PasswordW);
+ free(PasswordW);
return -1;
}
if (!pbOut)
{
- if (PasswordW)
- free (PasswordW);
+ free(PasswordW);
return -1;
}
if (status != 1)
{
- if (PasswordW)
- free (PasswordW);
- free (pbOut);
+ free(PasswordW);
+ free(pbOut);
return -1;
}
if (status != 1)
{
WLog_ERR(TAG, "EVP_DecryptFinal_ex failure");
- if (PasswordW)
- free (PasswordW);
- free (pbOut);
+ free(PasswordW);
+ free(pbOut);
return -1;
}
for (i = 0; i < (int)x509_cert_chain->count; i++)
{
- if (x509_cert_chain->array[i].data)
- free(x509_cert_chain->array[i].data);
+ free(x509_cert_chain->array[i].data);
}
free(x509_cert_chain->array);
DEBUG_LICENSE("modulus length:%d", (int) cert_info.ModulusLength);
- if (cert_info.Modulus)
- free(cert_info.Modulus);
+ free(cert_info.Modulus);
if (!ret)
{
out_free:
if (fp)
fclose(fp);
- if (buffer)
- free(buffer);
+ free(buffer);
free(key);
return NULL;
}
if (!key)
return;
- if (key->Modulus)
- free(key->Modulus);
-
+ free(key->Modulus);
free(key->PrivateExponent);
free(key);
}
certificate_free_x509_certificate_chain(certificate->x509_cert_chain);
- if (certificate->cert_info.Modulus)
- free(certificate->cert_info.Modulus);
+ free(certificate->cert_info.Modulus);
free(certificate);
}
/* encrypt client random */
- if (settings->ClientRandom)
- free(settings->ClientRandom);
+ free(settings->ClientRandom);
settings->ClientRandomLength = CLIENT_RANDOM_LENGTH;
settings->ClientRandom = malloc(settings->ClientRandomLength);
}
ret = TRUE;
end:
- if (crypt_client_random)
- free(crypt_client_random);
+ free(crypt_client_random);
return ret;
}
}
ret = TRUE;
end:
- if (crypt_client_random)
- free(crypt_client_random);
+ free(crypt_client_random);
end2:
- if (client_random)
- free(client_random);
+ free(client_random);
return ret;
}
*/
void freerdp_free(freerdp* instance)
{
- if (instance)
- {
- free(instance);
- }
+ free(instance);
}
FREERDP_API ULONG freerdp_get_transport_sent(rdpContext* context, BOOL resetCount) {
static void string_free(void* obj1)
{
- if (!obj1)
- return;
-
free(obj1);
}
Stream_Rewind(s, 1); /* don't include null terminator in length */
Stream_Length(s) = Stream_GetPosition(s);
return s;
-out_free:
- for (i = 0; i < 9; i++)
- {
- if (lines[i])
- free(lines[i]);
- }
+out_free:
+ for (i = 0; i < count; i++)
+ free(lines[i]);
free(lines);
return NULL;
if (!request)
return;
- if (request->AuthParam)
- free(request->AuthParam);
-
- if (request->AuthScheme)
- free(request->AuthScheme);
-
- if (request->Authorization)
- free(request->Authorization);
-
+ free(request->AuthParam);
+ free(request->AuthScheme);
+ free(request->Authorization);
free(request->Content);
free(request->Method);
free(request->URI);
-
free(request->TransferEncoding);
-
free(request);
}
if (!response)
return;
- for (i = 0; i < response->count; i++)
- {
- if (response->lines && response->lines[i])
+ if (response->lines)
+ for (i = 0; i < response->count; i++)
free(response->lines[i]);
- }
free(response->lines);
free(response->ReasonPhrase);
{
if (packet->tsgPacket.packetMsgResponse)
{
- if (packet->tsgPacket.packetMsgResponse->messagePacket.reauthMessage)
- free(packet->tsgPacket.packetMsgResponse->messagePacket.reauthMessage);
-
+ free(packet->tsgPacket.packetMsgResponse->messagePacket.reauthMessage);
free(packet->tsgPacket.packetMsgResponse);
}
void license_free_product_info(LICENSE_PRODUCT_INFO* productInfo)
{
- if (productInfo->pbCompanyName != NULL)
+ if (productInfo)
+ {
free(productInfo->pbCompanyName);
-
- if (productInfo->pbProductId != NULL)
free(productInfo->pbProductId);
-
- free(productInfo);
+ free(productInfo);
+ }
}
/**
void license_free_binary_blob(LICENSE_BLOB* blob)
{
- if (blob->data != NULL)
+ if (blob)
+ {
free(blob->data);
-
- free(blob);
+ free(blob);
+ }
}
/**
#endif
WLog_DBG(TAG, "accept");
- if (client)
- free(client);
+ free(client);
return FALSE;
}
break;
case Update_SetBounds:
- if (msg->wParam)
- free(msg->wParam);
+ free(msg->wParam);
break;
case Update_Synchronize:
break;
case Update_SuppressOutput:
- if (msg->lParam)
- free(msg->lParam);
+ free(msg->lParam);
break;
case Update_SurfaceCommand:
case PrimaryUpdate_FastGlyph:
{
FAST_GLYPH_ORDER* wParam = (FAST_GLYPH_ORDER*) msg->wParam;
- if (wParam->glyphData.aj)
- free(wParam->glyphData.aj);
+ free(wParam->glyphData.aj);
free(wParam);
}
break;
void input_message_proxy_free(rdpInputProxy* proxy)
{
- if (proxy)
- {
- free(proxy);
- }
+ free(proxy);
}
void metrics_free(rdpMetrics* metrics)
{
- if (!metrics)
- return;
-
free(metrics);
}
void nego_free(rdpNego* nego)
{
- free(nego->RoutingToken);
- free(nego->cookie);
- free(nego);
+ if (nego)
+ {
+ free(nego->RoutingToken);
+ free(nego->cookie);
+ free(nego);
+ }
}
/**
if (nego->cookie)
{
free(nego->cookie);
- nego->cookie = 0;
+ nego->cookie = NULL;
}
if (!cookie)
{
if (strlen(settings->PasswordHash) == 32)
{
- if (identity->Password)
- free(identity->Password);
+ free(identity->Password);
identity->PasswordLength = ConvertToUnicode(CP_UTF8, 0,
settings->PasswordHash, -1, &identity->Password, 0) - 1;
{
if (redirection)
{
- if (redirection->TsvUrl)
- free(redirection->TsvUrl);
-
- if (redirection->Username)
- free(redirection->Username);
-
- if (redirection->Domain)
- free(redirection->Domain);
-
- if (redirection->TargetFQDN)
- free(redirection->TargetFQDN);
-
- if (redirection->TargetNetBiosName)
- free(redirection->TargetNetBiosName);
-
- if (redirection->TargetNetAddress)
- free(redirection->TargetNetAddress);
-
- if (redirection->LoadBalanceInfo)
- free(redirection->LoadBalanceInfo);
-
- if (redirection->Password)
- free(redirection->Password);
+ free(redirection->TsvUrl);
+ free(redirection->Username);
+ free(redirection->Domain);
+ free(redirection->TargetFQDN);
+ free(redirection->TargetNetBiosName);
+ free(redirection->TargetNetAddress);
+ free(redirection->LoadBalanceInfo);
+ free(redirection->Password);
if (redirection->TargetNetAddresses)
{
for (i = 0; i < (int) redirection->TargetNetAddressesCount; i++)
{
- if (redirection->TargetNetAddresses[i])
- free(redirection->TargetNetAddresses[i]);
+ free(redirection->TargetNetAddresses[i]);
}
free(redirection->TargetNetAddresses);
free(update->primary->polyline.points);
free(update->primary->polygon_sc.points);
- if (update->primary->fast_glyph.glyphData.aj)
- free(update->primary->fast_glyph.glyphData.aj);
+ free(update->primary->fast_glyph.glyphData.aj);
free(update->primary);
free(update->secondary);
void crypto_rc4_free(CryptoRc4 rc4)
{
- if (rc4)
- free(rc4);
+ free(rc4);
}
CryptoDes3 crypto_des3_encrypt_init(const BYTE* key, const BYTE* ivec)
{
int i;
- if (lengths)
- free(lengths);
+ free(lengths);
if (alt_name)
{
}
#ifndef _WIN32
- if (common_name)
- free(common_name);
+ free(common_name);
#endif
if (alt_names)
void rdtk_button_free(rdtkButton* button)
{
- if (!button)
- return;
-
free(button);
}
return font;
cleanup:
- if (fontImageFile)
- free (fontImageFile);
- if (fontDescriptorFile)
- free (fontDescriptorFile);
+ free(fontImageFile);
+ free(fontDescriptorFile);
if (font)
{
if (font->image)
void rdtk_font_free(rdtkFont* font)
{
- if (!font)
- return;
-
free(font);
}
void rdtk_label_free(rdtkLabel* label)
{
- if (!label)
- return;
-
free(label);
}
void rdtk_text_field_free(rdtkTextField* textField)
{
- if (!textField)
- return;
-
free(textField);
}
void wfreerdp_server_free(wfServer* server)
{
- if (server)
- {
- free(server);
- }
+ free(server);
WSACleanup();
}
SHADOW_MSG_IN_REFRESH_OUTPUT* wParam;
wMessagePipe* MsgPipe = client->subsystem->MsgPipe;
- wParam = (SHADOW_MSG_IN_REFRESH_OUTPUT*) calloc(1, sizeof(SHADOW_MSG_IN_REFRESH_OUTPUT));
+ if (!areas)
+ return;
- if (!wParam || !areas)
- {
- if (wParam)
- free (wParam);
+ if (!(wParam = (SHADOW_MSG_IN_REFRESH_OUTPUT*) calloc(1, sizeof(SHADOW_MSG_IN_REFRESH_OUTPUT))))
return;
- }
wParam->numRects = (UINT32) count;
void ASN1_CloseModule(ASN1module_t pModule)
{
- if (!pModule)
- free(pModule);
+ free(pModule);
}
ASN1error_e ASN1_CreateEncoder(ASN1module_t pModule, ASN1encoding_t* ppEncoderInfo,
free(lpLocalDcb);
return TRUE;
-
- error_handle:
- if (lpLocalDcb)
- free(lpLocalDcb);
+error_handle:
+ free(lpLocalDcb);
return FALSE;
}
return TRUE;
- error_handle:
- if (storedDeviceName != NULL)
- free(storedDeviceName);
-
- if (storedTargetPath != NULL)
- free(storedTargetPath);
+error_handle:
+ free(storedDeviceName);
+ free(storedTargetPath);
LeaveCriticalSection(&_CommDevicesLock);
return FALSE;
certstore = (WINPR_CERTSTORE*) hCertStore;
- if (certstore)
- {
- free(certstore);
- }
+ free(certstore);
return TRUE;
}
BOOL FreeEnvironmentStringsA(LPCH lpszEnvironmentBlock)
{
- if (lpszEnvironmentBlock)
- free(lpszEnvironmentBlock);
+ free(lpszEnvironmentBlock);
return TRUE;
}
if (!tmp)
{
- if (lpszEnvironmentBlock)
- free(lpszEnvironmentBlock);
+ free(lpszEnvironmentBlock);
free (mergeStrings);
return NULL;
}
if (!tmp)
{
- if (lpszEnvironmentBlock)
- free(lpszEnvironmentBlock);
+ free(lpszEnvironmentBlock);
free (mergeStrings);
return NULL;
}
newEB = MergeEnvironmentStrings((LPCSTR) *envBlock, envstr);
free(envstr);
-
- if (*envBlock)
- free(*envBlock);
+ free(*envBlock);
*envBlock = newEB;
if (pNamedPipe->pfnUnrefNamedPipe)
pNamedPipe->pfnUnrefNamedPipe(pNamedPipe);
- if (pNamedPipe->lpFileName)
- free((void*)pNamedPipe->lpFileName);
-
- if (pNamedPipe->lpFilePath)
- free((void*)pNamedPipe->lpFilePath);
-
- if (pNamedPipe->name)
- free((void*)pNamedPipe->name);
-
+ free(pNamedPipe->lpFileName);
+ free(pNamedPipe->lpFilePath);
+ free(pNamedPipe->name);
free(pNamedPipe);
return TRUE;
if (pFileSearch)
{
- if (pFileSearch->lpPath)
- free(pFileSearch->lpPath);
-
- if (pFileSearch->lpPattern)
- free(pFileSearch->lpPattern);
+ free(pFileSearch->lpPath);
+ free(pFileSearch->lpPattern);
if (pFileSearch->pDir)
closedir(pFileSearch->pDir);
BOOL HeapFree(HANDLE hHeap, DWORD dwFlags, LPVOID lpMem)
{
free(lpMem);
- return 1;
+ return TRUE;
}
#endif
{
if (UnicodeString)
{
- if (UnicodeString->Buffer)
- free(UnicodeString->Buffer);
+ free(UnicodeString->Buffer);
UnicodeString->Length = 0;
UnicodeString->MaximumLength = 0;
if (pNamedPipe->pfnUnrefNamedPipe)
pNamedPipe->pfnUnrefNamedPipe(pNamedPipe);
- if (pNamedPipe->name)
- free(pNamedPipe->name);
-
- if (pNamedPipe->lpFileName)
- free((void*)pNamedPipe->lpFileName);
-
- if (pNamedPipe->lpFilePath)
- free((void*)pNamedPipe->lpFilePath);
+ free(pNamedPipe->name);
+ free(pNamedPipe->lpFileName);
+ free(pNamedPipe->lpFilePath);
if (pNamedPipe->serverfd != -1)
close(pNamedPipe->serverfd);
if (!pReadPipe || !pWritePipe)
{
- if (pReadPipe)
- free(pReadPipe);
-
- if (pWritePipe)
- free(pWritePipe);
-
+ free(pReadPipe);
+ free(pWritePipe);
return FALSE;
}
void credssp_ContextFree(CREDSSP_CONTEXT* context)
{
- if (!context)
- return;
-
free(context);
}
if (status <= 0)
{
- if (TargetName)
- free(TargetName);
+ free(TargetName);
return -1;
}
status = ntlm_InitializeSecurityContextW(phCredential, phContext, pszTargetNameW, fContextReq,
Reserved1, TargetDataRep, pInput, Reserved2, phNewContext, pOutput, pfContextAttr, ptsExpiry);
- if (pszTargetNameW)
- free(pszTargetNameW);
+ free(pszTargetNameW);
return status;
}
{
if (string->Length > 0)
{
- if (string->Buffer)
- free(string->Buffer);
+ free(string->Buffer);
string->Buffer = NULL;
string->Length = 0;
void negotiate_ContextFree(NEGOTIATE_CONTEXT* context)
{
- if (!context)
- return;
-
free(context);
}
void schannel_CredentialsFree(SCHANNEL_CREDENTIALS* credentials)
{
- if (!credentials)
- return;
-
free(credentials);
}
status = schannel_InitializeSecurityContextW(phCredential, phContext, pszTargetNameW, fContextReq,
Reserved1, TargetDataRep, pInput, Reserved2, phNewContext, pOutput, pfContextAttr, ptsExpiry);
- if (pszTargetNameW != NULL)
- free(pszTargetNameW);
+ free(pszTargetNameW);
return status;
}
void sspi_SecBufferFree(PSecBuffer SecBuffer)
{
+ if (!SecBuffer)
+ return;
+
free(SecBuffer->pvBuffer);
SecBuffer->pvBuffer = NULL;
SecBuffer->cbBuffer = 0;
void sspi_SecureHandleFree(SecHandle* handle)
{
- if (!handle)
- return;
-
free(handle);
}
identity->Flags = SEC_WINNT_AUTH_IDENTITY_UNICODE;
- if (identity->User)
- free(identity->User);
+ free(identity->User);
identity->User = (UINT16*) NULL;
identity->UserLength = 0;
identity->UserLength = (ULONG) (status - 1);
}
- if (identity->Domain)
- free(identity->Domain);
+ free(identity->Domain);
identity->Domain = (UINT16*) NULL;
identity->DomainLength = 0;
identity->DomainLength = (ULONG) (status - 1);
}
- if (identity->Password)
- free(identity->Password);
+ free(identity->Password);
identity->Password = NULL;
identity->PasswordLength = 0;
for (index = 0; index < (int) cPackages; index++)
{
- if (pPackageInfo[index].Name)
- free(pPackageInfo[index].Name);
-
- if (pPackageInfo[index].Comment)
- free(pPackageInfo[index].Comment);
+ free(pPackageInfo[index].Name);
+ free(pPackageInfo[index].Comment);
}
free(pPackageInfo);
{
SecPkgInfo* pPackageInfo = (SecPkgInfo*) contextBuffer;
- if (pPackageInfo->Name)
- free(pPackageInfo->Name);
-
- if (pPackageInfo->Comment)
- free(pPackageInfo->Comment);
+ if (!pPackageInfo)
+ return;
+ free(pPackageInfo->Name);
+ free(pPackageInfo->Comment);
free(pPackageInfo);
}
BOOL LogonUserCloseHandle(HANDLE handle) {
WINPR_ACCESS_TOKEN *token = (WINPR_ACCESS_TOKEN *) handle;
- if (!LogonUserIsHandled(handle))
+ if (!handle || !LogonUserIsHandled(handle))
return FALSE;
- if (token->Username)
- free(token->Username);
-
- if (token->Domain)
- free(token->Domain);
-
+ free(token->Username);
+ free(token->Domain);
free(token);
return TRUE;
CloseHandle(hTimer);
if (g_Event)
CloseHandle(g_Event);
- if (apcData)
- free(apcData);
+ free(apcData);
return status;
}
currentIndex = (p - lpCmdLine);
}
- if (lpEscapedCmdLine)
- free(lpEscapedCmdLine);
-
- if (lpEscapedChars)
- free(lpEscapedChars);
+ free(lpEscapedCmdLine);
+ free(lpEscapedChars);
*pNumArgs = numArgs;
return pArgs;
ret = TRUE;
finish:
- if (filename)
- {
- free(filename);
- }
+ free(filename);
if (pArgs)
{
}
}
- if (keys)
- free(keys);
+ free(keys);
ListDictionary_Unlock(thread_list);
}
wBitStream* bs = NULL;
bs = (wBitStream*) calloc(1, sizeof(wBitStream));
- if (bs)
- {
- }
-
return bs;
}
void BitStream_Free(wBitStream* bs)
{
- if (bs)
- {
- free(bs);
- }
+ free(bs);
}
if (pubSub->synchronized)
DeleteCriticalSection(&pubSub->lock);
- if (pubSub->events)
- free(pubSub->events);
-
+ free(pubSub->events);
free(pubSub);
}
}
if (!referenceTable->size)
{
- if (referenceTable->array)
- free(referenceTable->array);
+ free(referenceTable->array);
referenceTable->array = NULL;
return NULL;
}
#if defined(HAVE_EXECINFO_H)
t_execinfo *data = (t_execinfo *)buffer;
- if (data->buffer)
- free(data->buffer);
+ free(data->buffer);
free(data);
#elif defined(ANDROID)
t_corkscrew_data *data = (t_corkscrew_data *)buffer;
- if (data->buffer)
- free(data->buffer);
+ free(data->buffer);
free(data);
#elif defined(_WIN32) || defined(_WIN64)
{
t_win_stack *data = (t_win_stack*)buffer;
- if (data->stack)
- free(data->stack);
+ free(data->stack);
free(data);
}
#else
void lodepng_palette_clear(LodePNGColorMode* info)
{
- if(info->palette) free(info->palette);
+ free(info->palette);
info->palette = 0;
info->palettesize = 0;
}
static unsigned readChunk_PLTE(LodePNGColorMode* color, const unsigned char* data, size_t chunkLength)
{
unsigned pos = 0, i;
- if(color->palette) free(color->palette);
+ free(color->palette);
color->palettesize = chunkLength / 3;
color->palette = (unsigned char*)malloc(4 * color->palettesize);
if(!color->palette && color->palettesize)
if (s)
{
if (bFreeBuffer)
- {
- if (s->buffer)
- free(s->buffer);
- }
+ free(s->buffer);
free(s);
}
cmp = memcmp(a, b, asize);
cleanup:
- if (a)
- free(a);
- if (b)
- free(b);
+ free(a);
+ free(b);
return cmp;
}
winpr_image_free(image3, TRUE);
if (image4)
winpr_image_free(image4, TRUE);
- if (buffer)
- free(buffer);
+
+ free(buffer);
return ret;
}
{
if (appender)
{
- if (appender->FileName)
- free(appender->FileName);
-
- if (appender->FilePath)
- free(appender->FilePath);
-
- if (appender->FullFileName)
- free(appender->FullFileName);
-
+ free(appender->FileName);
+ free(appender->FilePath);
+ free(appender->FullFileName);
free(appender);
}
}
void WLog_CallbackAppender_Free(wLog* log, wLogCallbackAppender* appender)
{
- if (appender)
- {
- free(appender);
- }
+ free(appender);
}
{
if (appender)
{
- if (appender->FileName)
- free(appender->FileName);
-
- if (appender->FilePath)
- free(appender->FilePath);
-
- if (appender->FullFileName)
- free(appender->FullFileName);
-
+ free(appender->FileName);
+ free(appender->FilePath);
+ free(appender->FullFileName);
free(appender);
}
}
void WLog_Layout_SetPrefixFormat(wLog* log, wLogLayout* layout, const char* format)
{
- if (layout->FormatString)
- {
- free(layout->FormatString);
- layout->FormatString = NULL;
- }
+ free(layout->FormatString);
+ layout->FormatString = NULL;
if (format)
layout->FormatString = _strdup(format);
#endif
- if (msg)
- free(msg);
+ free(msg);
winpr_backtrace_free(bt);
}
free(pWnd->lpClassName);
- if (pWnd->lpWindowName)
- free(pWnd->lpWindowName);
+ free(pWnd->lpWindowName);
free(pWnd);
if (status < 0)
{
- if (x509_str)
- free(x509_str);
+ free(x509_str);
free(filename);
free(fullpath);
fclose (fp);