Erreur de segmentation avec des chaines de caractères

**Safadoo** · 31/05/2013, 12h28

Bonjours,

Alors voila moi problème mon programme est un serveur TCP qui doit recevoir des trames Gcode d'un client externe, les commandes sont par exemple pour mon test "G21 G90 G00 X100 Y150 Z120" j'utilise un strtok avec " " pour isoler chaque Gxx ou XYZxx dans une boucle pour les analyser un par un.

Mon problème comme son nom l'indique j'ai une erreur de segmentation, je fais un premier tour dans la boucle et j'isole G21 mais au deuxieme tour G90 j'ai mon erreur et je ne comprend pas, voila mon 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
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
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
#include <stdio.h>
#include <sys/types.h>
#include <sys/socket.h>
#include <netinet/in.h>
#include <stdlib.h>
#include <unistd.h>
#include <arpa/inet.h>
 
#define INVALID_SOCKET -1
#define SOCKET_ERROR -1
#define PORT 5000
 
 
#define TRUE 1              /* Creation d'un type bool */
#define FALSE 0
 
 
typedef int SOCKET;
typedef struct sockaddr_in SOCKADDR_IN;
typedef struct sockaddr SOCKADDR;
 
 
 
int main(){
    int first_time = TRUE;
    char connect[20] = "Connexion etablie";
    char *pointeur,*pointeur_tmp,*sepa = ' ';
    char *cp_buffer_reception,*buffer_envoie;
    int get_out = 0;
    /* Socket et adressage du serveur */
    int i = 0, X_int = 0, Y_int = 0, Z_int = 0;
    char *X_char = NULL, *Y_char = NULL, *Z_char = NULL;
    int absolu,pouce;
    int taille;
 
    SOCKET sock;
    SOCKADDR_IN sin;
    socklen_t size = sizeof(sin);   /* Taille de l'adresse */
    char buffer_reception[40] ="";
 
    /* Socket et adressage du client */
    SOCKET csock;
    SOCKADDR_IN csin;
    socklen_t csize = sizeof(sin);  /* Taille de l'adresse */
 
 
    if( (sock = socket( AF_INET, SOCK_STREAM, 0) ) != INVALID_SOCKET ){
        printf("Ouverture socket en mode TCP/IP \n");
 
/********************************* Configuration de la connexion ***********************************************/
 
        sin.sin_addr.s_addr = htonl(INADDR_ANY);  /* Adresse IP auto */
        sin.sin_family = AF_INET;  
        sin.sin_port = htons(PORT); /* Port definie 5000 */
 
        printf("Ouverture du port %d \n\n",PORT);
 
/*********************************************** Etablissement de la connexion *********************************/
        if( (bind(sock, (SOCKADDR*)&sin, sizeof(sin) ) ) != SOCKET_ERROR){
 
/******************** ***********On met la socket en état d'écoute *********************************************/
            if( (listen(sock,1) ) != SOCKET_ERROR ){           
/******************************* On attend que le client se connecte *******************************************/
                csock = accept(sock, (SOCKADDR*)&csin, &csize );
                send(csock, connect, sizeof(connect), 0);   /* Envoie chaine "caractère envoyé" pour tester la connexion du client au serveur */
 
                if( (recv( csock, buffer_reception, sizeof(buffer_reception), 0 ) ) != SOCKET_ERROR ){
                    printf("Connexion client socket %d de %s : %d \n",csock, inet_ntoa(csin.sin_addr), htons(csin.sin_port) );
                    printf("Buffer recue :");
 
                    for( i = 0; i < sizeof(buffer_reception) ; i++){
                        printf( " %c",buffer_reception[i]);
                    }
                    printf("\n");
 
                    while(get_out == 0){
 
/**************************************** Puis on recoit les données du client *********************************/
 
 
                        /* On copie le contenu de buffer_reception */
                        cp_buffer_reception = strdup(buffer_reception);
 
                        if(cp_buffer_reception == NULL ){
                            printf("erreur copie buffer\n");
                            exit(-1);
                        }
                /* On separe et analyse les données recus pour pouvoir les analysés séparement */
                /*On utilise la méthode strtok pour isoler les différentes méthode et coordonnée*/
 
 
                        /* Premier appel de strtok */
                        if(first_time == TRUE ){
                            pointeur = strtok( cp_buffer_reception, " " );
                            first_time = 0;
                        }
                        /* puis on cherche les autres séparateur */
                        else if(first_time == FALSE){
                            pointeur = strtok( NULL , " " );
                        }
 
/************************************************** Test du buffer recu *******************************************************/
                        if( strcmp( pointeur,"G20" ) == 0 ){
                            printf("Programmation en pouce\n");
                            /* On utilise l'unité de mesure pouce, sauf si on rencontre G21 */
                            /* On utilise un bool qui defini si les coordonnées sont en pouce ou mm, ici pouce vaut TRUE */
                            pouce = 1;  /*1 = TRUE */
 
                        }
                        if( strcmp ( pointeur,"G21" ) == 0 ){
                            printf("Programmation en mm\n\n");
                            /* On utilise l'unité de mesure mm, sauf si on rencontre G20 */
                            /* On utilise un bool qui defini si les coordonnées sont en pouce ou mm, ici pouce vaut FALSE */
                            pouce = 0;  /*0 = FALSE */
 
                        }
                        if( strcmp ( pointeur,"G90" ) == 0 ){
                            printf("Déplacement en mode absolue donc @0M \n");
                            /* On utilise que la méthode @0M, sauf si on rencontre G91 */
                            /* On utilise un bool qui definit si la méthode est absolue ou relative, ici absolue vaut TRUE */
                            absolu = 1;
 
                        }
                        if( strcmp ( pointeur,"G91" ) == 0 ){
                            printf("Déplacement en mode relatif donc @0A \n");
                            /* On utilise que la méthode @0M, sauf si on rencontre G90 */
                            /* On utilise un bool qui definit si la méthode est absolue ou relative, ici absolue vaut FALSE */
                            absolu = 0;
                        }
                        if( strcmp ( pointeur,"G28" ) == 0 ){
                            printf(" POM \n");
                            POM();
 
                        }printf("testtetetetetete");
                        if( strcmp ( pointeur,"G00" ) == 0 ){
        -----> Problème dans ces alentours       /* TODO Lecture des coordonnées */
                            while(pointeur != NULL){
                                printf("test while");
                                pointeur = strtok( NULL , " " );   
                                taille = strlen(pointeur);                  /*On calcule la taille des données contenu dans pointeur */
                                pointeur_tmp = strndup(pointeur+1,taille-1);        /*On copie le contenu a partir des coordonnées */
                                /* On lit les coordonnées de déplacement */
                                if(strcmp( pointeur[0],'X') == 0 ){
                                    X_int = atoi(pointeur_tmp);
                                }
                                else if(strcmp( pointeur[0],'Y') == 0 ){
                                    Y_int = atoi(pointeur_tmp);
                                }
                                else if(strcmp( pointeur[0],'Z') == 0 ){
                                    Z_int = atoi(pointeur_tmp);
                                }
                            }
                            if(pouce == TRUE ){
                                X_int = X_int * 100;
                                Y_int = Y_int * 100;        /*On convertie les coordonnées en pas du charlyrobot */
                                Z_int = Z_int * 100;
                            }
                            else{
                                X_int = X_int * 25.4 * 100;
                                Y_int = Y_int * 25.4 * 100;
                                Z_int = Z_int * 25.4 * 100;
                            }
                            sprintf(X_char,"%d",X_int);
                            sprintf(Y_char,"%d",Y_int);
                            sprintf(Z_char,"%d",Z_int);
 
/* ******************************** Création de "chaine entrante" pour la fonction mouvement *******************************/
                            /* On entre la méthode */
                            if(absolu == TRUE ){
                                /* On ajoute "@0M" au buffer_envoie */
                                strcat(buffer_envoie,"@0M ");
                            }
                            else{
                                /* On ajoute "@0A" au buffer_envoie */
                                strcat(buffer_envoie,"@0A ");
                            }
 
                            /* Puis les coordonnées et la vitesse */
                            /* X */
                            if( X_char != NULL ){
                                strcat(buffer_envoie," ");
                                strcat(buffer_envoie,X_char);
                            }
                            else{
                                strcat(buffer_envoie," 0");
                            }
                            /* Ajout de la vitesse */  
                            strcat(buffer_envoie," 1000");
 
                            /* Y */
                            if( Y_char != NULL){
                                strcat(buffer_envoie," ");
                                strcat(buffer_envoie,Z_char);
                            }
                            else{
                                strcat(buffer_envoie," 0");
                            }
                            /* Ajout de la vitesse */
                            strcat(buffer_envoie," 1000");
 
                            /* Z */
                            if( Z_char != NULL){
                                strcat(buffer_envoie," ");
                                strcat(buffer_envoie,Z_char);
                            }
                            else{
                                strcat(buffer_envoie," ");
                                strcat(buffer_envoie,"0");
                            }
                            /* Ajout de la vitesse */
                            strcat(buffer_envoie," 1000");
 
                            printf("Chaine envoyé :%s \n",buffer_envoie);
                            Mouvement(buffer_envoie);
 
                        }
                        if( strcmp ( pointeur,"G01" ) == 0 ){
                            /* TODO Lecture de coordonnées */
 
 
                        }
                        if( strcmp ( pointeur,"M2" ) == 0 ){
                            printf("End of program !\n");
                            get_out = TRUE;
 
                        }
 
 
                    } /* Fin du while */
                }
                else{
                    printf("erreur revc !\n");
                }  
            }
            else{
                printf("Erreur listen !\n");
            }
        }
        else{
            printf("Erreur bind !\n");
        }
 
        /* On ferme les sockets client et serveur */
        printf("Fermeture des sockets client et serveur \n");
        close(csock);
        close(sock);
 
    }
    else{
        printf("Erreur socket !\n");   
    }
    return EXIT_SUCCESS;
}

