[erreur] Segmentation fault

Version imprimable

Bonjour,

J'ai un léger souci avec mon programme.
Ce programme tourne sous linux et la cible est de l'embarqué avec un réseau Konnex et un maquette.

Quand je cherche à lire la trame de changement de l'état d'une sortie provoqué par l'appui d'un bouton poussoir le programme me retourne Segmentation fault :

Voici un bout du code :

Code:

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
 
 
 do
    {
     		read(file_def, &lecture, 1);
     		switch(lecture)
     		{
     			case  '\x68' : if (etape == 0)
     								{
     									etape ++;
     									buf[i++] = lecture;
     									printf(" %02x", lecture);
     								}
     						   else if ((etape == 1)&&(nb_length == 2))
     						   		{
     						   			etape ++;
     						   			buf[i++] = lecture;
     						   			printf(" %02x", lecture);
     						   		}
     						   break;
 
     			case '\x16' : if ((etape == 2)&&(nb_corps == 11))//&&(checksum = buf[i-1]))
     						  {
     						  	buf[i] = lecture;
     						  	printf(" %02x", lecture);
     						  	fin_lecture = 1;
     						  }
     						  break;
 
     			default : if (etape == 1)
     							{
     								length = lecture;
     								buf[i++] = lecture;
     								printf(" %02x", lecture);
     								nb_length ++;
     							}
     					  else if (etape == 2)
     					  		{
     					  			buf[i++] = lecture;
     					  			printf(" %02x", lecture);
     					  			nb_corps ++; 
     					  			//if (checksum != lecture)
     					  				checksum +=lecture; 
 
     					  		}
     					  break;     		
     		}
 
      }while (!fin_lecture);

Merci pour vos reponses

Je n'ai rien modifié à part l'indentation, mais ça devrait être un peu plus lisible comme ça.
(je passe sur les espaces entre if et parenthèse, je trouve ça illisible, mais il parait que c'est une question de goût : je ne les supprimerai que si d'autres pensent comme moi)
Code:

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 do { read(file_def, &lecture, 1); switch(lecture) { case '\x68' : if (etape == 0) { etape ++; buf[i++] = lecture; printf(" %02x", lecture); } else if ((etape == 1)&&(nb_length == 2)) { etape ++; buf[i++] = lecture; printf(" %02x", lecture); } break; case '\x16' : if ((etape == 2)&&(nb_corps == 11))//&&(checksum = buf[i-1])) { buf[i] = lecture; printf(" %02x", lecture); fin_lecture = 1; } break; default : if (etape == 1) { length = lecture; buf[i++] = lecture; printf(" %02x", lecture); nb_length ++; } else if (etape == 2) { buf[i++] = lecture; printf(" %02x", lecture); nb_corps ++; //if (checksum != lecture) checksum +=lecture; } break; }//switch } while (!fin_lecture);
PS: On ne connait ni buf, ni lecture, ni aucune autre variable en fait (mais celles-ci sont les plus importantes, puisqu'utilisées comme buffers)...

Code:

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
 
#include <termios.h>
#include <stdio.h>
#include <unistd.h>
#include <fcntl.h>
#include <string.h>
 
 
#define TAILLE_BUF 16
 
 
 
int main(void)
 {
 
    //Local variables    
    int i, balise, nb_length, nb_corps, etape, fin_lecture, compare, file_def;
    char lecture, checksum, length;
    char buf[TAILLE_BUF];
 
    bzero(buf, TAILLE_BUF);
    write (file_def, "\x68\x08\x08\x68\x73\xA9\x00\x18\x34\x56\x78\x0A\x40\x16",14);
     compare = 0;    
    i = 0;
    balise = 0;
    etape = 0;
    length = 0;
    nb_length = 0;
    nb_corps = 0;
    fin_lecture = 0;
    checksum = '\x00';
    sleep(1);
     do
    {
     	read(file_def, &lecture, 1);
     	switch(lecture)
     	{
     	        case  '\x68' : if (etape == 0)
     	                          {
     	                          	etape ++;
	                          	buf[i++] = lecture;
                                  		printf(" %02x", lecture);
                                        }
     		              else if ((etape == 1)&&(nb_length == 2))
     			 {
     				etape ++;
     				buf[i++] = lecture;
     				printf(" %02x", lecture);
     			 }
     			  break;
 
    	         case '\x16' : if ((etape == 2)&&(nb_corps == 11))
		             {
     			  	buf[i] = lecture;
     				printf(" %02x", lecture);
     				fin_lecture = 1;
     		              }
     			  break;
 
     	          default : if (etape == 1)
     		 	{
     			             length = lecture;
     				buf[i++] = lecture;
     				printf(" %02x", lecture);
     				nb_length ++;
     		             }
                                    else if (etape == 2)
     			{
     				buf[i++] = lecture;
     				printf(" %02x", lecture);
     				nb_corps ++; 
     			             chexksum += lecture; 
		  	}
     			  break;   
 
 
     	   }
 
      }while (!fin_lecture);
 
 
            return (0);

Désolé pour la mise en forme mais je suis obligé de faire des copiés collé en passant par le bloc notes.

15/06/2006, 14h27
Emmanuel Delahaye

Citation:

Envoyé par fveysseire

Voici un bout du code :

Magnifique !

http://emmanuel-delahaye.developpez.com/goret.htm
15/06/2006, 14h27
Médinoc

Même avec le bloc-notes, tu pourrais au moins corriger...
Edit: première chose: Tu te rends compte qu'une de tes premières instructions, c'est d'écrire avec les fonctions bas niveau non-portables dans un descripteur de fichier non-initialisé?
15/06/2006, 14h31
fveysseire

C'est à dire Medinoc ?

avec le write?
15/06/2006, 14h33
Médinoc
Code:

1 2 int file_def; write (file_def, "\x68\x08\x08\x68\x73\xA9\x00\x18\x34\x56\x78\x0A\x40\x16",14);
1°) aucune initialisation de file_def entre les deux,
2°) il est inutile et non-portable d'employer write() là où fwrite() suffirait...
15/06/2006, 14h34
fveysseire

Citation:

Envoyé par Emmanuel Delahaye

Magnifique !

http://emmanuel-delahaye.developpez.com/goret.htm

J'ai comme l'impression que tu te moques ?
15/06/2006, 14h35
fveysseire

Cher medinoc le file_def est definit cependant je ne peux sortir tout le code de mon entreprise.

Le file_def est défini grace à un file_def = open(DEVICE, FLAGS);
15/06/2006, 14h37
fveysseire

C vrai que j'aurais pu le dire car avec je que j'ai montrer je serais arriver à la meme conclusion que toi.
15/06/2006, 14h42
joellel

Tu as un candidat sérieux à la segmentation fault, c'est ton buffer buf...
Vérifie si buf[i] ou buf[i++] ne te fait pas sortir de ta zone de 16 caractères...
15/06/2006, 14h44
fveysseire

Citation:

Envoyé par joellel

Tu as un candidat sérieux à la segmentation fault, c'est ton buffer buf...
Vérifie si buf[i] ou buf[i++] ne te fait pas sortir de ta zone de 16 caractères...

Merci Joellel ca vient de la