Re: [sdl-qemu] [PATCH] fix leaks found wtih fuzzing

2023-08-25 Thread Alexey Khoroshilov
On 25.08.2023 12:29, Dmitry Frolov wrote: > It is true, that there is no problem during runtime > from the first sight, because the memmory is lost just > before qemu exits. Nevertheless, this change is necessary, > because AddressSanitizer is not able to recognize this > situation and produces

Re: [PATCH] fix leaks found wtih fuzzing

2023-08-25 Thread Michael Tokarev
25.08.2023 14:58, Dmitry Frolov wrote: It is true, that there is no problem during runtime from the first sight, because the memory is lost just before qemu exits. Nevertheless, this change is necessary, because AddressSanitizer is not able to recognize this situation and produces crash-report

[PATCH] fix leaks found wtih fuzzing

2023-08-25 Thread Dmitry Frolov
It is true, that there is no problem during runtime from the first sight, because the memory is lost just before qemu exits. Nevertheless, this change is necessary, because AddressSanitizer is not able to recognize this situation and produces crash-report (which is false-positive in fact). Lots of

[PATCH] fix leaks found wtih fuzzing

2023-08-25 Thread Dmitry Frolov
It is true, that there is no problem during runtime from the first sight, because the memmory is lost just before qemu exits. Nevertheless, this change is necessary, because AddressSanitizer is not able to recognize this situation and produces crash-report (which is false-positive in fact). Lots

Re: [PATCH] fix leaks found wtih fuzzing

2023-08-24 Thread Peter Maydell
On Thu, 24 Aug 2023 at 17:28, Dmitry Frolov wrote: > > Fuzzing causes thousands of identical crashes with message: > "AddressSanitizer: 3744 byte(s) leaked in 1 allocation(s)" > > Fixes: 060ab76356 ("gtk: don't exit early in case gtk init fails") > > Signed-off-by: Dmitry Frolov > --- >

[PATCH] fix leaks found wtih fuzzing

2023-08-24 Thread Dmitry Frolov
Fuzzing causes thousands of identical crashes with message: "AddressSanitizer: 3744 byte(s) leaked in 1 allocation(s)" Fixes: 060ab76356 ("gtk: don't exit early in case gtk init fails") Signed-off-by: Dmitry Frolov --- ui/gtk.c | 8 1 file changed, 4 insertions(+), 4 deletions(-)