et ce qui s'affiche

**Neckara** · 31/05/2013, 12h34

Bonjour,

En ce qui concerne tes warning, tu as oublié d'inclure string.h.

Pour ton erreur de segmentation peux-tu utiliser un débogueur ainsi tu sauras exactement à quelle ligne l'erreur de segmentation survient.

Sinon : RTFM.

**Bktero** · 31/05/2013, 12h35

Exécute ton programme à travers valgrind. Il te dira où ça plante et peut-être même pourquoi

**kwariz** · 31/05/2013, 12h37

Bonjour,

alors une remarque rapide car j'ai lu ton code en diagonal mais vitesse mach 2. Il n'est jamais interdit d'écrire des fonctions pour éviter un main de cette taille qui est assez rébarbatif à lire.
Les warnings sont toujours importants ... pense à inclure les headers pour avoir les bonnes déclarations des fonctions que tu utilises.
Par curiosité j'ai fait une recherche sur buffer_envoie, tu le déclares comme un char * mais tu ne l'alloues jamais, tu l'utilises avec strcat qui s'attend non seulement qu'il soit alloué mais également qu'il contienne au moins la chaine vide ...
Utiliser strtok est déconseillé, mais ce n'est pas le plus grave.

EDIT: grillé par Neckara et Bktero ! Mazette !
Évidemment leurs conseils sont hyper judicieux ... debug+valgrind ...

