We recently started running Coverity on our tests/ code, which has revealed a pile of issues that we were previously blissfully unaware of. This series fixes three easy ones: * use of free() where g_free() was meant * unnecessary NULL check * side effect inside g_assert() argument
thanks -- PMM Peter Maydell (3): tests/qtest/tpm-util.c: Free memory with correct free function tests/qtest/rtc-test: Remove pointless NULL check tests: Avoid side effects inside g_assert() arguments tests/qtest/ipmi-bt-test.c | 6 ++++-- tests/qtest/ipmi-kcs-test.c | 3 ++- tests/qtest/rtc-test.c | 6 ++---- tests/qtest/tpm-util.c | 4 ++-- 4 files changed, 10 insertions(+), 9 deletions(-) -- 2.20.1
