Plantage du programme sans printf("j %d\n",j);

**User** · 10/05/2008, 21h28

Bonjour à tous !

J'ai un problème avec la fonction str_strip du tuto:

A la fin de la fonction,
quand je mets printf("j %d\n",j);
le programme ne plante pas mais sans le printf ca plante

Code :

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

1
2
3
4
...
printf("j   %d\n",j);  
strip[j]='\0';
...

extrait du code du tuto

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
char *str_strip (const char *string)
{
   char *strip = NULL;
 
   if (string != NULL)
   {   
      strip = malloc (sizeof (*strip) * (strlen (string) + 1));
      if (strip != NULL)
      {
         int i, j;
         int ps = 0;
 
         for (i = 0, j = 0; string[i]; i++)
         {
            if (string[i] == ' ')
            {
               /*if (ps == 0)
               {
                  strip[j] = string[i];
                  ps = 1;
                  j++;
               }*/
            }
            else
            {
               strip[j] = string[i];
               ps = 0;
               j++;
            }
         }
 
         printf("j   %d\n",j);  
         strip[j]='\0';
 
      }
      else
      {
         fprintf (stderr, "Memoire insuffisante\n");
         exit (EXIT_FAILURE);
      }
   }
   return strip;
}

merci à vous !

**crocodilex** · 11/05/2008, 01h05

A première vue, cette fonction a l'air correcte.
Montres ton code avec lequel cette fonction pose problème.

**User** · 11/05/2008, 01h32

Le code est un peu long:

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
int read_proteines(char * src, entete t[1000])
{
    FILE * fsrc;
    char str[15000];
    char * res; 
    char * ch;
    char ch1[1500];
    char * occ;  
    int l;
    int i;  
    char flag[2];
    
    i=0;

    fsrc = fopen(src, "r");
 
    if (fsrc == NULL)
    {
        perror(src); 
        exit(EXIT_FAILURE);
    }
    
    while ((res=super_gets(str, sizeof str, fsrc))!=NULL) 
    {         

      strncpy(flag, res, 2);
                    

           if (strncmp(flag,"ID",2)==0) 
              {
              l=strlen (res)-1;
              ch=str_sub (res,5,l);
              t[i].ID=ch;
              
              } 
           else if (strncmp(flag,"AC",2)==0) 
              {
              l=strlen (res)-1;
              ch=str_sub (res,5,l);          

              res=super_gets(str, sizeof str, fsrc);
              strncpy(flag, res, 2);  

              while (strncmp(flag,"AC",2)==0)
                 {
                 l=strlen (res)-1;
                 occ=str_sub (res,5,l);
                 strcat (ch, " ");  
                 strcat (ch, occ);

                 res=super_gets(str, sizeof str, fsrc); 
                 strncpy(flag, res, 2); 
                 
                 }

              t[i].AC=ch;
              
              }
 
           else if (strncmp(flag,"DE",2)==0) 
              {
              l=strlen (res)-1;
              ch=str_sub (res,5,l);

              res=super_gets(str, sizeof str, fsrc);
              strncpy(flag, res, 2);  

              while (strncmp(flag,"DE",2)==0)
                 {
                 l=strlen (res)-1;
                 occ=str_sub (res,5,l);
                 strcat (ch, " ");  
                 strcat (ch, occ);

                 res=super_gets(str, sizeof str, fsrc); 
                 strncpy(flag, res, 2); 
                 
                 } 

              t[i].DE=ch;
               
              }

           else if (strncmp(flag,"  ",2)==0)
              {           
              
              strcpy(ch1,"");
              ch1[0]='\0';

              l=strlen (res)-1;
              occ=str_sub (res,5,l); 
              strcat (ch1, occ);

              res=super_gets(str, sizeof str, fsrc);

              strncpy(flag, res, 2);  

              while (strncmp(flag,"  ",2)==0)
                 {
                 l=strlen (res)-1;
                 occ=str_sub (res,5,l); 
                 strcat (ch1, occ);

                 res=super_gets(str, sizeof str, fsrc); 
                 strncpy(flag, res, 2);                 
                 }
                        
              t[i].SQ=str_strip(ch1);
 
              i=i+1;
              }
          
                   
    }

    fclose(fsrc);

return i; 

}

