You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@apr.apache.org by iv...@apache.org on 2022/06/21 09:54:46 UTC

svn commit: r1902129 - /apr/apr/trunk/threadproc/win32/thread.c

Author: ivan
Date: Tue Jun 21 09:54:45 2022
New Revision: 1902129

URL: http://svn.apache.org/viewvc?rev=1902129&view=rev
Log:
win32: Fix spurious apr_thread_once() failures.

* threadproc/win32/thread.c
  (apr_thread_once): Pass NULL as lpContext argument in call to
   InitOnceExecuteOnce(). According to documentation pointer stored in lpContext
   should be DWORD aligned. We do not use lpContext argument and lpContext is
   optional, so just use NULL.
 

Modified:
    apr/apr/trunk/threadproc/win32/thread.c

Modified: apr/apr/trunk/threadproc/win32/thread.c
URL: http://svn.apache.org/viewvc/apr/apr/trunk/threadproc/win32/thread.c?rev=1902129&r1=1902128&r2=1902129&view=diff
==============================================================================
--- apr/apr/trunk/threadproc/win32/thread.c (original)
+++ apr/apr/trunk/threadproc/win32/thread.c Tue Jun 21 09:54:45 2022
@@ -371,10 +371,8 @@ static BOOL WINAPI init_once_callback(PI
 APR_DECLARE(apr_status_t) apr_thread_once(apr_thread_once_t *control,
                                           void (*func)(void))
 {
-    PVOID lpContext;
-
     if (!InitOnceExecuteOnce(&control->once, init_once_callback, func,
-                             &lpContext)) {
+                             NULL)) {
         return apr_get_os_error();
     }