Acked-by: Frank Lichtenheld <[email protected]> Seems trivial.
> Antonio Quartulli <[email protected]> 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 [email protected] https://lists.sourceforge.net/lists/listinfo/openvpn-devel
