Il seguente frammento di codice mostra "con ambito locale oggetto statico inizializzazione" non è thread-safe:
#include <windows.h>
#include <stdio.h>
#include <process.h>
struct X {
~X() { puts("~X()"); }
int i_ ;
void print(void) {
printf("thread id=%u, i = %d\n", GetCurrentThreadId(), i_);
}
X(int i) {
puts("begin to sleep 10 seconds");
Sleep(1000 * 10);
i_ = i;
printf("X(int) i = %d\n", i_);
puts("end");
}
};
X & getX()
{
static X static_x(1000);
return static_x;
}
void thread_proc(void *)
{
X & x = getX();
x.print();
}
int main(int argc, char *argv[])
{
HANDLE all_threads[2] = {};
all_threads[0] = HANDLE(_beginthread(thread_proc, 0, 0));
printf("First thread Id: %u\n", GetThreadId(all_threads[0]));
Sleep(1000);
all_threads[1] = HANDLE(_beginthread(thread_proc, 0, 0));
printf("Second thread Id: %u\n", GetThreadId(all_threads[1]));
WaitForMultipleObjects(_countof(all_threads), all_threads, TRUE, 1000 * 20);
puts("main exit");
return 0;
}
L'uscita sarà (ovviamente filo id sarà diverso sulla vostra macchina):
First thread Id: 20104
begin to sleep 10 seconds
Second thread Id: 20248
thread id=20248, i = 0
X(int) i = 4247392
end
thread id=20104, i = 1000
main exit
~X()
Prima le prime dichiarazioni filo che significa ctor del Singleton viene chiamato e restituito, il secondo thread ottenere l'oggetto non inizializzato e chiamare è il metodo membro (perché l'oggetto statico è in segmento BSS, sarà initilized a zero dopo loader caricare il file eseguibile) e ottenere il valore errato: 0.
Accensione di montaggio messa in vendita da/fasc /Fastatic.asm otterrà il codice assembly per la funzione GetX():
01: [email protected]@[email protected]@XZ PROC ; getX
02:
03: ; 20 : {
04:
05: 00000 55 push ebp
06: 00001 8b ec mov ebp, esp
07:
08: ; 21 : static X static_x(1000);
09:
10: 00003 a1 00 00 00 00 mov eax, DWORD PTR [email protected][email protected]@[email protected]@[email protected]
11: 00008 83 e0 01 and eax, 1
12: 0000b 75 2b jne SHORT [email protected]
13: 0000d 8b 0d 00 00 00
14: 00 mov ecx, DWORD PTR [email protected][email protected]@[email protected]@[email protected]
15: 00013 83 c9 01 or ecx, 1
16: 00016 89 0d 00 00 00
17: 00 mov DWORD PTR [email protected][email protected]@[email protected]@[email protected], ecx
18: 0001c 68 e8 03 00 00 push 1000 ; 000003e8H
19: 00021 b9 00 00 00 00 mov ecx, OFFSET [email protected][email protected]@[email protected]@[email protected]@A
20: 00026 e8 00 00 00 00 call [email protected]@[email protected]@Z ; X::X
21: 0002b 68 00 00 00 00 push OFFSET [email protected][email protected]@[email protected]@[email protected] ; `getX'::`2'::`dynamic atexit destructor for 'static_x''
22: 00030 e8 00 00 00 00 call _atexit
23: 00035 83 c4 04 add esp, 4
24: [email protected]:
25:
26: ; 22 : return static_x;
27:
28: 00038 b8 00 00 00 00 mov eax, OFFSET [email protected][email protected]@[email protected]@[email protected]@A
29:
30: ; 23 : }
Alla riga 10 il simbolo criptico [? $ S1 @? 1 ?? getX @@ YAAAUX @@ XZ @ 4IA] è l'indicatore globale (anche in BSS) che segnala se il singleton è cifrato o no, sarà flaged come vero dalla riga 14-17, appena prima di chiamare nel ctor, questo è il problema, anche questo spiega perché il secondo thread ha immediatamente ottenuto l'oggetto singleton non inizializzato e chiama felicemente la sua funzione membro. Non c'è alcun codice relativo alla sicurezza del thread inserito dal compilatore.
Vedere http: // stackoverflow.it/questions/164496/how-can-i-create-a-thread-safe-singleton-pattern-in-windows – MerickOWA
@MerickOWA: risale al 2008 e non fornisce una risposta chiara e accettata globalmente. Inoltre, non copre nemmeno VC2010 (a causa dell'età dell'argomento). –
@ IC3M4N VS2010 è stato implementato prima che venisse pubblicato C++ 11, Se non implementa la costruzione thread safe di variabili locali statiche, allora si rimane con tecniche che esistono da molti anni. Non vedo nulla che non si applica a VS2010 – MerickOWA