Discussion :

[kromartien]

bonjour. J'ai reussi a trouver en plaçant des messages de sortie avant et après dans l'exécution d'un programme que une fonction que j'avais écrite était responsable d'une erreur de segmentation. Je vous donne le code de cette fonction car je ne comprends pas pourquoi elle génère une erreur de segmentation lorsqu'elle est appelée. C'est une fonction qui concatène deux chaînes en faisant une allocation dynamique pour avoir une chaîne finale complète dont la taille en mémoire correspond strictement au nombre d'élèment. Voici ce que j'ai fait :

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
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
char * add_suffixe(const char * name_in, const char * suffixe)
{
     char * name_out;
     size_t name_length;
     size_t suffixe_length;
     size_t total_length;
     name_length = strlen( name_in );
     suffixe_length = strlen( suffixe );
     total_length=name_length+suffixe_length+1;
     name_out = malloc(total_length);
     if(name_out==NULL)
     {
          printf("erreur d'allocation memoire sur la chaine name_out\n");
     }
     else
     {
          strncpy(name_out, name_in, name_length+1);
          printf("Le nom du fichier original est %s\n",name_in);
          strcat(name_out, suffixe);
          printf("le nom du fichier de sortie est %s\n",name_out);
     }
     return name_out;
}

Je ne comprends pas pourquoi le programme plante juste à cet endroit, sans me donner la moindre indication de débogage. Je peux postuler que c'est parce que je passe peut être des chaînes trop longues en suffixe mais ça me paraît bizarre puisqque tout est calculé à l'octet près. Merci en tous cas de votre aide.

Je dois dire aussi que le deuxième argument de cette fonction (char * suffixe) est systématiquement un char * que j'obitens avec cette fonction :

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
 
#include <string.h>
#include <stdlib.h>
#include <stdio.h>
 
char * get_optiontextstring(const char * input_string)
{
     char * index;
     int guillemet = '"';
     index = strstr(input_string,"OptionText");
     if(index!=NULL){
     index = strchr(index,guillemet);
     }
     return index;
}

Je ne veux pas poster mon algo en entier tout de suite je crois vraiment que l'erreur est dans add_suffixe mais pas moyen de la détecter.

[jowo]

Bonjour kromartien,

La fonction 'get_optiontextstring' peut retourner la valeur NULL:

1) Si le string "OptionText" n'est pas trouvé dans l'argument 'input_string'
2) Si le carctère '"' n'est pas trouvé.


La fonction 'add_suffixe' n'est pas sécurisée. Tu ne testes pas si les arguments passés ont la valeur NULL.

[todor]

Une allocation dynamique pour ton index pourrait peut-être t'aider...

[jowo]

Une allocation dynamique pour ton index pourrait peut-être t'aider...

+10 pour todor

Je n'ai aucune excuse pour ne pas avoir vu une si grosse erreur

[crocodilex]

Une allocation dynamique pour ton index pourrait peut-être t'aider...

Pourquoi faire ?
Je ne vois pas où est le problème.
Pour info : http://www.linux-kheops.com/doc/man/.../strstr.3.html

[Franck.H]

+10 pour todor

Je n'ai aucune excuse pour ne pas avoir vu une si grosse erreur

Je dirais -10 car c'est complètement inutile, la fonction renvoie un pointeur qui se trouve être la position de départ de la sous-chaîne recherchée donc pas de soucis !

Pour la première fonction j'aurais plutôt utilisé strcpy au lieu de strncpy étant donné que la chaîne complète est de toute façon copiée !

Un test aussi pour vérifier la validité des arguments ne serait pas un luxe !
Aère aussi un peu plus ton code, les lignes vides ca existe !

[Franck.H]

Est-ce que tu peux nous montrer comment tu fait l'appel des deux fonctions dans ton programme ? J'ai ma petite idée mais j'aimerais voir si chez toi ton problème viens de là

[Emmanuel Delahaye]

bonjour. J'ai reussi a trouver en plaçant des messages de sortie avant et après dans l'exécution d'un programme que une fonction que j'avais écrite était responsable d'une erreur de segmentation. Je vous donne le code de cette fonction car je ne comprends pas pourquoi elle génère une erreur de segmentation lorsqu'elle est appelée. C'est une fonction qui concatène deux chaînes en faisant une allocation dynamique pour avoir une chaîne finale complète dont la taille en mémoire correspond strictement au nombre d'élèment. Voici ce que j'ai fait :

Je ne comprends pas pourquoi le programme plante juste à cet endroit,

Code pas clair avec strncpy(). Inutile, utiliser strcpy().

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
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <assert.h>
 
static char *add_suffixe (const char *name_in, const char *suffixe)
{
   char *name_out = NULL;
   size_t const suffixe_length = strlen (suffixe);
   if (suffixe_length > 0)
   {
      size_t const total_length = strlen (name_in) + suffixe_length + 1;
 
      name_out = malloc (total_length);
      if (name_out != NULL)
      {
         strcpy (name_out, name_in);
         printf ("Le nom du fichier original est %s\n", name_in);
         strcat (name_out, suffixe);
         printf ("le nom du fichier de sortie est %s\n", name_out);
      }
      else
      {
         printf ("erreur d'allocation memoire sur la chaine name_out\n");
      }
   }
   return name_out;
}
 
int main (void)
{
   char *s = add_suffixe ("hello", ".world");
 
   if (s != NULL)
   {
      puts (s);
      free (s), s = NULL;
   }
 
   assert (s == NULL);
   return 0;
}

Code :

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

1
2
3
4
5
6
 
Le nom du fichier original est hello
le nom du fichier de sortie est hello.world
hello.world
 
Press ENTER to continue.

[kromartien]

Bonjour kromartien,

La fonction 'get_optiontextstring' peut retourner la valeur NULL:

1) Si le string "OptionText" n'est pas trouvé dans l'argument 'input_string'
2) Si le carctère '"' n'est pas trouvé.


La fonction 'add_suffixe' n'est pas sécurisée. Tu ne testes pas si les arguments passés ont la valeur NULL.

Oui je crois que c'est tout à fait vrai, un des points qui ont fait que ma fonction n'est pas très bonne. J'ai utilisé strdup au final dans get_optionstring avec le free correspondant bien placé. De même c'est du bidouillage sur fichier texte, c'est une peu hasardeux tout ça il suffit d'un champ manquant et je me retrouve le bec dans l'eau c'est vrai.