Program received signal SIGSEGV, Segmentation fault.

**boby435** · 05/11/2019, 13h46

Bonjour, je réalise un programme sur des coureurs mais je ne suis pas encore totalement à l'aise en C.
Lors de la compilaiton de mon code, le programme se lance mais je n'obtiens pas le résultat escompté. En utilisant le debbuger je reçois le message suivant :
"Program received signal SIGSEGV, Segmentation fault."
Dans le call stack, les fonctions incriminées sont afficheCoureur() et afficherTabCoureur().
Lors de mes recherches sur cette erreur, j'ai pu voir que cela arrive quand on initialise mal un pointeur. Cependant je n'ai pas vraiment compris comment régler le problème.

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
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
 
#include <stdio.h>
#include <stdlib.h>
#include "structure.h"
 
 
temps saisieTemps(temps *tps);
void afficheTemps( temps *tps);
int temps2secondes (temps tps);
temps seconde2temps (int sec);
temps difference2Temps(temps tps1, temps tps2);
categorie saisirCategorie(categorie *p_cat);
void afficheCategorie(categorie *p_cat);
coureur saisirCoureur(coureur *p_cour);
void tabDeCoureur(coureur *tab, int *p_compt);
void afficherTabCoureur(coureur *tab, int *p_compt);
coureur afficheCoureur(coureur *p_cour);
 
int main()
{
    /*
    temps t1;
    temps t2;
    categorie cat;
    */
    coureur c1;
 
//    saisirCoureur(&c1);
 
    coureur *tab = NULL;
    int compteur = NULL ;
    tabDeCoureur(tab, &compteur);
    //afficheCoureur(&c1);
    afficherTabCoureur(tab, &compteur);
    return 0;
}
 
temps saisieTemps(temps *tps){
    tps->heures=0;
    tps->minutes=-1;
    tps->secondes=-1;
    printf("Veuillez saisir l'heure : ");
    scanf("%d", &tps->heures);
 
    printf("\nVeuillez saisir les minutes : ");
    while (tps->minutes <0 || tps->minutes >59){
        scanf("%d", &tps->minutes);
    }
 
    printf("\nVeuillez saisir les secondes : ");
    while (tps->secondes <0 || tps->secondes >59){
        scanf("%d", &tps->secondes);
    }
 
}
 
void afficheTemps(temps *tps){
    tps->heures <10 ? printf("0%d:", tps->heures) : printf("%d:", tps->heures);
    tps->minutes <10 ? printf("0%d:", tps->minutes) : printf("%d:", tps->minutes);
    tps->secondes <10 ? printf("0%d", tps->secondes) : printf("%d", tps->secondes);
}
 
int temps2secondes (temps tps){
    tps.heures *=3600;
    tps.minutes *=60;
    tps.secondes += tps.heures + tps.minutes;
    return tps.secondes;
}
 
temps seconde2temps (int sec){
    temps t1;
    t1.secondes = sec %60;
    sec/=60;
    t1.minutes = sec %60 ;
    sec/=60;
    t1.heures = sec;
    return t1;
    printf("%d h", sec);
}
 
temps difference2Temps(temps tps1, temps tps2){
    temps differenceTemps ;
    differenceTemps.heures = tps1.heures - tps2.heures ;
    differenceTemps.minutes = tps1.minutes - tps2.minutes ;
    differenceTemps.secondes = tps1.secondes - tps2.secondes ;
    printf("\n%d:%d:%d\n", differenceTemps.heures, differenceTemps.minutes,differenceTemps.secondes);
}
 
temps difference2Temps2(temps tps1, temps tps2){
    return seconde2temps(temps2secondes(tps1)-temps2secondes(tps2));
}
 
categorie saisirCategorie(categorie *p_cat){
 
    int choixCat;
    do {
            printf("\nVoici les differentes categories possibles, veuillez choisir laquelle vous appartient :\n");
            printf("0 : Espoir\n");
            printf("1 : Senior\n");
            printf("2 : Master1\n");
            printf("3 : Master2\n");
            printf("4 : Master3\n");
            printf("5 : Master4\n");
            printf("6 : Master5\n");
            scanf("%d", &choixCat);
    }while (choixCat >master5|| choixCat<espoir);
    *p_cat = choixCat ;
    printf("%d", *p_cat);
 
 
}
 
