Wie übergeben Sie eine lokale Struktur als Parameter an pthread_create?

Question

Die folgende Funktion funktioniert nicht. pin_thread_function erhalten manchmal Müll anstelle der Strukturdaten. Was ist falsch? Ich weiß, dass das ein grundsätzliches Problem ist, aber ich kann es nicht erklären.

typedef void (*state_callback_t)(int state); 

struct pin_thread_params 
{ 
    char pin[3 + 1]; 
    state_callback_t callback_onchange; 
}; 

extern int pin_monitor_create(const char * pin, 
     state_callback_t callback_onchange) 
{ 
    int ret; 
    unsigned long int thread; 
    struct pin_thread_params params; 

    //Setup struct 
    strcpy(params.pin, "123"); 
    params.callback_onchange = callback_onchange; 

    //Create thread with struc as parameter 
    ret = pthread_create(&thread, NULL, pin_thread_function, &params); 
    if (!ret) 
    { 
     ret = pthread_detach(thread); 
    } 

    return ret; 
} 

static void * pin_thread_function(void * context) 
{ 
    struct pin_thread_params params; 
    memcpy(&params, context, sizeof(params)); 

    //Sometimes the correct string, most time garbage 
    printf("Started monitoring %s", params.pin); 
} 

Wenn params vor pthread_create malloc'ed ist alles in Ordnung, wie das funktioniert:

... 
    struct pin_thread_params * params; 

    //Setup struct with malloc 
    params = malloc(sizeof(struct pin_thread_params)); 
    strcpy(params->pin, "123"); 
    params->callback_onchange = callback_onchange; 
... 

Answer 1

struct pin_thread_params params statisch zugeordnet ist und die Adresse ist nicht sicher zu benutzen, wenn der Umfang der es vorbei ist (das heißt, nachdem pin_monitor_create zurückgekehrt). Es kann vorkommen, dass die Thread-Ausführung manchmal startet, bevor die pin_monitor_create beendet wurde und Sie die gültigen Daten in params sehen. Der dynamisch zugewiesene Speicher stammt jedoch vom Heapspeicher und kann verwendet werden, bis Sie ihn freigeben. Daher erhalten Sie immer gültige Daten unter params innerhalb von pin_thread_function.

Answer 2

Ich bin nicht besonders gut informiert über pThreads (kann nur noch nicht ganz Kommentar), aber Sie geben einen Zeiger an den Stapelspeicher, der dem Thread zugewiesen wurde, der schließlich durch Ausführen von Funktionsaufrufen verfälscht wird.