High-Tech Santé Médecine Droit-Finances

Grippe A

Que dois-je faire ?

Rechercher : dans
Par :

[langage C] segmentation fault

Dernière réponse le 24 sep 2006 à 11:33:12 Benja85, le 21 sep 2006 à 00:14:05 
 Signaler ce message aux modérateurs

Bonjour !

Je sais que ce sujet a déjà été abordé dans des messages précédents, mais malgré ça j'arrive toujours à une erreur.

Mon programme est simplement la définition de strcpy :

char my_strcpy(char *s1, const char *s2)
{
s1 = (char *) malloc (sizeof s2);
int i;
for (i=0; s2[i] != '\0'; i++) s1[i] = s2[i]; s1[i]= '\0';

return *s1;
}

Je suis sur que le problème est vraiment trivial mais je débute en C, merci d'avance !

Meilleures réponses pour « [langage C] segmentation fault » dans :
Langage C++ - Les types de données Voir Les types de données Les données manipulées en langage C++, comme en langage C, sont typées, c'est-à-dire que pour chaque donnée que l'on utilise (dans les variables par exemple) il faut préciser le type de donnée, ce qui permet de connaître...
Langage C - Les types de données Voir Les types de données Les données manipulées en langage C sont typées, c'est-à-dire que pour chaque donnée que l'on utilise (dans les variables par exemple) il faut préciser le type de donnée, ce qui permet de connaître l'occupation mémoire (le...
Langage C - Les variables Voir Le concept de variable Une variable est un objet repéré par son nom, pouvant contenir des données, qui pourront être modifiées lors de l'exécution du programme. Les variables en langage C sont typées, c'est-à-dire que les données contenues dans...
[Langage C] C/C++ Erreur de segmentation VoirQu'est ce qu'une erreur de segmentation Vous êtes en train de développer une application sous Linux en C/C++. Tout va bien, ça compile, les oiseaux chantent. Donc vous lancez votre application pour la tester. Et vous obtenez l'un de ces deux...
Les files en langage C VoirLes files - Premier Entré Premier Sorti Requis I. INTRODUCTION II. Définition III. La construction du prototype d'un élément de la file IV. Opérations sur les files A. Initialisation B. Insertion d'un élément dans la file C. Oter un...
Les piles en langage C VoirLes piles Requis I. INTRODUCTION II. Définition III. La construction du prototype d'un élément de la pile IV. Opérations sur les piles A. Initialisation B. Insertion d'un élément dans la pile C. Ôter un élément de la pile D. Affichage...
Langage C - Les chaînes de caractères VoirQu'est-ce qu'une chaîne de caractères ? Une chaîne de caractères (appelée string en anglais) est une suite de caractères, c'est-à-dire un ensemble de symboles faisant partie du jeu de caractères, défini par le code ASCII. En langage C, une chaîne...
Introduction au langage C VoirPetite histoire du C Le langage C a été mis au point par D.Ritchie et B.W.Kernighan au début des années 70. Leur but était de permettre de développer un langage qui permettrait d'obtenir un système d'exploitation de type UNIX portable. D.Ritchie...
Langage C++ - Les pointeurs VoirComme en langage C, le langage C++ permet d'utiliser des pointeurs pour manipuler des données, mais il introduit aussi le concept de référence, très pratique pour permettre la modification d'une donnée passée en paramètre d'une fonction. Définition...
Posez votre question Répondre

1

Char Snipeur, le 21 sep 2006 à 08:32:21

Salut.
Je suis pas très calé en allocation de mémoire, mais je vais essayer de t'aider.
Je ne suis pas sur que sizeof s2 te donne la longueur de la chaine.
Tout d'abord, pour debugger, fait un printf de "sizeof s2" voir si cela correspond bien à la longueur de s2. Si oui, ne manquerai t'il pas une largeur de type ?
tu peux peut être remplacer sizeof s2 par strlen(s2).
Bonne chance Salutation !
Char Snipeur

   
Répondre à Char Snipeur

2

Benja85, le 21 sep 2006 à 10:12:24

Sizeof (s2) me donne bien la longueur de s2 (avec le dernier char \0).

Le problème se situe ailleurs, ça doit être une histoire de pointeur mais je nage un peu pour le moment là dedans...

   
Répondre à Benja85

3

carkiasis, le 21 sep 2006 à 21:53:23

Si tu travailles sous linux, tu peux utiliser ddd (je ne sais pas si ce prog existe aussi sous Windows). Cela te permettra de voir à quelle ligne se situe le segmentation fault...

