Bonjour,
 

 
Ceci est censez me remplir pRet d'une chaine MD5 (32 caractères).
Je lui allou donc sizeof(char)*32.
 
Le probleme c'est que je segfault et que dans pRet j'ai n'importe quoi en début de chaine.
Je pense que cela est dû à que sprintf li et écrit sur un pointeur en meme temps.
 
Quelle solution ?
 
Merci

 
malloc(33), en comptant le caractere nul en fin de chaine
l'allocation dynamique est vraiment nécéssaire ?
 
tu veux pas plutot faire
 

Si pRet n'est pas initialisé au début, ça va t'écrire ce qu'il y a en mémoire jusqu'à ce que ça trouve un char à 0.
 
Ca peut mener très loin dans la mémoire... jusqu'à un segfault.
 
Et mettre à NULL pRet n'est pas une solution non plus, parce que ...printf() réagit assez mal aux pointeurs NULL
 
La solution est, après avoir alloué pRet, de l'initialiser avec le premier bout de ta chaîne, puis de faire ta boucle sur 16-1 itérations.
 
Autre chose : il faut lui allouer "(32 + 1) * sizeof char" à pRet, pour pouvoir y stocker le caractère \0 terminal.

char *pRet = malloc(sizeof(char) * (32 + 1));
 pRet[0] = '\0';
 
avec le pRet[0] = '\0'; je n'est plus les reste de mémoire.
Mais comment se fait il qu'après un malloc, si j'affiche le contenu de la variable il ya des débris de mémoire ?
 
Et non l'allocation dynamique n'est pas nécéssaire mais si je fait char pRet[33]; comment je peut le return ?

Je seconde Skelter. Je pense donc aussi que le malloc n'est pas nécessaire dans ce cas, et pourrait être avantageusement remplacé par une réservation sur la pile par un simple

Pour avoir juste une petite idée de la complexité qui se cache derrière un simple appel de la fonction malloc(), lire http://www-128.ibm.com/developerwo [...] /l-memory/ et plus spécifiquement les liens en bas de l'article pour des implémentations sur des environnements particuliers.
Edit : Je viens de voir la question pour le return, et il suffit allors de déclarer la variable avec le scope nécessaire, en l'occurence en dehors de la fonction pour qu'elle soit visible depuis la fonction appelante. Habituellement, c'est même la fonction appelante qui fait la réservation mémoire, et qui passe le pointeur à la fonction (comme le propose aussi Trap D).

Tu peux passer un tableau de caractères assez grand en paramètre de ta fonction et écrire dedans, comme ça pas de problème de return.
D'autre part, sizeof(char) = 1 par définition.

et dans ce cas vu que la tailles est fixe, on peut prototyper la fonction de maniere à ce qu'elle ne recoivent que l'adresse de ce type de tableau
 

Merci tous cela marche, mais pourquoi ma variable n'est pas vide après l'avoir initialisé ? (valeur aléatoire de la mémoire)

c'est le pointeur que tu initialise avec une adresse, pas la zone mémoire pointée
pour initiliser la zone mémoire pointée tu peux allouer avec calloc utiliser memset

char SESSID[33];
SESSID[0] = '\0';
 
En faisant la mon sprintf ne déconne plus, je sait pas si c'est très propre, mais je vais opter pour cette solution.
 
Merci à vous.

tu peux faire directement
char SESSID[33] = "";

asprintf

Exactement. Il y a recouvrement, ce n'est pas permis. Il faut une variable intermédiaire.