Besoin d'une petite précision sur un malloc

**Sin-an** · 28/04/2009, 16h52

Voici une partie de code sur laquelle je suis tombé :

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
 
char * retour_getenv;
char * opt_adr = NULL;
static char * adresse_serveur = ADR_SRV_DEFAUT;
....	
        retour_getenv = getenv("OPT_ADR");
	if ((retour_getenv != NULL) && (strlen(retour_getenv) != 0) {
		opt_adr = malloc(strlen(retour_getenv + 1);
		if (opt_adr != NULL) {
			strcpy(opt_adr, retour_getenv);
			adresse_serveur = opt_adr;
		}
		else {
			perror("malloc");
			exit(EXIT_FAILURE);
		}
	}

Ma question : Pourquoi à la place de ça :

Code :

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

1
2
3
4
5
opt_adr = malloc(strlen(retour_getenv + 1);
		if (opt_adr != NULL) {
			strcpy(opt_adr, retour_getenv);
			adresse_serveur = opt_adr;
		}

on fait pas ça :

Code :

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

1
2
strcpy(opt_adr, retour_getenv);
			adresse_serveur = opt_adr;

Du coup je me suis posé une seconde question, soit un autre exemple :

Code :

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

1
2
char * chaine = "exemple\0";
chaine = "nouvel exemple\0";

>>> c'est dangeureux de faire ça? En fait puisqu'on alloue pas de place spécifique pour la chaine ça risque pas d'écrire sur autre chose ?

**kamouminator** · 28/04/2009, 17h42

dans:

Code :

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

char * chaine = "exemple\0";

"exemple\0" correspond en fait a une adresse contenant la chaine exemple\0
logique vu qu'un pointeur contient une adresse.

en faisant directement:

Code :

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

strcpy(opt_adr, retour_getenv);

tu va copier le contenu de retour_getenv a l'adresse: 0x00000000 (NULL)
tu ne sais pas ce que contien cette adresse....tu risque d'ecraser des choses..

malloc va lui te donner l'adresse de debut d'une zone memoire de taille finie libre.

PS: ta question n'a rien a voir avec linux !
c'est du C, il aurait fallu la poser dans le forum correspondant.

**Sin-an** · 28/04/2009, 18h44

T'es sur que ça inscrit en 0x00000000 ? pq dans ce cas si on faisait

Code :

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

char *a="salut", *b="SAL\0";

ça donnerait printf("%s",a); // = SAL
non ?
En fait je posais cette question parce que j'ai l'habitude de faire ça mais je viens de realiser que c'était pas vraiment le mieux, enfin bon...

Ué désolé c'est vrai que ça a pas rapport avec prog linux^^, la prochaine fois je le met ailleurs

**kamouminator** · 28/04/2009, 19h03

Envoyé par Sin-an

T'es sur que ça inscrit en 0x00000000 ? pq dans ce cas si on faisait
char *a="salut", *b="SAL\0";
ça donnerait printf("%s",a); // = SAL
non ?
En fait je posais cette question parce que j'ai l'habitude de faire ça mais je viens de realiser que c'était pas vraiment le mieux, enfin bon...

Ué désolé c'est vrai que ça a pas rapport avec prog linux^^, la prochaine fois je le met ailleurs

heu...non !
t'as qu'a essayer...

si tu a l'habitude de faire ca, je te conceil de relire ton cours de C.

**Sin-an** · 28/04/2009, 19h56

Ben justement j'ai essayé, je peux faire char * un = "UN", *deux = "DEUX", *trois = "TRoiss", quand je vais printf un, deux et trois ça affichera UN, DEUX, et TRoiss... donc chaque pointeur ne peut pas pointer vers 0x00000000
:s