Re: [PATCH] tests/unit/test-util-sockets: fix mem-leak on error object

2025-05-30 Thread Daniel P . Berrangé
On Mon, May 26, 2025 at 10:20:55AM -0700, Matheus Tavares Bernardino wrote: > The test fails with --enable-asan as the error struct is never freed. > In the case where the test expects a success but it fails, let's also > report the error for debugging (it will be freed internally). > > Fixes 316e

Re: [PATCH] tests/unit/test-util-sockets: fix mem-leak on error object

2025-05-27 Thread Juraj Marcin
Hi Matheus On 2025-05-26 10:20, Matheus Tavares Bernardino wrote: > The test fails with --enable-asan as the error struct is never freed. > In the case where the test expects a success but it fails, let's also > report the error for debugging (it will be freed internally). Oh, nice catch. Thanks!

[PATCH] tests/unit/test-util-sockets: fix mem-leak on error object

2025-05-26 Thread Matheus Tavares Bernardino
The test fails with --enable-asan as the error struct is never freed. In the case where the test expects a success but it fails, let's also report the error for debugging (it will be freed internally). Fixes 316e8ee8d6 ("util/qemu-sockets: Refactor inet_parse() to use QemuOpts") Signed-off-by: Mat