void afficheCategorie(categorie *p_cat){
     switch(*p_cat){
 
        case 0 :
            printf("ES");
            break;
        case 1 :
            printf("SE");
            break;
        case 2 :
            printf("M1");
            break;
        case 3 :
            printf("M2");
            break;
        case 4 :
            printf("M3");
            break;
        case 5 :
            printf("M4");
            break;
        case 6 :
            printf("M5");
            break;
        }
 
 
     }
 
coureur saisirCoureur(coureur *p_cour){
 
    printf("Veuillez saisir les informations relatives aux coureurs :\n");
    printf("Dossard : ");
    scanf("%d", &p_cour->dossard);
    printf("\nNom : ");
    scanf("%s", &p_cour->nom);
    printf("\nPrenom : ");
    scanf("%s", &p_cour->prenom);
    printf("\nQuelle categorie :");
    saisirCategorie(&p_cour->cat);
    do {
        printf("\nL'heure de depart doit etre inferieur a l'heure d'arrivee");
        printf("\nHeure de depart :");
        saisieTemps(&p_cour->depart);
        printf("\nHeure d'arrivee :");
        saisieTemps(&p_cour->arrivee);
    } while((p_cour)->arrivee.heures < (p_cour)->depart.heures);
 
    p_cour->duree = difference2Temps2(p_cour->arrivee, p_cour->depart);
    afficheTemps(&p_cour->duree);
 
}
 
 
void tabDeCoureur (coureur *tab, int *p_compt){
 
 
    printf("Veuillez saisir le nombre de coureurs qui participent a cette course :\n");
    scanf("%d",p_compt);
    if (*p_compt>0){
        tab = malloc(*p_compt *sizeof(coureur));
        if (tab ==NULL){
            exit(0);
        }
        for (int i=0; i<*p_compt; i++){
            tab[i] = saisirCoureur(tab);
            printf("\n");
        }
   // free(tab);
    }
 
}
 
coureur afficheCoureur(coureur *p_cour){
    printf("\n--------------------------------");
    printf("\nDossard : %d\n%s \n%s\n", p_cour->dossard, p_cour->nom, p_cour->prenom);
    printf("Categorie : "); afficheCategorie(&p_cour->cat);
    printf("\nA mis " ); afficheTemps(&p_cour->duree); printf(" pour faire la course");
    printf("\n--------------------------------");
}
 
 
void afficherTabCoureur(coureur *tab, int *p_compt){
    for (int i=0; i<*p_compt; i++){
        afficheCoureur(tab+i);
    }
}

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
20
21
 
typedef struct temps temps;
struct temps{
    int heures;
    int minutes;
    int secondes ;
 
};
 
enum categorie {espoir, senior, master1, master2, master3, master4, master5};
typedef enum categorie categorie;
typedef struct coureur coureur;
struct coureur {
    int dossard;
    char nom[30];
    char prenom[30];
    categorie cat;
    temps depart;
    temps arrivee ;
    temps duree;
};

**Médinoc** · 05/11/2019, 14h29

Le problème vient de ta fonction tableDeCoureur(): Pourquoi ma fonction ne modifie pas ma variable ?

**boby435** · 05/11/2019, 15h27

Le problème vient-il donc de cette ligne : tab[i] = saisirCoureur(tab) ?
dans la fonction tabDeCoureur()
Si j'ai bien compris le lien que vous m'avez envoyé (ce dont je ne suis pas sûr), il faudrait que ma fonction saisirCoureur(coureur *p_cour ) qui se trouve dans la fonction tabDeCoureur() soit plutôt de forme saisirCoureur(coureur **p_cour) ?

**Médinoc** · 05/11/2019, 15h45

Envoyé par boby435

Si j'ai bien compris le lien que vous m'avez envoyé (ce dont je ne suis pas sûr), il faudrait que ma fonction saisirCoureur(coureur *p_cour ) qui se trouve dans la fonction tabDeCoureur() soit plutôt de forme saisirCoureur(coureur **p_cour) ?

Oui, vu que c'est elle qui alloue le tableau, elle doit pouvoir modifier le pointeur.