From: Heiko Hund <[email protected]> Check if the list argument to ListContainsDomain() is NULL. Otherwise the call to wcsstr() will bail out.
Reported-by: Marc Heuse <[email protected]> Reported-by: [email protected] Change-Id: Icd7c7b08e317aefd91a60bfc62e92cd8707b6fac Signed-off-by: Heiko Hund <[email protected]> Acked-by: Gert Doering <[email protected]> Gerrit URL: https://gerrit.openvpn.net/c/openvpn/+/1393 --- This change was reviewed on Gerrit and approved by at least one developer. I request to merge it to master. Gerrit URL: https://gerrit.openvpn.net/c/openvpn/+/1393 This mail reflects revision 1 of this Change. Acked-by according to Gerrit (reflected above): Gert Doering <[email protected]> diff --git a/src/openvpnserv/interactive.c b/src/openvpnserv/interactive.c index ec80a30..006f748 100644 --- a/src/openvpnserv/interactive.c +++ b/src/openvpnserv/interactive.c @@ -2106,7 +2106,7 @@ ListContainsDomain(PCWSTR list, PCWSTR domain, size_t len) { PCWSTR match = list; - while (TRUE) + while (match) { match = wcsstr(match, domain); if (!match) _______________________________________________ Openvpn-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openvpn-devel
