I've stared at the code for quite a bit, and this looks good, and cleans up quite a few of the odd corner cases ("do not set signal if some other signal is there already") that have grown over time, like the one in proxy.c.
I have not manually tested this to see if I could break it, or reproduce one of the "known issues" - I might go back and do so, but today I am lacking time. I *have* fed this to the full set of client/server tests, FreeBSD, Linux, Linux with DCO, which do excercise the SIGUSR1/SIGTERM on the p2mp server a bit and SIGUSR1/SIGTERM on the client, and all is nicely well-behaved. Your patch has been applied to the master and release/2.6 branch. commit 3a7257925791a086c1ac88362a8eb422be518b14 (master) commit b3b1436955b9db8e557fc58b7e37ba3a881109a6 (release/2.6) Author: Selva Nair Date: Sun Jan 1 16:51:07 2023 -0500 Assign and honour signal priority order Signed-off-by: Selva Nair <selva.n...@gmail.com> Acked-by: Arne Schwabe <a...@rfc2549.org> Message-Id: <20230101215109.1521549-4-selva.n...@gmail.com> URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg25871.html Signed-off-by: Gert Doering <g...@greenie.muc.de> -- kind regards, Gert Doering _______________________________________________ Openvpn-devel mailing list Openvpn-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openvpn-devel