From aa3d29e6ac19f9a135e16f241352e958934d0b01 Mon Sep 17 00:00:00 2001
From: Philippe Teuwen <phil@teuwen.org>
Date: Sun, 4 Oct 2020 15:11:36 +0200
Subject: [PATCH] CLIParamStrToBuf: add string null terminator

---
 client/deps/cliparser/cliparser.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/client/deps/cliparser/cliparser.c b/client/deps/cliparser/cliparser.c
index 4ec4eb303..8ce757792 100644
--- a/client/deps/cliparser/cliparser.c
+++ b/client/deps/cliparser/cliparser.c
@@ -251,12 +251,13 @@ int CLIParamStrToBuf(struct arg_str *argstr, uint8_t *data, int maxdatalen, int
     if (!ibuf)
         return 0;
 
-    if (ibuf > maxdatalen) {
+    if (ibuf + 1 > maxdatalen) {
+        printf("Parameter error: string too long, expect max %i chars\n", maxdatalen - 1);
         fflush(stdout);
         return 2;
     }
 
-    memcpy(data, tmp_buf, ibuf);
+    memcpy(data, tmp_buf, ibuf + 1);
     *datalen = ibuf;
     return 0;
 }