[C]probleme d'adresse memoire Résolu/Fermé

Question

Bonjour,
actuellement je code un minishell, le probleme c'est que des fois ça segfault des fois non, ça a l'air d'etre au ptit bonheur la chance.
Je lis sur l'entrée standard, une fois la lecture effectuée je "colle" ce que j'ai récupéré a la suite de la chaine que je renverrai quand il n'y aura plus rien a lire sur l'entrée standard, pour ça j'utilise my_strndup, c'est comme strncat sauf que là ya un malloc integré.
Le probleme étant que je rentre des trucs au pif =>ça marche
Je rentre des trucs au pif encore une fois=> ça marche pas
et pas necessairement dans cet ordre
Donc gdb et là c'est le drame:
Program received signal SIGSEGV, Segmentation fault.
0x08048a29 in my_strlen (str=0x64 <Address 0x64 out of bounds>) at my_strlen.c:8
8         while (str[i] != '\0')
alors apparement l'adresse memoire de la chaine dont je cherche a recuperer la longueur cad la chaine finale a été modifiée par je ne sais quel miracle, a la place une valeur arbitraire, donc evidemment => core dump.
J'ai lu que c'était un stack overflow, alors si on pouvait me dire comment l'eviter ça serait vraiment sympa parce que là je pedale dans lvide depuis trop longtemps, voila si vous voulez un bout de sources ya qu'a demander.
merci d'avance ^^

fiddy · Accepted Answer

Ah, je comprends mieux ce que tu veux. :)


char *my_strndup(char *str1, char *str2, int n)
{
  int   i;
  int   i2;

  i =strlen(str1);
  char *result = (char*)malloc((i+n+1)*sizeof(char));

  i=0;
  while(i<strlen(str1))
    result[i]=str1[i++];


  i=0;
  while(i<n)
    result[strlen(str1)+i]=str2[i++];

  result[strlen(str1)+n]='\0';

  return (result);
}


J'espère que c'est ce que tu attendais.

fiddy · Answer

Salut,

Pourrais-tu mettre ton code ici, et un exemple de ce que tu as mis qui a fait bugguer le programme.
Cordialement

fiddy · Answer

Dans my_strdump, pourquoi tu calcules my_strlen(strl1) pour refaire un malloc dessus ? Si tu utilises strlen sur une chaine, c'est qu'elle est déjà allouée, sinon il ne faut pas l'utiliser. Peut-être que tu voulais écrire i=my_strlen(strl2); Non ?

Mahmah · Answer

Bonjour,

Moi je le prends le pari que my_strndup s'appelle en fait my_strncat...
La fonction strndup ne prend que deux paramètres elle, par contre strncat ne ferait pas d'allocation...

Quel est le but exacte de ta fonction ? Là on dirait un mix des deux.

Fiddy++; Il est louche le malloc...

M.

PS:
Faudra appeler le plombier, y a des fuites ^^"

Mahmah · Answer

Fiddy rocks ;-)

3 petits warnings tout de même (oui, je sais je suis maniaque ^^)

warning : '=' : conversion from 'size_t' to 'int', possible loss of data
pour le i = strlen

warning : '<' : signed/unsigned mismatch
pour le while(i<strlen(str1))

warning : 'i2' : unreferenced local variable
Ah...

M.

Mais ça fuit toujours. Ah oui, je suis buté aussi, confirmes-tu Fiddy ? (Juste pour la fuite hein ;-))

Mahmah · Answer

On peut avoir le code de my_strlen s'il te plait ?

C'est quand même elle qui se fait écraser son pointeur...

M.

Yep, 2 posts d'un coup   ^

Tisiph0n3 · Answer

ceci dit ça vient peut etre d'une autre fonction je pensais, parce qu'en fait j'avais remarqué qu'après plusieurs appels a get_next_line je retrouvait les commandes que j'avais passées precdemment, donc au bout de 10 commandes je me retrouvait avec une chaine qui contenait toutes les commandes, vraiment bizarre, le truc que j'ai donc fais c'est que juste après avoir recuperé la ligne sur l'entrée standard et en avoir fait ce que j'avais a faire, je passe l'adresse où était stockée la ligne a une fonction chargée de malloquer l'espace a cet endroit et de le remplir de 0 c'est peut etre ça

fiddy · Answer

Petite question : Pourquoi avoir mis str en static char ?

Mahmah · Answer

Au fait... (Lalalaaa...)


  free(str);
  return (str);


C'est pas un peu dangereux ?



Je suis d'accord aussi pour le static, il est à justifier.


M, plombier ^^

Tisiph0n3 · Answer

bon pour eviter que le pointeur soit écrasé lorque je cherche a connaitre la longueur de str1 j'ai rajouté le mot clé const devant le premier paramètre de strlen ça a déplacé le probleme maintenant c'est un autre pointeur dans une autre fonction qui est écrasé il me cherche on dirait

Tisiph0n3 · Answer

up! :)

Mahmah · Answer

A ce niveau là, il faut un bon debuggeur ou beaucoup de chance...

Je n'ai pas de théorie sur le fait que le const sur le paramètre change quoi que ce soit à lui seul...

On a plusieurs solutions en admettant que ce soit bien un écrasement et non une allocation échouée ou oubliée.

Commence par vérifier tous les tableaux (et donc chaînes de caractères), d'abord ceux en pile, puis les dynamiques, et après les statiques, c'est plus rare mais pourquoi pas...

Cela peut également aider de sécuriser via des assertions pour se rendre compte le plus vite possible d'une erreur.

L'état du précédent patient est stabilisé ? (Plus de fuites et plus de free hâtif ?)

Bonne chasse,

M.

Keybee · Answer

pff c'est tek1 et leur minishell non vraiment voit avec autre etudiant au lieu de man google lol ^^

[C]probleme d'adresse memoire

13 réponses

Discussions similaires