Acked-by: Gert Doering <g...@greenie.muc.de>
This is basically the same thing as 1/5, just for the getaddrinfo()
related functions. And no, I have no idea why they are treated "special",
and getaddrinfo() is not a syscall. OTOH, getaddrinfo() spends most of
its time in "waiting for DNS packets", which might result in visible
behaviour similar to a restarted/restartable syscall on signals coming
in... the "let's rewrite that thing" part would be 2.7 material, though.
The code changes look reasonable ("straightforward as 1/5"), the test
rigs still works I do not have specific "test signal handling / timeout
in DNS" test cases today, though.
Your patch has been applied to the master and release/2.6 branch.
commit eff95d500481c7927c5a9edd6b5c0dfa056a0cbb (master)
commit 1ff87267f819b512c34a202b9a023c76e7031af4 (release/2.6)
Author: Selva Nair
Date: Sun Jan 1 16:51:06 2023 -0500
Refactor signal handling in openvpn_getaddrinfo
Signed-off-by: Selva Nair <selva.n...@gmail.com>
Acked-by: Gert Doering <g...@greenie.muc.de>
Message-Id: <20230101215109.1521549-3-selva.n...@gmail.com>
URL:
https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg25872.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
