From 9a147d86715da50610c905d347c88ae10756c9f5 Mon Sep 17 00:00:00 2001 From: Paul Cruz Date: Wed, 5 Jul 2017 09:37:52 -0700 Subject: [PATCH] removed unnecessary checks for null pointer on free --- contrib/adaptive-compression/v2.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/contrib/adaptive-compression/v2.c b/contrib/adaptive-compression/v2.c index 9a050bf7d..df6bf5705 100644 --- a/contrib/adaptive-compression/v2.c +++ b/contrib/adaptive-compression/v2.c @@ -98,8 +98,8 @@ static void freeCompressionJobs(adaptCCtx* ctx) unsigned u; for (u=0; unumJobs; u++) { jobDescription job = ctx->jobs[u]; - if (job.dst.start) free(job.dst.start); - if (job.src.start) free(job.src.start); + free(job.dst.start); + free(job.src.start); } } @@ -237,8 +237,8 @@ static int createCompressionJob(adaptCCtx* ctx, BYTE* data, size_t srcSize) if (!job->src.start || !job->dst.start) { /* problem occurred, free things then return */ DISPLAY("Error: problem occurred during job creation\n"); - if (job->src.start) free(job->src.start); - if (job->dst.start) free(job->dst.start); + free(job->src.start); + free(job->dst.start); return 1; } memcpy(job->src.start, data, srcSize); @@ -332,6 +332,6 @@ cleanup: /* file compression completed */ ret |= (srcFile != NULL) ? fclose(srcFile) : 0; ret |= (ctx != NULL) ? freeCCtx(ctx) : 0; - if (src != NULL) free(src); + free(src); return ret; }