l'instruction est en bleu

(je ne suis pas un expert

)

@+

**souviron34** · 11/05/2008, 05h29

déjà flag doit être dimensionné à 3 : il faut la place pour le '\0' final....

dans tes strncpy ça déborde...

**Emmanuel Delahaye** · 11/05/2008, 14h27

Envoyé par User

Bonjour à tous !

J'ai un problème avec la fonction str_strip du tuto:

A la fin de la fonction,
quand je mets printf("j %d\n",j);
le programme ne plante pas mais sans le printf ca plante

Ceci

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
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
char *str_strip (const char *string)
{
   char *strip = NULL;
 
   if (string != NULL)
   {
      strip = malloc (sizeof (*strip) * (strlen (string) + 1));
      if (strip != NULL)
      {
         int i, j;
         int ps = 0;
 
         for (i = 0, j = 0; string[i]; i++)
         {
            if (string[i] != ' ')
            {
               strip[j] = string[i];
               ps = 0;
               j++;
            }
         }
         strip[j] = '\0';
 
      }
      else
      {
         fprintf (stderr, "Memoire insuffisante\n");
         exit (EXIT_FAILURE);
      }
   }
   return strip;
}
 
#ifdef TEST
 
#include "ed/inc/prt.h"
#include <assert.h>
 
int main (void)
{
   char const *sa = "Hello world";
 
   char *sb = str_strip (sa);
   if (sb != NULL)
   {
      PRT_S (sb);
 
      free (sb), sb = NULL;
   }
   assert (sb == NULL);
 
   return 0;
}
#endif