**Neckara** · 31/05/2013, 12h40

A mon avis l'erreur vient de la ligne 143 :
strcmp( pointeur[0],'X').

D'où mon petit RTFM.

**Safadoo** · 31/05/2013, 13h14

Ba je voudrait juste tester le premier caractères du pointeur, comme je pourrai autrement ?

Ps: désolés pour ce magnifique code jacky

**Neckara** · 31/05/2013, 13h19

Envoyé par Safadoo

Ba je voudrait juste tester le premier caractères du pointeur

Non, tu voulais tester le premier caractère de ta chaîne de caractère.

Sinon, pour tester un caractère :

Code :

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

1
2
3
char c;
if( c == '\n')
       ; // fait ce que tu veux

**Safadoo** · 31/05/2013, 13h23

ba moi ici mon pointeur contient soit Xxx Yxx ou Zxx ou xx sont des nombres, et donc je veux juste tester s'il y a la lettre X Y ou Z en temps que premier caractères de pointeur, mais je ne voie pas trop comment faire car j'ai assez de mal avec les pointeur

**Neckara** · 31/05/2013, 13h27

Si tu as du mal avec les pointeurs, cet article pourrait t'aider.

Sinon :

Code :

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

1
2
char * p; // p est de type char *
p[0]; // p[0] est de type char

