The ret variable is freed twice, but on the second time we actually want to free ret3 instead. Don't know why this didn't explode.
Reported-by: Peter Maydell <peter.mayd...@linaro.org> Tested-by: Peter Maydell <peter.mayd...@linaro.org> Signed-off-by: Luiz Capitulino <lcapitul...@redhat.com> --- tests/test-qmp-commands.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/test-qmp-commands.c b/tests/test-qmp-commands.c index 8e62c2d..554e222 100644 --- a/tests/test-qmp-commands.c +++ b/tests/test-qmp-commands.c @@ -141,7 +141,7 @@ static void test_dispatch_cmd_io(void) ret3 = qobject_to_qint(test_qmp_dispatch(req)); assert(qint_get_int(ret3) == 66); - QDECREF(ret); + QDECREF(ret3); QDECREF(req); } -- 1.8.1.4