BUG/MINOR: sample: fix the closing bracket and LF in the debug converter

The closing bracket was emitted for the "debug" converter even when the
opening one was not sent, and the new line was not always emitted. Let's
fix this. This is harmless since this converter is not built by default.

(cherry picked from commit 50603267981a002d2593bfe219e5071d66a8ea65)
Signed-off-by: Willy Tarreau <w@1wt.eu>

diff --git a/src/sample.c b/src/sample.c
index 1e4039d..b3d28ca 100644 (file)
--- a/src/sample.c
+++ b/src/sample.c
@@ -1463,7 +1463,7 @@ static int sample_conv_debug(const struct arg *arg_p, struct sample *smp, void *
 
                        else {
                                /* Display the displayable chars*. */
-                               fprintf(stderr, "<");
+                               fputc('<', stderr);
                                for (i = 0; i < tmp.data.u.str.data; i++) {
                                        if (isprint(tmp.data.u.str.area[i]))
                                                fputc(tmp.data.u.str.area[i],
@@ -1471,9 +1471,10 @@ static int sample_conv_debug(const struct arg *arg_p, struct sample *smp, void *
                                        else
                                                fputc('.', stderr);
                                }
+                               fputc('>', stderr);
                        }
-                       fprintf(stderr, ">\n");
                }
+               fputc('\n', stderr);
        }
        return 1;
 }