Hi,

> -        msg(M_USAGE, "--dhcp-options requires --ip-win32 dynamic or 
> adaptive");
> +        msg(M_USAGE, "--dhcp-option requires --ip-win32 dynamic or 
> adaptive");

Nice, this typo has been there since at least 2005.

It looks like that warning is not quite correct - for example,
DNS can be set via iservice or netsh, which means that

--ip-win32 netsh
--dhcp-option DNS 8.8.8.8

is a valid combination regardless of driver type. On the other hand,
NBDD/NTP are set via DHCP.

It would be good to make the warning condition more fine-grained, but
this patch is good enough as it is, since it fixes important case - specify
DNS in the ovpn profile when using wintun.

Built and tested with MSVC.

Acked-by: Lev Stipakov <lstipa...@gmail.com>


_______________________________________________
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel

Reply via email to