tests: fix leaks in test-io-channel-command
No need for strdup, fix leaks when socat is missing. Spotted by ASAN. Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com> Reviewed-by: "Daniel P. Berrange" <berrange@redhat.com> Reviewed-by: Greg Kurz <groug@kaod.org>
This commit is contained in:
parent
dc491fead0
commit
14324f585d
|
@ -29,8 +29,8 @@ static void test_io_channel_command_fifo(bool async)
|
||||||
#define TEST_FIFO "tests/test-io-channel-command.fifo"
|
#define TEST_FIFO "tests/test-io-channel-command.fifo"
|
||||||
QIOChannel *src, *dst;
|
QIOChannel *src, *dst;
|
||||||
QIOChannelTest *test;
|
QIOChannelTest *test;
|
||||||
char *srcfifo = g_strdup_printf("PIPE:%s,wronly", TEST_FIFO);
|
const char *srcfifo = "PIPE:" TEST_FIFO ",wronly";
|
||||||
char *dstfifo = g_strdup_printf("PIPE:%s,rdonly", TEST_FIFO);
|
const char *dstfifo = "PIPE:" TEST_FIFO ",rdonly";
|
||||||
const char *srcargv[] = {
|
const char *srcargv[] = {
|
||||||
"/bin/socat", "-", srcfifo, NULL,
|
"/bin/socat", "-", srcfifo, NULL,
|
||||||
};
|
};
|
||||||
|
@ -59,8 +59,6 @@ static void test_io_channel_command_fifo(bool async)
|
||||||
object_unref(OBJECT(src));
|
object_unref(OBJECT(src));
|
||||||
object_unref(OBJECT(dst));
|
object_unref(OBJECT(dst));
|
||||||
|
|
||||||
g_free(srcfifo);
|
|
||||||
g_free(dstfifo);
|
|
||||||
unlink(TEST_FIFO);
|
unlink(TEST_FIFO);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue