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

[Alexandre Julliard]

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

Author: Rémi Bernon 
Date:   Tue Mar 31 15:12:27 2020 +0200

winedbg: Force packet data to be NUL terminated.

Signed-off-by: Rémi Bernon 
Signed-off-by: Alexandre Julliard

---

programs/winedbg/gdbproxy.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/programs/winedbg/gdbproxy.c b/programs/winedbg/gdbproxy.c
index e2954c08dd..fde6c556c2 100644
--- a/programs/winedbg/gdbproxy.c
+++ b/programs/winedbg/gdbproxy.c
@@ -1201,7 +1201,6 @@ static enum packet_return packet_read_memory(struct gdb_context* gdbctx)
     SIZE_T              r = 0;

assert(gdbctx->in_trap);
-    /* FIXME:check in_packet_len for reading %p,%x */
     if (sscanf(gdbctx->in_packet, "%p,%x", &addr, &len) != 2) return packet_error;
     if (len <= 0) return packet_error;
     TRACE("Read %u bytes at %p\n", len, addr);
@@ -1794,6 +1793,7 @@ static BOOL extract_packets(struct gdb_context* gdbctx)
                 {
                     gdbctx->in_packet = gdbctx->in_buf + 2;
                     gdbctx->in_packet_len = plen - 1;
+                    gdbctx->in_packet[gdbctx->in_packet_len] = '\0';
                     ret = (packet_entries[i].handler)(gdbctx);
                 }
                 switch (ret & ~packet_last_f)