Script 'mail_helper' called by obssrc Hello community, here is the log from the commit of package libssh for openSUSE:Factory checked in at 2025-06-04 20:27:56 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/libssh (Old) and /work/SRC/openSUSE:Factory/.libssh.new.16005 (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "libssh" Wed Jun 4 20:27:56 2025 rev:78 rq:1282109 version:0.11.1 Changes: -------- --- /work/SRC/openSUSE:Factory/libssh/libssh.changes 2025-04-25 22:18:31.933576958 +0200 +++ /work/SRC/openSUSE:Factory/.libssh.new.16005/libssh.changes 2025-06-04 20:27:57.147489128 +0200 @@ -1,0 +2,6 @@ +Thu May 29 19:31:17 UTC 2025 - Lucas Mulling <lucas.mull...@suse.com> + +- Fix hang in torture_session test (bsc#1243799) + * Add patch libssh-tests-Fix-an-issue-where-torture_session-request-a-SIGTERM-too-early.patch + +------------------------------------------------------------------- New: ---- libssh-tests-Fix-an-issue-where-torture_session-request-a-SIGTERM-too-early.patch BETA DEBUG BEGIN: New:- Fix hang in torture_session test (bsc#1243799) * Add patch libssh-tests-Fix-an-issue-where-torture_session-request-a-SIGTERM-too-early.patch BETA DEBUG END: ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ libssh.spec ++++++ --- /var/tmp/diff_new_pack.lmOnBi/_old 2025-06-04 20:27:57.651510011 +0200 +++ /var/tmp/diff_new_pack.lmOnBi/_new 2025-06-04 20:27:57.655510176 +0200 @@ -50,6 +50,8 @@ Patch1: libssh-CmakeLists-Fix-multiple-digit-major-version-for-OpenSSH.patch # PATCH-FIX-UPSTREAM: fix OpenSSH banner parsing Patch2: libssh-misc-Fix-OpenSSH-banner-parsing.patch +# PATCH-FIX-SUSE: fix hang in torture_channel tests (bsc#1243799) +Patch3: libssh-tests-Fix-an-issue-where-torture_session-request-a-SIGTERM-too-early.patch BuildRequires: cmake BuildRequires: gcc-c++ BuildRequires: krb5-devel ++++++ libssh-tests-Fix-an-issue-where-torture_session-request-a-SIGTERM-too-early.patch ++++++ >From 886e0288a7f27d11fc0020b48c5d4be997293b04 Mon Sep 17 00:00:00 2001 From: Lucas Mulling <lucas.mull...@suse.com> Date: Mon, 2 Jun 2025 11:48:58 -0300 Subject: [PATCH] tests: Fix an issue where torture_session request a SIGTERM too early Signed-off-by: Lucas Mulling <lucas.mull...@suse.com> --- tests/client/torture_session.c | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/tests/client/torture_session.c b/tests/client/torture_session.c index f95002f4..93d86995 100644 --- a/tests/client/torture_session.c +++ b/tests/client/torture_session.c @@ -447,15 +447,25 @@ static void torture_channel_exit_signal(void **state) /* Make the request, read parts with close */ rc = ssh_channel_request_exec(channel, request); assert_ssh_return_code(session, rc); + + /* FIX-SUSE bsc#1243799: Wait a bit before sending the SIGTERM. In real + * world use chases this should not happen has there will be network delays, + * since we are running the tests locally, it can happen that the server has + * yet to spawn the child (by this setting the correct group id). Meaning + * when we request TERM the correct gids are not setup and killpg will not + * work. + */ + sleep(1); + rc = ssh_channel_request_send_signal(channel, "TERM"); assert_ssh_return_code(session, rc); - exit_status = ssh_channel_get_exit_state(channel, + rc = ssh_channel_get_exit_state(channel, &exit_status, &exit_signal, &core_dumped); assert_ssh_return_code(session, rc); - assert_int_equal(exit_status, 0); + assert_int_equal(exit_status, (uint32_t)-1); assert_string_equal(exit_signal, "TERM"); SAFE_FREE(exit_signal); } -- 2.49.0