fonctionne correctement.
(je me serais contenté d'un return NULL en cas d'erreur d'allocation...)

Code :

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

1
2
3
4
 
sb           = 'Helloworld'
 
Press ENTER to continue.

Le problème est ailleurs.

**Emmanuel Delahaye** · 11/05/2008, 14h44

Envoyé par User

Le code est un peu long:

Peux-tu donner un exemple de fichier de données à traiter. Que se passe-t-il si il y a plus de 1000 résultats ?

Après investigation rapide, ton code est extrêmement faux, car il utilise des chaines incomplètes, de taille insuffisantes... (comportement indéterminé). En C de base, rien n'est automatique. Il faut gérer les chaines à la main.

Néanmoins, et pour simplifier le travail, j'ai mis au point un système qui permet de gérer les chaines automatiquement.

http://emmanuel-delahaye.developpez.com/clib.htm
Module FSTR (Flexible STRings)

C'est un projet scolaire ?

**Emmanuel Delahaye** · 11/05/2008, 14h47

Envoyé par souviron34

déjà flag doit être dimensionné à 3 : il faut la place pour le '\0' final....

dans tes strncpy ça déborde...

Vu qu'il utlise strncpy() et strncmp(), c'est pas obligatoire... Mais il faut être très strict sur l'usage des fonctions et ne jamais utiliser de fonctions 'chaines', de celles qui attendent un 0 en fin de chaine...

Or, il y a un strcat() sur un ch ! C'est mort !

Donc, OK, il faut bien construire des chaines valides.

(en fait, c'est pire que ça, car ch fait 2 et il fait un strcat() sur de la mémoire qui n'existe pas...)

**User** · 11/05/2008, 17h17

Desole pour mon retard,

Emmanuel Delahay,
comme je l'ai mentionne plus haut,
mon code fonctionne parfaitement bien avec le printf au niveau de str_strip mais sans le printf ca plante

pour le flag je vais changer,

en fait tu critiques mais tu ne proposes rien de concret a la place du strcat...

tu ne sais pas te mettre au niveau des gens, tout le monde n est pas aussi expert que toi

**souviron34** · 11/05/2008, 20h00

Envoyé par User

comme je l'ai mentionne plus haut,
mon code fonctionne parfaitement bien avec le printf au niveau de str_strip mais sans le printf ca plante

Ce qu'on te dit, c'est que ce comportement est NORMAL lors d'un dépassement de mémoire...

C'en est même le signe le plus évident..

Donc vérifie tes allocations et les longueurs allouées..

En particulier, je viens de voir ça :

Code :

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

1
2
 
              t[i].ID=ch;

Si c'est une chaîne, c'est raté. Ici tu copies le pointeur...

**User** · 11/05/2008, 20h27

merci a toi,

je vais regarder allocation et les longueurs allouees de plus pres,

pour ta question:
char * t[i].ID
et
char * ch

@+

**Emmanuel Delahaye** · 11/05/2008, 21h44

Envoyé par User

comme je l'ai mentionne plus haut,
mon code fonctionne parfaitement bien avec le printf au niveau de str_strip mais sans le printf ca plante

Et alors ? La fonction en question n'est pas en cause. A partir du moment où il y a débordement de la mémoire (ton code en est bourré), le comportement est indéfini. Tout peut arriver et ce n'importe quand. Tu comprends ça ?

Alors c'est pas la peine d'essayer de dire que c'est de la faute du code que tu as trouvé sur ce site. Pas possible de voir ça.

en fait tu critiques

Je ne critique pas, j'explique pourquoi ton code ne fonctionne pas. Nuance. Si tu ne le supportes pas, il ne faut pas venir chercher de l'aide sur les forums...

mais tu ne proposes rien de concret a la place du strcat...

Qu'est-ce que tu racontes ?

1 - Je n'ai pas dit qu'il ne fallait pas utiliser strcat()
2 - J'ai bien expliqué qu'il y avait de dépassements de mémoire
3 - J'ai proposé une solution de gestion automatique des chaines
4 - j'ai posé une question à laquelle tu n'as pas répondu.

**User** · 11/05/2008, 21h53

JE N'AI JAMAIS DIS QUE LE CODE TROUVE SUR CE SITE ETAIT ERRONE AU CONTRAIRE. JE N'AI PAS ARRETER DE REMERCIER LE GARS QUI L'A ECRIT;
Il M'A DAILLEURS REMERCIER POUR UNE CORRECTION...)

POURQUOI TU CHERCHES SANS ARRET LES CONFLITS.

JE NE SUIS PAS ICI POUR M'ENGUEULER AVEC QUELQU'UN.

JE N'AI JAMAIS EU AUCUN PROBLEME SUR CE SITE;

QUE CE SOIT SUR LES FORUM ACCESS, DELPHI, PERL...
J'AI TOUJOURS AIDE ET JE ME SUIS TOUJOURS FAIT AIDER PAR DES GENS SUPER COOL ET JE N'AI JAMAIS EU LE MOINDRE PROBLEME AVEC PERSONNE !!!

ALORS ARRETE DE CHERCHER LES CONFLITS !!!

J'AI COMPRIS OU JE DOIS CHECHER MAINTENANT LAISSE MOI TRANQUILLE !

**Emmanuel Delahaye** · 11/05/2008, 22h27

Envoyé par User

JE N'AI JAMAIS DIS QUE LE CODE TROUVE SUR CE SITE ETAIT ERRONE AU CONTRAIRE. JE N'AI PAS ARRETER DE REMERCIER LE GARS QUI L'A ECRIT;
Il M'A DAILLEURS REMERCIER POUR UNE CORRECTION...)

POURQUOI TU CHERCHES SANS ARRET LES CONFLITS.
<...>

Euh, t'as fumé un truc ? Tu devrais nettoyer ton clavier, la touche MAJ est bloquée...

Plantage du programme sans printf("j %d\n",j);

C

Discussions similaires

Partager

Partager