Fix ul_buffer_finalize().

ul_buffer_finalize() was off by one, and instead of overriding the
trailing ",", it overwrote the trailing nul-byte.

Signed-off-by: Gergely Nagy <algernon@balabit.hu>

1 files changed, 3 insertions, 2 deletions

diff --git a/lib/buffer.c b/lib/buffer.c
index 4f795eb..d538fda 100644
--- a/lib/buffer.c
+++ b/lib/buffer.c
@@ -82,13 +82,14 @@ ul_buffer_append (ul_buffer_t *buffer, const char *key, const char *value)
 char *
 ul_buffer_finalize (ul_buffer_t *buffer)
 {
-  if (buffer->msg[buffer->len] == ',')
-    buffer->msg[buffer->len] = '}';
+  if (buffer->msg[buffer->len - 1] == ',')
+    buffer->msg[buffer->len - 1] = '}';
   else
     {
       if (!_ul_buffer_ensure_size (buffer, buffer->len + 1))
         return NULL;
       buffer->msg[buffer->len++] = '}';
+      buffer->msg[buffer->len] = '\0';
     }
   return buffer->msg;
 }