Discussion :

[chicorico]

Bonjour,

Je ne suis pas familier des threads et j'aimerais bien faire les choses.

Tout d'abord un bout de code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
for (i = 0 ; i < MAX ; i++){
    char * data = NULL;
    data = malloc(1024);
    if (data == NULL){
        printf("Impossible d'allouer la mémoire");
        exit(1);
    }
    memset(data ,0,1024);
    /* On remplie les données */
    .....
    /* On traite les données */
    int res = pthread_create(&threads[i], NULL, sendEvent, (void *) data);
    if (res != 0){
        printf("Impossible de créer le thread : %s.",strerror(errno));
        free(data);
        exit(1);
    }
    free(data);
}

Ma question est de savoir si le free(data) doit être fait à la fin de chaque tour de boucle, ou dans la fonction sendEvent.
Mon intuition me dirait à chaque tour de boucle, mais le thread ne risque-t-il pas de ne plus avoir les données car déréférencées par le prog principal ?

Note : le prog est volontairement simplistepour ne pas compliquer le post et ce concentrer sur le free.

Merci d'avance pour vos lumières

[fregolo52]

Oula !!!
Tu passes un buffer à un thread et dans la foulée, tu fais un free !!
Donc, ton thread reçoit un pointeur (sur une zone de 1k) qui n'est plus valide !!

A mon avis, le free doit être fait dans le thread.

[matafan]

Ou dans le thread qui fait le pthread_create, mais après le pthread_join.

[chicorico]

Oula !!!

:-) , c'est pour ça que je post ici, c'est pour bien faire les choses.

Je pense que mon problème est résolu (le free dans le thread) car je viens de comprendre l'erreur avec le message de fregolo52.

Par contre si le thread n'est pas terminé au prochain tour de boucle, est ce que le buffer du thread va être modifié ?

[matafan]

Non puisque tu passes un nouveau buffer (un buffer différent) à chaque thread.

[chicorico]

Merci à vous deux, c'est plus clair maintenant.