From: Willy Tarreau <w@1wt.eu>
Date: Wed, 10 Oct 2012 06:56:47 +0000 (+0200)
Subject: MINOR: standard: make indent_msg() support empty messages
X-Git-Tag: v1.5-dev13~171
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=70eec3832f5009c3bc3fa7cf57a44902b5a16928;p=thirdparty%2Fhaproxy.git

MINOR: standard: make indent_msg() support empty messages

indent_msg() is called with dynamically generated messages, so these
may be empty (NULL) when an empty list is being dumped. Support this
and return a NULL too.
---

diff --git a/src/standard.c b/src/standard.c
index 49810fccac..287931adf3 100644
--- a/src/standard.c
+++ b/src/standard.c
@@ -1840,7 +1840,8 @@ char *memprintf(char **out, const char *format, ...)
 
 /* Used to add <level> spaces before each line of <out>, unless there is only one line.
  * The input argument is automatically freed and reassigned. The result will have to be
- * freed by the caller.
+ * freed by the caller. It also supports being passed a NULL which results in the same
+ * output.
  * Example of use :
  *   parse(cmd, &err); (callee: memprintf(&err, ...))
  *   fprintf(stderr, "Parser said: %s\n", indent_error(&err));
@@ -1854,6 +1855,9 @@ char *indent_msg(char **out, int level)
 	int lastlf = 0;
 	int len;
 
+	if (!out || !*out)
+		return NULL;
+
 	in = *out - 1;
 	while ((in = strchr(in + 1, '\n')) != NULL) {
 		lastlf = in - *out;