Attention is currently required from: flichtenheld, plaisthos.
Hello plaisthos, flichtenheld,
I'd like you to do a code review.
Please visit
http://gerrit.openvpn.net/c/openvpn/+/917?usp=email
to review the following change.
Change subject: Explicit-exit-notify and multisocket interaction
......................................................................
Explicit-exit-notify and multisocket interaction
Check signals after timeout handling to avoid
unusual delay when using explicit-exit-notify.
Github: #702
Change-Id: Id9dbb4d3c550bd4c13f6ee97e67749686457d9fc
Signed-off-by: Gianmarco De Gregori <[email protected]>
---
M src/openvpn/multi.c
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.openvpn.net:29418/openvpn refs/changes/17/917/1
diff --git a/src/openvpn/multi.c b/src/openvpn/multi.c
index a673ec1..8e2c4d4 100644
--- a/src/openvpn/multi.c
+++ b/src/openvpn/multi.c
@@ -4265,13 +4265,13 @@
{
/* process the I/O which triggered select */
multi_io_process_io(multi);
- MULTI_CHECK_SIG(multi);
}
else if (status == 0)
{
multi_io_action(multi, NULL, TA_TIMEOUT, false);
}
+ MULTI_CHECK_SIG(multi);
perf_pop();
}
}
--
To view, visit http://gerrit.openvpn.net/c/openvpn/+/917?usp=email
To unsubscribe, or for help writing mail filters, visit
http://gerrit.openvpn.net/settings
Gerrit-Project: openvpn
Gerrit-Branch: master
Gerrit-Change-Id: Id9dbb4d3c550bd4c13f6ee97e67749686457d9fc
Gerrit-Change-Number: 917
Gerrit-PatchSet: 1
Gerrit-Owner: its_Giaan <[email protected]>
Gerrit-Reviewer: flichtenheld <[email protected]>
Gerrit-Reviewer: plaisthos <[email protected]>
Gerrit-CC: openvpn-devel <[email protected]>
Gerrit-Attention: plaisthos <[email protected]>
Gerrit-Attention: flichtenheld <[email protected]>
Gerrit-MessageType: newchange
_______________________________________________
Openvpn-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openvpn-devel
