Discussion :

[rod59]

Bonsoir à tous,
je suis étonné de voir de ce code me fait une segmentation fault !!!

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
#include <stdio.h>
#include <string.h>
 
int main (int argc,char* argv[])
{
	char* res ="";
	res=strcat(res,"coucou");
	printf(res);
        return 0;
}

Aprés debuggage je me suis rendu compte que c'est le char* res="" qui marche pas!
Pourquoi?
Comment résoudre mon problème?
Bye
Rod

[Skyounet]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
#include <stdio.h>
#include <string.h>
 
int main (int argc,char* argv[])
{
	char* res ="";
	res=strcat(res,"coucou");
	printf(res);
        return 0;
}

Aprés debuggage je me suis rendu compte que c'est le char* res="" qui marche pas!
Pourquoi?
Comment résoudre mon problème?
Bye
Rod

Comme 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
 
#include <stdio.h>
#include <string.h>
#include <stdlib.h>
 
int main (void)
{
    char *res=NULL;
    res=malloc(sizeof(char)*100);
    if(res != NULL) {
	    sprintf(res,"coucou");
	    printf("%s\n", res);
        free(res);
        res=NULL;
    }
    return 0;
}

char* res ="";
ca c'est juste mais res n'est plus modifiable du coup.

[maniaco_jazz]

ou alors tu recodes une fonction "strcat" qui alloue automatiquement la mémoire suffisante pour la nouvelle chaine... cela évite d'avoir 5 lignes de codes...

[Trap D]

Le pire est que ça peut parfois fonctionner !

[Emmanuel Delahaye]

ou alors tu recodes une fonction "strcat" qui alloue automatiquement la mémoire suffisante pour la nouvelle chaine... cela évite d'avoir 5 lignes de codes

http://emmanuel-delahaye.developpez.com/clib.htm
Module FSTR

[Emmanuel Delahaye]

je suis étonné de voir de ce code me fait une segmentation fault !!!

Rien d'étonnant, le comportement est indéfini.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
#include <stdio.h>
#include <string.h>
 
int main (int argc,char* argv[])
{
	char* res ="";
	res=strcat(res,"coucou");
	printf(res);
        return 0;
}

Aprés debuggage je me suis rendu compte que c'est le char* res="" qui marche pas!
Pourquoi?

1 - 'res' est un pointeur vers une chaine littérale, qui est donc non modifiable.
2 - cette chaine à une taille définie qui ne peut pas être augmentée.

Code :

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

1
2
 
Comment résoudre mon problème?

On peut faire ceci :

Code :

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

1
2
3
4
 
   char res[BIG_ENOUGH] = "";
 
   strcat (res, "coucou");

on peut aussi travailler complètement en dynamique à coup de malloc() / realloc()...

[Médinoc]

Un truc utile sur GCC pour éviter ce genre de choses, c'est:

-Wwrite-strings
Give string constants the type const char[length] so that copy-
ing the address of one into a non-const char * pointer will get
a warning. These warnings will help you find at compile time
code that can try to write into a string constant, but only if
you have been very careful about using const in declarations and
prototypes. Otherwise, it will just be a nuisance; this is why
we did not make `-Wall' request these warnings.

Le truc qui peut embêter, c'est que ni -Wall ni -Wextra ne l'incluent: Il faut l'ajouter soi-même explicitement à la ligne de commande de GCC.