[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

[tor-commits] [tor/master] doc: Fix an incorrect comment about calling FreeLibrary() on Windows



commit f186f21a4efb9dafe2c29f850126d509079c0c37
Author: teor <teor@xxxxxxxxxxxxxx>
Date:   Mon Mar 4 11:22:02 2019 +1000

    doc: Fix an incorrect comment about calling FreeLibrary() on Windows
    
    There's an incorrect comment in compat_time.c that suggests we call
    FreeLibrary() before we're done using the library's functions.
    See 29642 for background.
    
    Closes ticket 29643.
---
 src/lib/time/compat_time.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/lib/time/compat_time.c b/src/lib/time/compat_time.c
index 33e077a58..c6625c780 100644
--- a/src/lib/time/compat_time.c
+++ b/src/lib/time/compat_time.c
@@ -522,7 +522,9 @@ monotime_init_internal(void)
     GetTickCount64_fn = (GetTickCount64_fn_t)
       GetProcAddress(h, "GetTickCount64");
   }
-  // FreeLibrary(h) ?
+  // We can't call FreeLibrary(h) here, because freeing the handle may
+  // unload the library, and cause future calls to GetTickCount64_fn()
+  // to fail. See 29642 for details.
 }
 
 void



_______________________________________________
tor-commits mailing list
tor-commits@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits