On 09/04/2018 07:58, Stefan Hajnoczi wrote: > On Sun, Apr 08, 2018 at 11:59:33AM -0300, Philippe Mathieu-Daudé wrote: >> memtohex() adds an extra trailing NUL character. >> >> Reported-by: AddressSanitizer >> Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> >> --- >> (gdb) dump binary memory /tmp/dram.bin 0x94000000 0x94100000 >> Remote connection closed >> >> ================================================================= >> ==22732==ERROR: AddressSanitizer: stack-buffer-overflow on address >> 0x7ffe43018340 at pc 0x55f2655fde81 bp 0x7ffe43017210 sp 0x7ffe43017208 >> WRITE of size 1 at 0x7ffe43018340 thread T0 >> >> --- >> gdbstub.c | 4 +++- >> 1 file changed, 3 insertions(+), 1 deletion(-) > > Reviewed-by: Stefan Hajnoczi <stefa...@redhat.com>
Peter, can you apply this directly to master? Thanks, Paolo
signature.asc
Description: OpenPGP digital signature