Remove null-pointer checks before free()

These are unnecessary, and the coding style was used inconsistently
and randomly.

test/test_crypto.c

@@ -108,8 +108,7 @@ static const char *run_hash(const char *str, const char *hexstr, const struct Di
 
 	digest_free(ctx);
 
-	if (buf)
-		free(buf);
+	free(buf);
 
 	if (memcmp(res, res2, reslen) != 0)
 		return "FAIL";

test/test_tls.c

@@ -930,8 +930,7 @@ static const char *do_verify(const char *hostname, const char *commonName, ...)
 	va_end(ap);
 
 	ret = tls_check_servername(&ctx, &cert, hostname);
-	if (ctx.errmsg)
-		free(ctx.errmsg);
+	free(ctx.errmsg);
 	if (ret == 0)
 		return "OK";
 	if (ret == -1)

test/tinytest_demo.c

@@ -158,8 +158,7 @@ test_memcpy(void *ptr)
 
  end:
 	/* This time our end block has something to do. */
-	if (mem)
-		free(mem);
+	free(mem);
 }
 
 /* ============================================================ */

usual/cfparser.c

@@ -369,8 +369,7 @@ static bool load_handler(void *arg, bool is_sect, const char *key, const char *v
 	struct LoaderCtx *ctx = arg;
 
 	if (is_sect) {
-		if (ctx->cur_sect)
-			free(ctx->cur_sect);
+		free(ctx->cur_sect);
 		ctx->cur_sect = strdup(key);
 		if (!ctx->cur_sect)
 			return false;
@@ -391,8 +390,7 @@ bool cf_load_file(const struct CfContext *cf, const char *fn)
 	ctx.cf = cf;
 
 	ok = parse_ini_file(fn, load_handler, &ctx);
-	if (ctx.cur_sect)
-		free(ctx.cur_sect);
+	free(ctx.cur_sect);
 	if (ok && !ctx.got_main_sect) {
 		log_error("load_init_file: main section missing from config file");
 		return false;
@@ -449,8 +447,7 @@ bool cf_set_str(struct CfValue *cv, const char *value)
 		log_error("cf_set_str: no mem");
 		return false;
 	}
-	if (*dst_p)
-		free(*dst_p);
+	free(*dst_p);
 	*dst_p = tmp;
 	return true;
 }
@@ -506,8 +503,7 @@ bool cf_set_filename(struct CfValue *cv, const char *value)
 
 	log_debug("expanded '%s' -> '%s'", value, tmp);
 
-	if (*dst_p)
-		free(*dst_p);
+	free(*dst_p);
 	*dst_p = tmp;
 	return true;
 fail:

usual/daemon.c

@@ -119,8 +119,7 @@ static void write_pidfile(const char *pidfile, bool first_write)
 	if (!pidfile || !pidfile[0])
 		return;
 
-	if (g_pidfile)
-		free(g_pidfile);
+	free(g_pidfile);
 	g_pidfile = strdup(pidfile);
 	if (!g_pidfile)
 		die("out of memory");