Re: [Openvpn-devel] XP broken (wrt IPv6 in 2.3.9)

[Gert Doering]

Hi,

On Sun, Dec 27, 2015 at 03:19:46PM +, David Woodhouse wrote:
> On Sat, 2015-12-26 at 10:35 +0100, Gert Doering wrote:
> > but, unfortunately, it is not available in MinGW. 
> 
> Point of order: this is the kind of statement which should normally
> never be seen without an explicit reference to a filed bug. 

Yes, right you are.  Though I don't truly feel qualified to open 
bugs against mingw, as I do not follow their releases well enough
to know what *should* be where (I don't even know which windows 
version stuff is in...)

> In this case, however, I think it was fixed without a bug ever having
> been filed: http://sourceforge.net/p/mingw-w64/mingw-w64/ci/a883b47a
> 
> It was also backported to the 4.x branch since the 4.0.4 release and
> will be in 4.0.5 (and is in the Fedora packages already).

We might want to add it to our compat/ stuff...  (this is a case where
I find autoconf totally useful :-) - "if this header file is not present,
use compat-versionhelper.h instead").

But anyway.  Working on other corners right now, leaving this to the
windows team.

thanks for the pointers!

gert
-- 
USENET is *not* the non-clickable part of WWW!
   //www.muc.de/~gert/
Gert Doering - Munich, Germany g...@greenie.muc.de
fax: +49-89-35655025g...@net.informatik.tu-muenchen.de


signature.asc
Description: PGP signature

Re: [Openvpn-devel] XP broken (wrt IPv6 in 2.3.9)

[Gert Doering]

Hi,

On Sun, Dec 20, 2015 at 07:19:20PM +0100, Gert Doering wrote:
> > Moving to openvpn-devel.
> > 
> > We could probably detect XP only (or technically "less then Vista") by
> > checking that IsWindowsVistaOrGreater() == false which seems to be
> > simpler. It should fix the problem.
> 
> That would help with "part 2", right...  (XP has a function 
> "IsWindowsVistaOrGreater"?  wtf!?)

JFTR, Microsoft documentation says that this is available in 
 starting with Windows2000 (!) - but, unfortunately,
it is not available in MinGW.  Not a *real* problem as this seems to be
just a wrapper around windowsVersion() anyway...

gert

-- 
USENET is *not* the non-clickable part of WWW!
   //www.muc.de/~gert/
Gert Doering - Munich, Germany g...@greenie.muc.de
fax: +49-89-35655025g...@net.informatik.tu-muenchen.de


signature.asc
Description: PGP signature

Re: [Openvpn-devel] XP broken (wrt IPv6 in 2.3.9)

[ValdikSS]

I don't like the idea with additional win32-route option, it really should just 
use interface name on XP and index on Vista+.

On 12/20/2015 09:19 PM, Gert Doering wrote:
> Hi,
>
> On Sun, Dec 20, 2015 at 07:32:56PM +0200, Lev Stipakov wrote:
> The usual problem of  "attached you'll find... (nothing)".  Resend :-)
>
> That would help with "part 2", right...  (XP has a function 
> "IsWindowsVistaOrGreater"?  wtf!?)
>
> Maybe IV_PLAT_VER=, to be extended for other platforms if we want to.
>
> gert
>
>
>
> --
>
>
> ___
> Openvpn-devel mailing list
> Openvpn-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/openvpn-devel



signature.asc
Description: OpenPGP digital signature

Re: [Openvpn-devel] XP broken (wrt IPv6 in 2.3.9)

[Gert Doering]

Hi,

On Sun, Dec 20, 2015 at 07:32:56PM +0200, Lev Stipakov wrote:
> Screenshot seems to be lost in transmission.

The usual problem of  "attached you'll find... (nothing)".  Resend :-)

> Moving to openvpn-devel.
> 
> We could probably detect XP only (or technically "less then Vista") by
> checking that IsWindowsVistaOrGreater() == false which seems to be
> simpler. It should fix the problem.

That would help with "part 2", right...  (XP has a function 
"IsWindowsVistaOrGreater"?  wtf!?)

> However I like the idea that server will be able to know client's win
> version, which might help with troubleshooting. I wouldn't not change
> "IV_PLAT" value, so shall we add something like "IV_PLAT_WINVER=6.1"?

Maybe IV_PLAT_VER=, to be extended for other platforms if we want to.

gert

-- 
USENET is *not* the non-clickable part of WWW!
   //www.muc.de/~gert/
Gert Doering - Munich, Germany g...@greenie.muc.de
fax: +49-89-35655025g...@net.informatik.tu-muenchen.de


signature.asc
Description: PGP signature

Re: [Openvpn-devel] XP broken

[Lev Stipakov]

Hi,

Screenshot seems to be lost in transmission.

Moving to openvpn-devel.

We could probably detect XP only (or technically "less then Vista") by
checking that IsWindowsVistaOrGreater() == false which seems to be
simpler. It should fix the problem.

However I like the idea that server will be able to know client's win
version, which might help with troubleshooting. I wouldn't not change
"IV_PLAT" value, so shall we add something like "IV_PLAT_WINVER=6.1"?

-Lev

2015-12-20 15:46 GMT+02:00 Gert Doering :

> Unfortunately, this is a bit complicated.
>
> First, XP really doesn't *do* interface indexes in netsh (*rant*) - I
> have attached a screen shot.  German, unfortunately, but "Zeichenkette"
> is "string", and the example clearly has an interface name there...
>
> We *could* introduce yet another option ("--ip-win32 netsh-names"),
> but the usefulness of that would depend on "whoever writes the .conf file
> knows what client OS it is for" (plus, we add an option that 2.4 won't
> have because it has no XP support anyway).  This does not sound like
> the way forward.
>
> So, I think we need to do this in two steps:
>
>  - add (in win32.c) a "win32_version_info()" function that will do the
>necessary things to figure out what Windows version we're running on.
>Stackoverflow suggests that this is complicated, as Windows will lie
>to programs most of the time, so "Win10" is reported as "Win8" unless
>there is something in the manifest (whatever that is) that tells windows
>"this program is prepared to deal with THE TRUTH"
>
>  - while at it, add "win32_version_string()" which would produce a printed
>version of this - so we can log it at startup, to help people diagnose
>why something isn't working for a user - and we could send it to the
>server as well if --push-peer-info is enabled (ssl.c, push_peer_info())
>
>  - and then, make the interface index bit conditional...
>
> if ( win32_version_info() == WIN_XP )
> device = tt->actual;
> else
>   {
> buf_printf (, "interface=%d", tt->adapter_index );
> device = buf_bptr();
>   }
>
>(for add/delete_route_ipv6(), but ifconfig would look the same)
>
>
> the win32_version_* stuff would go into 2.3 and master, because having
> that information is generally useful - the conditional setting only into
> 2.3.10
>
> what do you think?

-- 
-Lev