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

No backport needed.
---
 src/tools.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/src/tools.c b/src/tools.c
index 49398ae3d..12481ce1e 100644
--- 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;
 }
-- 
Egor Shestakov
egor ascii(0x40) ved1 ascii(0x2E) me



Reply via email to