cron2 has submitted this change. ( http://gerrit.openvpn.net/c/openvpn/+/1341?usp=email )
Change subject: manage: Correctly handle port 65535 in man_kill ...................................................................... manage: Correctly handle port 65535 in man_kill Commit c66d433111c958c42cc98f4ba12d3cfc1bd5b3b4 introduced an off-by-one when changing a literal to a constant. Github: fixes OpenVPN/openvpn#887 Change-Id: I19373408a82332d7b2e5c634e5509c8be3727b5a Signed-off-by: Frank Lichtenheld <[email protected]> Acked-by: Gert Doering <[email protected]> Gerrit URL: https://gerrit.openvpn.net/c/openvpn/+/1341 Message-Id: <[email protected]> Signed-off-by: Gert Doering <[email protected]> --- M src/openvpn/manage.c 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/openvpn/manage.c b/src/openvpn/manage.c index e46b5f2..6efa100 100644 --- a/src/openvpn/manage.c +++ b/src/openvpn/manage.c @@ -612,7 +612,7 @@ : (streq(p1, "udp")) ? PROTO_UDP : PROTO_NONE; - if ((port > 0 && port < UINT16_MAX) && (proto != PROTO_NONE)) + if ((port > 0 && port <= UINT16_MAX) && (proto != PROTO_NONE)) { n_killed = (*man->persist.callback.kill_by_addr)(man->persist.callback.arg, addr, (uint16_t)port, proto); -- To view, visit http://gerrit.openvpn.net/c/openvpn/+/1341?usp=email To unsubscribe, or for help writing mail filters, visit http://gerrit.openvpn.net/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: openvpn Gerrit-Branch: master Gerrit-Change-Id: I19373408a82332d7b2e5c634e5509c8be3727b5a Gerrit-Change-Number: 1341 Gerrit-PatchSet: 2 Gerrit-Owner: flichtenheld <[email protected]> Gerrit-Reviewer: cron2 <[email protected]> Gerrit-Reviewer: plaisthos <[email protected]> Gerrit-CC: openvpn-devel <[email protected]>
_______________________________________________ Openvpn-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openvpn-devel
