Hi,

Compiled and tested, works as expected. One thing:


> +            if (tun_try_open_device(tt, *device_guid,
> device_instance_id_interface))
>              {
>                  break;
>              }
>

We leak tt->hand here if we don't break - in the next loop iteration we
overwrite tt->hand
and previous value left unclosed.

Maybe add

    CloseHandle(tt->hand)
    tt->hand = NULL;

after "if" statement and before "next" label ?

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

Reply via email to