bug ou probleme?

**totofe** · 26/11/2008, 10h03

Bonjour,

Lors de mon programme suis heurté à un problème que je comprend pas.
Mon programme fonctionne bien sous windows et que je le lance sous linux je rencontre le probleme suivant.

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
 
int lgMax[16]={30,6,100,100,40,20,40,40,100,100,100,100,100,10,40,40};
 
for(i=0;i<zone;i++)
    {
      strcpy(champ,&ligne[position[i]]); 
      lgMax[0]=30; //remet la valeur a 30 sinon bug... je c'est pas pourquoi
      if(strlen(champ)>lgMax[i])
      {
        strcpy(champ," ");
      }
      if(i==2)
        printf("%d ",lgMax[0]);//affiche 30
      while(strlen(champ)<lgMax[i]+1)
        strcat(champ," ");
        //lgMax bug après la boucle quand i=2
      if(i==2)
      {
        printf("%d ",lgMax[0]);//affiche 0
        exit(0);
      }
      fwrite (champ , 1 , lgMax[i] , fDest );
    }

Je comprend pas pourquoi dans mon for quand i=2 ca remet ma valeur lgMax[0] à 0 (les autres valeurs du tableau ne change pas)
Quelqu'un pourrait m'orienter sur le pourquoi de ce problème?

merci

**3DArchi** · 26/11/2008, 10h12

Intuitivement, je dirais que ton premier strcpy déborde et vient écraser le tableau lgMax.
Tu dois avoir quelque par un char champ[TAILLE_MAX]. Tu devrais donc utiliser quelque chose comme ça:

Code :

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

strncpy(champ,&ligne[position[i]],TAILLE_MAX);

**Médinoc** · 26/11/2008, 10h16

Sans la déclaration de champ, ça ne va pas être facile...

Mais ça sent le débordement de buffer, donc surveille un peu mieux ce que tu fais avec champ, et regarde si tu mets une variable entre les deux, si c'est elle qui est modifiée à la place...

**Emmanuel Delahaye** · 26/11/2008, 10h42

Envoyé par totofe

Lors de mon programme suis heurté à un problème que je comprend pas.
Mon programme fonctionne bien sous windows et que je le lance sous linux je rencontre le probleme suivant.

Poste un code compilable. Il est impossible d'analyser ton code. Trop d'inconnues.

**totofe** · 26/11/2008, 11h14

Envoyé par Emmanuel Delahaye

Poste un code compilable. Il est impossible d'analyser ton code. Trop d'inconnues.

désolé, je voulais pas copier le code enterrement, car il est plutôt immense.
j'ajoute les différentes variables, ainsi qu'un bou de code réalisé avant

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
 
