Discussion :

[PhilippeHen]

Bonjour,

j'ai un gros problème de pointeurs, enfin je pense que c'est ça qui me fait mon segmentation fault

j'ai bien lus le tuto sur ce site, puis le tuto "des piles", mais je ne trouve toujours pas l'erreur...
si quelqu'un à un tuto pour les gens qui ne comprennent toujours pas comme moi je suis preneur.
sinon si quelqu'un à une idée:

Code :

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

1
2
3
4
5
6
typedef struct SYNCHRONIZE
{
    pthread_mutex_t *mutex;
    pthread_cond_t *cond;
    int test;
}ST_syncro;

Code :

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

1
2
3
4
5
    ST_syncro *syncro;
    syncro->test=5;
    printf("\nsyncro->test=|%d|\n",syncro->test);
    syncro->mutex = PTHREAD_MUTEX_INITIALIZER;
    syncro->cond =PTHREAD_COND_INITIALIZER;

Merci ...

[Lucien63]

Effectivement ton pointeur syncro n'est pas initialisé.

Un pointeur doit être initialisé avec une adresse avant d'être utilisé.
adresse d'une variable existante ou d'un bloc memoire crée avec malloc ...

[PhilippeHen]

pourrais-tu développer un peu ..s'il te plait

[LittleWhite]

Bonjour,

Comme le dit Lucien63, votre pointeur n'est pas initialisé (pointe sur rien de précis).
Pour ce faire il faut demander de la mémoire au système, pour que celle ci appartienne au programme.
C'est très simple, voyez le code suivant:

Code :

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

1
2
3
4
5
6
 
ST_syncro *syncro;
syncro = (ST_syncro*) malloc(sizeof(ST_syncro));
if ( syncro == NULL )
       // Il y a une erreur ici est faut la géré
       return 1;

Le malloc demande au système de réservé la taille en mémoire de la stucture ST_syncro.
Le pointeur sur cette mémoire fraichement alloué est mis dans syncro.
Maintenant tu peux utilisé le pointeur. ( ne pas oublié de mettre des valeurs dans la structure car pour le moment elle sont non déterminé )

[PhilippeHen]

Exacte, ça marche, vous avez "JUSTE".

j'ignorais qu'il fallait faire un malloc à chaque fois que l'on déclare un pointer.

Dans tous les cas merci beaucoup à vous deux.

Bonne journée.

Philippe

[diogene]

j'ignorais qu'il fallait faire un malloc à chaque fois que l'on déclare un pointer.

Il n'est pas toujours nécessaire d'initialiser un pointeur par un malloc. Ce qu'il faut c'est lui donner une valeur légale : NULL ou l'adresse d'un objet existant.
Dans ce code :

Code :

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

1
2
    ST_syncro *syncro;
    syncro->test=5;

ce qui manque, c'est l'initialisation de syncro par l'adresse d'un objet ST_syncro.

On pourrait écrire, pour illustrer :

Code :

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

1
2
3
    ST_syncro syncobjet;
    ST_syncro *syncro =  & syncobjet;
    syncro->test=5; // ce qui équivaut ici à syncobjet.test = 5