From 765597e31f7b79267dbe53a9a8e01ae8c12bf4e8 Mon Sep 17 00:00:00 2001 From: Andreas Schneider Date: Mon, 26 Feb 2024 14:01:41 +0100 Subject: [PATCH] tests:client: We need to set channel to NULL after we freed it This fixes an invalid memory read in ssh_channel_get_exit_status() below. Signed-off-by: Andreas Schneider --- tests/client/torture_session.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/client/torture_session.c b/tests/client/torture_session.c index 8a43e586..57f53148 100644 --- a/tests/client/torture_session.c +++ b/tests/client/torture_session.c @@ -434,7 +434,7 @@ static void torture_freed_channel_get_exit_status(void **state) (channel->flags & SSH_CHANNEL_FLAG_NOT_BOUND)) { channel_freed = true; } - ssh_channel_free(channel); + SSH_CHANNEL_FREE(channel); if (!channel_freed) { rc = ssh_channel_get_exit_status(channel);