Rémi Bernon : winedbg: Force read data to be NUL terminated.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 46ec0b956d6c8018fef598606fc31e3f9cd6d729
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=46ec0b956d6c8018fef598606...

Author: Rémi Bernon <rbernon(a)codeweavers.com>
Date:   Tue Mar 31 15:12:26 2020 +0200

winedbg: Force read data to be NUL terminated.

Signed-off-by: Rémi Bernon <rbernon(a)codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard(a)winehq.org>

---

 programs/winedbg/gdbproxy.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/programs/winedbg/gdbproxy.c b/programs/winedbg/gdbproxy.c
index 052e73b2ad..e2954c08dd 100644
--- a/programs/winedbg/gdbproxy.c
+++ b/programs/winedbg/gdbproxy.c
@@ -1848,12 +1848,14 @@ static int fetch_data(struct gdb_context* gdbctx)
         if (gdbctx->in_len + STEP > gdbctx->in_buf_alloc)
             gdbctx->in_buf = packet_realloc(gdbctx->in_buf, gdbctx->in_buf_alloc += STEP);
 #undef STEP
-        len = read(gdbctx->sock, gdbctx->in_buf + gdbctx->in_len, gdbctx->in_buf_alloc - gdbctx->in_len);
+        len = read(gdbctx->sock, gdbctx->in_buf + gdbctx->in_len, gdbctx->in_buf_alloc - gdbctx->in_len - 1);
         if (len <= 0) break;
         gdbctx->in_len += len;
         assert(gdbctx->in_len <= gdbctx->in_buf_alloc);
         if (len < gdbctx->in_buf_alloc - gdbctx->in_len) break;
     }
+
+    gdbctx->in_buf[gdbctx->in_len] = '\0';
     return gdbctx->in_len - in_len;
 }