Discussion :

[SleliLu]

bonjour les developpez, aujourd'hui je decide de recoder la fonction strdup, mais j'ai un petit soucis, lorsque je compile avec une chaine vide, la fonction me renvoie bien NULL, mais lorsque que je lui passe une string ... segfault, je ne comprend pas d'où vient mon problème :/
voici mon code :

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
 
#include <stdlib.h>
#include <stdio.h>
#include <string.h>
 
char    *fct_strdup(char *src)
{
    char    *dest;
    int     i;
    int     LEN;
 
    i = 0;
    if (src == NULL)
        return (NULL);
    while (src[LEN] != '\0')
    {
        LEN++;                                  /* ma fct strlen pour LEN */
    }
    LEN = LEN + 1;
    dest = (char*)malloc(sizeof(*dest) * LEN);  /* appel malloc */
    while (src[i] != '\0')
    {
        dest[i] = src[i];
        i++;                                    /* ma fct strcpy */
    }
    dest[i] = '\0';
    free(dest);
    return(dest);
}
 
int main()
{
    char    *src;
 
    src = "";                                    /* mon problème ... */
    printf("string before strdup : %s\n\n", *src);
    *fct_strdup(*src);
    printf("string after strdup : %s\n", *src);
    return (0);
}

[Médinoc]

LEN n'est pas initialisé, et tu commets l'erreur de libérer la chaîne que tu t'apprêtes à retourner.

Au passage, src devrait être déclaré const, vu que la fonction ne modifie pas la chaîne de départ. Et caster le retour de malloc() est inutile.

[SleliLu]

Merci a toi je m'en vais de ce passer essayer tout ceci =)

...

J'ai donc initialiser mon LEN, enlever mon cast et essayer d'enlever mon free ou de le mettre apres le return, toujours le meme problème, des que je passe une chaine a src dans mon main, je segfault ..

[dahtah]

Merci a toi je m'en vais de ce passer essayer tout ceci =)

...

J'ai donc initialiser mon LEN, enlever mon cast et essayer d'enlever mon free ou de le mettre apres le return, toujours le meme problème, des que je passe une chaine a src dans mon main, je segfault ..

Le probleme ne vient plus de ta fonction, mais de comment tu l'appelle. Tu ne passes pas src correctement a fct_strdup. Passe le pointeur et non la valeur.
Enleve egalement le dereferencement (*) devant fct_strdup dans le main.

En gros, en plus des modifications de medinoc, ceci devrait marcher:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
int main()
{
    char    *src, *dst;
 
    src = "abcd";                                    
    dst = fct_strdup(src);
    printf("string after strdup : %s\n", dst);
    free(dst);
    return (0);
}

[SleliLu]

C'est parfait, merci mille fois, comme quoi j'ai bien fais de poster sur le forum, je comprend mieux mes erreurs =) !
si jamais mon code peut intéresser quelqu'un, le voici :

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
#include <stdlib.h>
#include <stdio.h>
#include <string.h>
 
char    *fct_strdup(char *src)
{
    char    *dest;
    int     i;
    int     LEN;
 
    i = 0;
    LEN = 0;
    if (src == NULL)
        return (NULL);
    while (src[LEN] != '\0')
    {
        LEN++;                                  /* ma fct strlen pour LEN */
    }
    LEN = LEN + 1;
    dest = malloc(sizeof(*dest) * LEN);         /* appel malloc */
    while (src[i] != '\0')
    {
        dest[i] = src[i];
        i++;                                    /* ma fct strcpy */
    }
    dest[i] = '\0';
    return(dest);
}
 
int main()
{
    char    *src;
    char    *dest;
 
    src = "trololo.net";
    dest = fct_strdup(src);
    printf("string after strdup : %s\n", dest);
    free(dest);
    return (0);
}

[Franck.H]

Pourquoi refaire une boucle pour la copie de la chaîne alors que tu inclus le header string.h ? Pareil pour le calcul de la taille de la chaîne passée en argument !

Personnellement, je n'aime pas le nom des variables en majuscule comme LEN car on pourrait croire à tord que tu utilises une constante.

Ta fonction pourrais se résumer à ça:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
char * fct_strdup (const char * src)
{
    char * dest = NULL;
 
    if (src)
    {
       dest = malloc (strlen (src) + 1);
       if (dest)
         strcpy (dest, src);
    }
 
    return dest;
}

Et inutile de mettre des commentaires genre

Code :

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

/* appel malloc */

On le vois donc ce n'est pas la peine et ça pollue le code !