int zone;
  int i,t;
  char ligne[1024];
  int position[16];
  int lgMax[16]={30,6,100,100,40,20,40,40,100,100,100,100,100,10,40,40};
  int pos; 
  char champ[101];
 
 if ((fDest = fopen(nomdest, "w")) == NULL)//Créé le fichier fixe test si erreure
  {
    sprintf(txterr,"impossible d'ouvrir le fichier %s \n",nomdest); 
    Ecriture_err();
    return(-2);
  }
 
  nb_ligne=0;
 
  while(fgets(ligne,sizeof(ligne), fic_header)) //compte nombre de ligne sans le header et ajout des data dans le fichier fixe
  {
    nb_ligne++;
    trimCRLF(ligne);
    //Ecriture
 
    t=strlen(ligne);
    zone=1;
    position[0]=0;
    for(i=0;i<t;i++)//remplace les ; par des \0
    {
      if(ligne[i]==';')
      { 
        ligne[i]=0;       
        position[zone++]=i+1; //donne la position de debut de chaque champs
      }                       
    }
    if (zone>16)// + de 15 champs
    {
      log=1;
      sprintf(txterr,"Attention il existe au moins un champ en trop à la ligne %d \n",nb_ligne+1);
      Ecriture_err();
      continue;
    }
for(i=0;i<zone;i++)
    {
      strcpy(champ,&ligne[position[i]]); 
      lgMax[0]=30; //remet la valeur a 30 sinon bug... je c'est pas pourquoi
      if(strlen(champ)>lgMax[i])
      {
        strcpy(champ," ");
      }
      if(i==2)
        printf("%d ",lgMax[0]);
      while(strlen(champ)<lgMax[i]+1)
        strcat(champ," ");
      if(i==2)
      {
        printf("%d ",lgMax[0]);
        exit(0);
      }
        //lgMax bug après la boucle quand i=2
    }

désolé je peux pas poster de code compilable à moins de mettre tout mon programme.

**BainE** · 26/11/2008, 11h45

j aime bien le titre, si c'est un bug c'est pas un problème ?

**Emmanuel Delahaye** · 26/11/2008, 12h05

Envoyé par totofe

désolé, je voulais pas copier le code enterrement, car il est plutôt immense.
j'ajoute les différentes variables, ainsi qu'un bou de code réalisé avant
<...>
désolé je peux pas poster de code compilable à moins de mettre tout mon programme.

Alors il faut apprendre à réduire me code au minimum compilable qui montre le problème, car ce n'est pas à nous de le faire. C'est d'ailleurs une technique de mise au point bien connue et très efficace.

D'une manière plus générale, ton code devrait être organisé en modules indépendants et testables individuellement. Ca fait partie des règles de Bonne Conception.

**totofe** · 26/11/2008, 15h42

J'ai trouvé.
C'était ma variable champ qui débordé étrange car elle ne pouvait pas dépassé 100 (valeur max dans lgMax). et ce problème ne se rencontré que sur linux...
j'ai augmenté la taille du tableau de 1 et ça fonctionne.

**Emmanuel Delahaye** · 26/11/2008, 17h12

Envoyé par totofe

J'ai trouvé.
C'était ma variable champ qui débordé étrange car elle ne pouvait pas dépassé 100 (valeur max dans lgMax). et ce problème ne se rencontré que sur linux...
j'ai augmenté la taille du tableau de 1 et ça fonctionne.

Non. Le problème se rencontre partout. Simplement comme le dépassement de tableau entraine un comportement est indéfini, les conséquences sont imprévisibles. Il peut arriver n'importe quoi, y compris un comportement conforme aux attentes, ce qui fait que ce bug (comportement indéfini) est particulièrement dangereux et fait que le C n'est pas un langage de débutant.

http://emmanuel-delahaye.developpez.com/notes.htm#ub

.

**Thes32** · 26/11/2008, 18h54

Envoyé par Emmanuel Delahaye

Il peut arriver n'importe quoi, y compris un comportement conforme aux attentes, ce qui fait que ce bug (comportement indéfini) est particulièrement dangereux et fait que le C n'est pas un langage de débutant.

Merci pour l'infos.

**3DArchi** · 27/11/2008, 09h33

Envoyé par totofe

J'ai trouvé.
C'était ma variable champ qui débordé étrange car elle ne pouvait pas dépassé 100 (valeur max dans lgMax). et ce problème ne se rencontré que sur linux...
j'ai augmenté la taille du tableau de 1 et ça fonctionne.

Donc, dès que tu vas lire une ligne de 101 caractères

**Médinoc** · 27/11/2008, 11h45

Ce code montre seulement ce qui doit être fait quand la variable champ a déjà débordé.
Il n'empêche en rien le débordement...

**totofe** · 27/11/2008, 11h49

non mais regarder un peut le programme, je vois pas comment elle peut dépasser 100.

je remet l'endroit ou cela produisait le bug avec l'initialisation des variable concerné

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
char champ[102]//avec char champ[101] ca bugué
int lgMax[16]={30,6,100,100,40,20,40,40,100,100,100,100,100,10,40,40};
 
if(strlen(champ)>lgMax[i])
      {
        strcpy(champ," ");
      }
 
while(strlen(champ)<lgMax[i]+1)
        strcat(champ," ");

lgMax à pour valeur 100 au maximum...
donc je vois pas pourquoi la variable champ déborde (et pour rappelle ça se produit uniquement sous linux).

**Médinoc** · 27/11/2008, 11h53

Mais c'est exactement ce que je dis: Tu ne vérifies le débordement qu'après coup...

Il faut empêcher AVANT...

... Et pense au caractère nul de fin de chaîne.

**totofe** · 27/11/2008, 11h55

Envoyé par Médinoc

Ce code montre seulement ce qui doit être fait quand la variable champ a déjà débordé.
Il n'empêche en rien le débordement...

Je vois pas ou il pourrait déborder. avant ceci je stocke dans champ une chaine de 10caractères au max.
et ça déborde quand i=2 soit quand lgMax[2]=100, je pense c'est dû au strcat...
mais je reste toujours dans le flou sur la raison.

**totofe** · 27/11/2008, 11h57

Envoyé par Médinoc

... Et pense au caractère nul de fin de chaîne.

oui pour cette raison que j'ai mis champ[101] et non champ[100]

**Médinoc** · 27/11/2008, 11h57

TU fais des strcat jusqu'à ce que la longueur caractère nul non-compris atteigne 101, ce qui fait 102 caractères en comptant le nul...

**totofe** · 27/11/2008, 12h01

effectivement tu à raison ^^.

Mais pourquoi cela ce produit uniquement sur linux?

**Médinoc** · 27/11/2008, 12h05

Parce que c'est un comportement indéfini, tout peut arriver.

Peut-être que sous Windows, l'alignement entre en jeu, ou que les variables locales ne sont pas dans le même ordre, etc.

**Emmanuel Delahaye** · 27/11/2008, 12h13

Envoyé par totofe

effectivement tu à raison ^^.

Mais pourquoi cela ce produit uniquement sur linux?

C'est un comportement indéfini. Il est illusoire d'essayer d'expliquer quelque chose qui n'est pas défini.

Nota : je t'ai déjà répondu sur ce point. Si tu ne lis pas mes réponses, c'est pas le peine que les poste. Je les efface ?

bug ou probleme?

C

Discussions similaires

Partager

Partager