From 66755c478cfe08f41b99ff487f01583eafcc8dc7 Mon Sep 17 00:00:00 2001 From: Jakub Jelen Date: Wed, 29 May 2019 18:11:26 +0200 Subject: [PATCH] tests: Do not free pcap context while it is in use by the server session Signed-off-by: Jakub Jelen Reviewed-by: Andreas Schneider --- tests/server/torture_server_auth_kbdint.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/tests/server/torture_server_auth_kbdint.c b/tests/server/torture_server_auth_kbdint.c index 9cb9a20c..4d038b9a 100644 --- a/tests/server/torture_server_auth_kbdint.c +++ b/tests/server/torture_server_auth_kbdint.c @@ -96,7 +96,11 @@ static void cleanup_pcap(struct session_data_st *sdata) return; } - ssh_pcap_file_free(sdata->pcap); + /* Do not free the pcap data context here since its ownership was + * transfered to the session object, which will take care of its cleanup. + * Morover it is still in use so we can very simply crash by freeing + * it here. + */ sdata->pcap = NULL; } #endif