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

Alexandre Julliard julliard at winehq.org
Tue Mar 31 16:44:24 CDT 2020


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

Author: Rémi Bernon <rbernon at 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 at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at 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;
 }
 




More information about the wine-cvs mailing list