"Segmentation Fault" avec malloc entre 757 et 1973.

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : "Segmentation Fault" avec malloc entre 757 et 1973.

thoduv

Bonjour,
Je recontre en ce moment une erreur très étrange : tous les malloc() que je fais pour une taille comprise entre 757 et 1973 font planter mon programme en renvoyant une "Segmentation fault" ... Si j'essaie avec 1 ou 65536 ca marche, mais entre 757 et 1973, non. En plus ces nombres ne correspondent à rien que je connaisse ...

:??:

Merci d'avance !

Publicité

Elmoricq

Modérateur

Comportement aléatoire = tu as un problème de gestion de la mémoire dans ton programme. Qui dépasse le cadre de ton seul malloc().

thoduv

Arg, juste ce que je redoutais ... Bon bah je vais tout revoir alors ... :cry:

Sinon tu peux me donner des examples de "problèmes habituels" de gestion de mémoire ?

Message cité 1 fois

Elmoricq

Modérateur

thoduv a écrit :

Sinon tu peux me donner des examples de "problèmes habituels" de gestion de mémoire ?

T'as le très classique débordement de capacité :

Code :

#include <stdio.h>
#include <stdlib.h>
#include <strings.h>
int main (void)
{
char source[] = "machin";
char *cible = malloc(strlen(source) * sizeof *cible);
if ( cible )
{
strcpy(cible, source);
printf("cible = %s\n", cible);
free(cible);
}
return 0;
}

Ca semble fonctionner, sauf qu'aléatoirement tu auras des surprises à l'affichage de "cible".
Et avec un programme plus étoffé avec ce genre d'erreur, tu te retrouves avec des comportements étranges et aléatoires.

Mais des exemples de problèmes mémoires, il y en a des tas d'autres.

Message édité par Elmoricq le 03-11-2005 à 21:49:18

thoduv

Apparement mon problème vient de cette fontion, qui splitte une chaine :

Code :

char** explodestring( char* str, const char* separator )
{
int i = 0;
size_t sizeSeparator = strlen( separator );
int explodedArraySize = 1;
char **explodedArray;
char *pos;
char *explodedStr;
explodedStr = malloc(strlen(str)+2);
strcpy(explodedStr, str);
pos = explodedStr;
while(pos=strstr(pos, separator))
{
explodedArraySize++;
pos+=sizeSeparator;
}
explodedArray = malloc((explodedArraySize + 1)* sizeof(char*));
pos = explodedStr;
explodedArray[0] = pos;
while(pos=strstr(pos, separator))
{
*pos = '\0';
pos+= sizeSeparator;
explodedArray[++i] = pos;
}
return explodedArray;
}

Je vois pas ce qui colle pas ...

Message cité 1 fois

Elmoricq

Modérateur

Hmm. Tu retournes un char**, mais comment une fonction qui le lira saura où s'arrêter dans son parcours ?
Tu ne retournes pas la taille de ton char**, et tu ne le termines pas par NULL non plus.

Emmanuel Delahaye

C is a sharp tool

thoduv a écrit :

Apparement mon problème vient de cette fontion, qui splitte une chaine :
<...>Je vois pas ce qui colle pas ...

Je ne sais pas trop ce que fait

explodedArray[++i] = pos;

mais si tu veux

i++;
explodedArray[i] = pos;

ecrit le comme ça, c'est plus clair pour tout le monde.

malloc() ne fait pas d'initialisation. Il manque un

i++;
explodedArray[i] = NULL;

après la dernière boucle.

En fait, je l'aurais ecrite comme ça :

{
int i = 0;
while(pos=strstr(pos, separator))
{
*pos = '\0';
pos += sizeSeparator;
explodedArray[i] = pos;
i++;
}
explodedArray[i] = NULL;
}

Message édité par Emmanuel Delahaye le 05-11-2005 à 12:20:18

---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/

thoduv

Voilà la fonction corrigée, mais ce ne marche toujours pas, l'erreur est peut-être ailleurs. Mais est-ce qu'il y a encore des erreurs sur cette fonction :

Code :

int explodestring(char ***out, char* str, const char* separator )
{
int i = 0;
size_t sizeSeparator = strlen( separator );
int explodedArraySize = 1;
char **explodedArray;
char *pos;
char *explodedStr;
explodedStr = malloc(strlen(str)+2);
strcpy(explodedStr, str);
pos = explodedStr;
while(pos=strstr(pos, separator))
{
explodedArraySize++;
pos+=sizeSeparator;
}
explodedArray = malloc((explodedArraySize + 2) * sizeof(char*));
pos = explodedStr;
explodedArray[i] = pos;
i++;
while(pos=strstr(pos, separator))
{
*pos = '\0';
pos+= sizeSeparator;
explodedArray[i] = pos;
i++;
}
explodedArray[i] = NULL;
*out = explodedArray;
return i;
}

thoduv

Finalement c'est bon, j'avais oublié d'initialiser un pointeur que je pensais être défini par un fonction, mais laquelle fonction verifiait son contenu (qui était érronné par les malloc/free precedents). Merci quand même !

chrisbk

(j'aime pas le fait que ta fonction modifie son entrée [:rogr] )

Message cité 1 fois

Publicité

el muchacho

Comfortably Numb

(sans oublier que la fonction standard strtok avec une boucle fait à peu près la même chose)

---------------
Les aéroports où il fait bon attendre, voila un topic qu'il est bien

thoduv

chrisbk a écrit :

(j'aime pas le fait que ta fonction modifie son entrée [:rogr] )

Moi non plus mais j'ai pas trouvé mieux.

Message cité 1 fois

Emmanuel Delahaye

C is a sharp tool

thoduv a écrit :

Moi non plus mais j'ai pas trouvé mieux.

Retourner un

struct tokens
{
char **a;
size_t n;
};

Message édité par Emmanuel Delahaye le 06-11-2005 à 20:48:55

---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/

FORUM HardWare.fr

Programmation

"Segmentation Fault" avec malloc entre 757 et 1973.

Sujets relatifs
Erreur Segmentation	segment fault pour du C (tabl d'élément)
[C++] petit pb de seg. fault	[C/C++] kernel 2.6.12 => segmentation fault
Erreur de segmentation	erreur de segmentation
[ C++ ] Passer par adresse un vecteur à 2 dimensions + malloc	problème avec malloc enfin je pense ...
erreur de segmentation pourtant simple
Plus de sujets relatifs à : "Segmentation Fault" avec malloc entre 757 et 1973.

Page générée en 0.055 secondes