Indeed, since the previous patch has resolved the concurrency issues
by splitting request/response and notification socket, locking against
a recursion "on the same socket" is no longer needed because it can no
longer happen. Simpler code, less surprises ;-)
(That said, the split of ovpn_nl_recvmsgs() and ..._report() introduced
by 1737 is now obsolete again, and could be reverted...)
Tested on the DCO t_server testbed, with "well behaved" clients, and
with a gremlin horde.
Your patch has been applied to the master and release/2.7 branch
(considered part of the larger bugfix).
commit e6670b335b99bed5810cf2d0465279e208d14462 (master)
commit 028acc7ccb3a1385f5c067cab0c58fec8f1fccf4 (release/2.7)
Author: Antonio Quartulli
Date: Fri Sep 25 15:18:24 2026 +0200
dco_linux: drop the now-redundant __is_locked re-entrancy guard
Signed-off-by: Antonio Quartulli <[email protected]>
Acked-by: Razvan Cojocaru <[email protected]>
Gerrit URL: https://gerrit.openvpn.net/c/openvpn/+/1738
Message-Id: <[email protected]>
URL:
https://www.mail-archive.com/[email protected]/msg39499.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
