Tested this quite extensively up to v9 ;-) - by intentionally making
openvpn server misbehave (ignoring first 2 signals sent to it) with
this patch
--- a/src/openvpn/multi.c
+++ b/src/openvpn/multi.c
@@ -3864,6 +3864,10 @@ multi_push_restart_schedule_exit(struct multi_context
*m, bool next_server)
bool
multi_process_signal(struct multi_context *m)
{
+static int n=0;
+ if (n++ < 1 )
+ { msg(M_INFO, "MPS: ignore signal"); signal_reset(m->top.sig,0); return
false; }
+
.. and v9 now properly reports this as "test fail" plus shows the
(previously root-owned and unreadable) log file, so we can more easily
see from the buildbot fail mails what happened and if this is "infra
problem" or "incoming patch problem".
BB + GHA agree that this is fine, for the non-broken-server case ;-)
Your patch has been applied to the master branch.
commit 37db7fe37afb555de651314a04c537fd7fbaa280
Author: Samuli Seppänen
Date: Fri Oct 25 12:36:31 2024 +0200
t_server_null: forcibly kill misbehaving servers
Signed-off-by: Samuli Seppänen <[email protected]>
Acked-by: Gert Doering <[email protected]>
Message-Id: <[email protected]>
URL:
https://www.mail-archive.com/[email protected]/msg29655.html
Signed-off-by: Gert Doering <[email protected]>
--
kind regards,
Gert Doering
_______________________________________________
Openvpn-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openvpn-devel
