MINOR: tools: memvprintf(): remove <out> check that always true

memvprintf() exits early if the <out> is NULL, so the further NULL check is
redundant.

No backport needed.

diff --git a/src/tools.c b/src/tools.c
index 49398ae3d9e2275ecd8ef2e1fecc42d3a707f664..12481ce1eecf75f1e531e5d9ce44ac56c6908a9d 100644 (file)
--- a/src/tools.c
+++ b/src/tools.c
@@ -4678,10 +4678,8 @@ char *memvprintf(char **out, const char *format, va_list orig_args)
                ha_free(&ret);
        }
 
-       if (out) {
-               free(*out);
-               *out = ret;
-       }
+       free(*out);
+       *out = ret;
 
        return ret;
 }