[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 03/28] tests: fix leaks in test-io-channel-comma
From: |
Daniel P. Berrange |
Subject: |
Re: [Qemu-devel] [PATCH 03/28] tests: fix leaks in test-io-channel-command |
Date: |
Tue, 7 Feb 2017 14:02:37 +0000 |
User-agent: |
Mutt/1.7.1 (2016-10-04) |
On Tue, Feb 07, 2017 at 05:51:46PM +0400, Marc-André Lureau wrote:
> No need for strdup, fix leaks when socat is missing.
>
> Spotted by ASAN.
>
> Cc: "Daniel P. Berrange" <address@hidden>
> Signed-off-by: Marc-André Lureau <address@hidden>
> ---
> tests/test-io-channel-command.c | 6 ++----
> 1 file changed, 2 insertions(+), 4 deletions(-)
ACK, I've added this to my queue, but if this merges via a
pull request for your entire series, that's fine too.
>
> diff --git a/tests/test-io-channel-command.c b/tests/test-io-channel-command.c
> index 1d1f461bed..46ce1ff01c 100644
> --- a/tests/test-io-channel-command.c
> +++ b/tests/test-io-channel-command.c
> @@ -29,8 +29,8 @@ static void test_io_channel_command_fifo(bool async)
> #define TEST_FIFO "tests/test-io-channel-command.fifo"
> QIOChannel *src, *dst;
> QIOChannelTest *test;
> - char *srcfifo = g_strdup_printf("PIPE:%s,wronly", TEST_FIFO);
> - char *dstfifo = g_strdup_printf("PIPE:%s,rdonly", TEST_FIFO);
> + const char *srcfifo = "PIPE:" TEST_FIFO ",wronly";
> + const char *dstfifo = "PIPE:" TEST_FIFO ",rdonly";
> const char *srcargv[] = {
> "/bin/socat", "-", srcfifo, NULL,
> };
> @@ -59,8 +59,6 @@ static void test_io_channel_command_fifo(bool async)
> object_unref(OBJECT(src));
> object_unref(OBJECT(dst));
>
> - g_free(srcfifo);
> - g_free(dstfifo);
> unlink(TEST_FIFO);
> }
Regards,
Daniel
--
|: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-manager.org :|
|: http://entangle-photo.org -o- http://search.cpan.org/~danberr/ :|
- [Qemu-devel] [PATCH 00/28] Various memory leak fixes, Marc-André Lureau, 2017/02/07
- [Qemu-devel] [PATCH 02/28] tests: fix qmp response leak, Marc-André Lureau, 2017/02/07
- [Qemu-devel] [PATCH 01/28] qtest: fix a memory leak, Marc-André Lureau, 2017/02/07
- [Qemu-devel] [PATCH 03/28] tests: fix leaks in test-io-channel-command, Marc-André Lureau, 2017/02/07
- [Qemu-devel] [PATCH 04/28] timer: use an inline function for free, Marc-André Lureau, 2017/02/07
- [Qemu-devel] [PATCH 05/28] tests: fix ptimer leaks, Marc-André Lureau, 2017/02/07
- [Qemu-devel] [PATCH 06/28] tests: fix endianness-test leaks, Marc-André Lureau, 2017/02/07
- [Qemu-devel] [PATCH 07/28] tests: fix q35-test leaks, Marc-André Lureau, 2017/02/07
- [Qemu-devel] [PATCH 08/28] tests: fix vhost-user-test leaks, Marc-André Lureau, 2017/02/07
- [Qemu-devel] [PATCH 09/28] tests: fix ide-test leaks, Marc-André Lureau, 2017/02/07