**Safadoo** · 31/05/2013, 13h59

J'ai regarder mais je ne comprend toujours pas comment isoler juste le premier élément pointer par pointeur !!!

**Neckara** · 31/05/2013, 14h09

Envoyé par Safadoo

J'ai regarder mais je ne comprend toujours pas comment isoler juste le premier élément pointer par pointeur !!!

Code :

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

ptr[0] <-> *(ptr + 0)

Quand un pointeur est additionné avec un entier E, on obtient un pointeur sur le (E) ième élément suivant l'élément pointé par le pointeur.

*ptr = élément pointé par le pointeur.

ptr[0] = *(ptr + 0) = *ptr = élément pointé par le pointeur :

Code :

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

1
2
3
char c = 'e';
ptr = &c;
ptr[0]; // vaut 'e'

ptr[1] = *(ptr + 1) = élément qui est après celui pointé par le pointeur.

Code :

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

1
2
3
char c = 'e';
ptr = &c;
ptr[1]; // erreur, mémoire non-allouée

Une chaîne de caractère est une suite de caractère terminée par un '\0'.

Ainsi :

Code :

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

char * p = "123456";

p est un pointeur sur le premier élément de ta chaîne de caractère c'est à dire '1'.
p + 1 sera un pointeur sur le deuxième élément de ta chaîne donc p[1] vaudra '2', etc...

**Safadoo** · 31/05/2013, 14h30

Donc si je fais un test sur *(pointeur+0) je fais un test sur le premier caractères de pointeur mais si je fais un test sur pointeur je test tout le contenu de pointer par pointeur!

**Bktero** · 31/05/2013, 14h46

Au risque de paraître brutal, je pense qu'il serait temps d'ouvrir son cours de C à la section Pointeurs.

Tu mélanges pointeur, adresse, tableau, chaîne de caractères. Tu ne sembles pas maîtriser l'arithmétique des pointeurs et les opérateurs tels que * et &. L'allocation de la mémoire est une notion qui te semble très vague. Tu ne sais pas comment comparer deux "tableaux" (pas au sens stricto sensu d'une variable type tableau mais au sens "suite d'octets").

Tant que ce genre de notions ne te seront pas un minimum claires, tu auras des problèmes de compréhension et tu auras du mal à coder.

**Neckara** · 31/05/2013, 14h52

Envoyé par Safadoo

Donc si je fais un test sur *(pointeur+0) je fais un test sur le premier caractères de pointeur

Oui, ce qui est équivalant à faire un test sur *pointeur.

Envoyé par Safadoo

mais si je fais un test sur pointeur je test tout le contenu de pointer par pointeur!

Non, en fait, je crois que tu es perturbé par les chaînes de caractères.
Quand on fait :

Code :

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

char * ptr = "errtt";

En mémoire on aura :
e r r t t '\0' ?? ?? ?? [...]

On fait pointer ptr sur le premier caractère. ptr ne pointe que sur un caractère.

Les fonctions qui utilisent des chaînes de caractères vont se dire :

Une chaîne de caractère est terminée par un '\0', donc je peux lire le caractère suivant en mémoire (en faisant *(ptr + i) ) tant que je ne rencontre pas '\0'.

Par contre si tu fais :

Code :

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