Acked-by: Frank Lichtenheld <fr...@lichtenheld.com>
Seems trivial.
> Antonio Quartulli <a...@unstable.cc> hat am 02.04.2022 09:09 geschrieben:
[...]
> + if (tt->hand != NULL)
> + {
> + dmsg(D_WIN32_IO_LOW, "Attempting CloseHandle on %s adapter",
> adaptertype);
> + if (!CloseHandle(tt->hand))
> + {
> + msg(M_WARN | M_ERRNO, "Warning: CloseHandle failed on %s
> adapter", adaptertype);
> + }
> + tt->hand = NULL;
Setting hand to NULL is a slight behavior change, it wasn't there before.
It might be nice to acknowledge it in the commit message just to confirm that
it was a deliberate change.
Regards,
--
Frank Lichtenheld
_______________________________________________
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel
