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

Reply via email to