Bon courage

   
Répondre à carkiasis

4

Char Snipeur, le 22 sep 2006 à 08:49:52

Oula !
J'avais pas fait gaffe !
les s1[i] en dehors du for
à éviter.
vérifie sa valeur avant de l'utiliser, il est possible que le problème vienne de là Salutation !
Char Snipeur

   
Répondre à Char Snipeur

5

Char Snipeur, le 22 sep 2006 à 08:56:27

Après test chez moi, en fait, il n'y a pas d'erreur.
Je pense donc que cela viens d'ailleur Salutation !
Char Snipeur

   
Répondre à Char Snipeur

6

Benja85, le 22 sep 2006 à 09:57:24

Je me suis débrouillé finalement, en enlevant ce qui posait problème, j'arrive à un truc de ce genre :

char* my_strcpy (char* s1, const char* s2)
{
int i;
for (i=0; s2[i] != '\0'; i++) s1[i] = s2[i];

return s1;
}

Ainsi je ne me soucie plus du problème de l'allocation de mémoire.
Il est tout de même logique que avant d'appeler la fonction my_strcpy, il faut faire de la place, alors j'ai fais ca dans ma méthode main (pour les tests uniquement) :

char* src = "Tests";
char* dst = malloc(sizeof(src));
printf("%s\n", my_strcpy(dst, src));

Voilà tout simplement :) merci à vous !

   
Répondre à Benja85

7

tafiscobar, le 22 sep 2006 à 10:26:54
  • +2

Salut, tout d'abord s2 est un pointeur sur un const char. Donc sizeof s2 te donne la taille du pointeur ie 4octets et non la taille de ta chaine (si c'est vrai, le compilateur n'est pas normé iso, ce n'est pas le comportement de sizeof sinon il n'y aurait pas de strlen).

Ensuite, si tu alloues de la mémoire dans la fonction my_strcpy, ben malloc te retourne un pointeur local et donc quand tu sors de ta fonction, ton pointeur local est perdu et donc d'ou le segmentation fault. Le malloc est fait par l'utilisateur et non dans la fonction strcpy.

Ensuite strcpy retourne le pointeur sur un char ie char* et non un char. Ta fonction devrait etre: 

char*
my_strcpy (char* dest, const char* src) {
    int i;
    for (i = 0; src[i]!='\0'; ++i)
          dest[i] = src[i];
   dest[i] = '\0';
   return dest;
}
tafiscobar "lou waye def bopame"
la nullite n'existe pas, l'ignorance oui, ah je suppose!!!

   
Répondre à tafiscobar

8

Benja85, le 22 sep 2006 à 22:18:00

Merci! Comme t'a vu + haut j'arrive au même résultat :p J'avais juste oublié de rajouter le dernier char \0 à ma destination.

Ca roule :p

   
Répondre à Benja85

9

mamiemando, le 23 sep 2006 à 11:21:51

L'erreur de segmentation était à mon avis plutôt dûe au fait qu'il fallait écrire : 

char *dst = (char *) malloc(sizeof(char)*strlen(src));

Si ton my_strcpy retourne le char *, rien ne t'empêche de faire le free dessus à l'extérieur. 
#include <stdlib.h>
#include <string.h>
#include <stdio.h>

char*
my_strcpy (char* dest, const char* src) {
  int i;
  dest = (char *) malloc(sizeof(char)*strlen(src));
  for (i = 0; src[i]!='\0'; ++i) dest[i] = src[i];
  dest[i] = '\0';
  return dest;
}

int main(){
  char *src = "tapir",dst;
  my_strcpy(dst,src);
  printf("dst = %s\n",dst);
  free(dst);
  getchar(); // pour les windowsiens
  return 0;
}

Je n'ai pas testé mais ça doit marcher ça, non ?

   
Répondre à mamiemando

10

arth, le 23 sep 2006 à 11:46:22

En même temps essayer de retourner des chaines de caractères en c ca devient risqué ^^
Le loup, solitaire et mystérieux.

   
Répondre à arth

11

 mamiemando, le 24 sep 2006 à 11:33:12

Je ne vois pas pourquoi. En fait de manière générale on essaye juste de mettre le calloc/malloc dans le même horizon que le free (ce qui n'est pas le cas de l'exemple que j'ai donné) et c'est pour ça que la version de tafiscobar est plus propre. Il faut juste avec sa version penser à allouer la chaine dest dans le main (et à la désallouer).

   
Répondre à mamiemando
Posez votre question Répondre
Ils ont besoin de votre aide
Collection CommentÇaMarche.net