Free() et Problème mémoire

Version imprimable

Bonjour bonjour,

Voilà je suis sur un projet d'indexation de fichier dans un répertoire.
j'utilise des pointeurs pour envoyer des données dans un tableau et récupérer les données d'un fichier.

j'ai cette erreur :

Citation:

*** glibc detected *** free(): invalid pointer: 0xfef00d44 ***

ou encore celle ci après modification de l'algo

Citation:

*** glibc detected *** double free or corruption: 0x080c75c7 ***

Je vous montre l'algo :

Code:

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
 
void Lecture()
{
    int test=0, test2=0, trouve=1;
    char *string1, *string2;
    temp=0;
    f1=fopen(nomfich,"r");
    while ((tableau=readdir(dossier))!=NULL)
    {
        test=strlen(tableau->d_name)+1;
        if (test>21)
        {
            rewind(f1);
            tabfich[temp]= malloc(strlen(tableau->d_name)+1);
            string1=tableau->d_name;
            trouve=1;
            while ((fgets(string2,test,f1)!=NULL)&&(trouve!=0))
            {
                test2=strlen(string2);
                if (test2>21)
                {
                    test2=strncmp(string1, string2, test-1);
                    if (test2==0)
                    {
                        trouve=0;
                    }
                }
            }
            if (trouve!=0)
            {
                ouinon=1;
                tabfich[temp]=string1;
                temp++;
            }
        }
    }
    fclose(f1);
    free(string1);
    free(string2);
}

Vous avez une idée ?

11/01/2006, 10h30
gangsoleil

Bonjour,

Tu déclares deux pointeurs de caractères string1 et string2. Et puis tu les utilises directement, sans même les avoir initialiser (regarde du coté des fonctions d'allocation mémoire comme malloc). Donc il est normal que lorsque tu essaies de libérer la mémoire allouée, il t'engueule, puisque tu n'as rien alloué.
11/01/2006, 11h41
Soum

Citation:

Envoyé par gangsoleil

Bonjour,

Tu déclares deux pointeurs de caractères string1 et string2. Et puis tu les utilises directement, sans même les avoir initialiser (regarde du coté des fonctions d'allocation mémoire comme malloc). Donc il est normal que lorsque tu essaies de libérer la mémoire allouée, il t'engueule, puisque tu n'as rien alloué.

lol vi j'ai vu ça ... et pourtant en bidouillant un peu la solution est arrivée toute seule.

En fait j'ai enlevé tous les malloc et les free et ça fonctionne dix fois mieux mdrrr

comme quoi des fois ...

Merci pour ton aide dans tous les cas ;)

Petit détail que je n'ai pas compris ... ma deuxième solution fonctionnelle marchait si simplement je faisais un free sur string1 ...

Quelqu'un a une idée ? :)
11/01/2006, 11h49
fearyourself

Ton free marchait parce que tu as fait cette affectation

Code:

string1=tableau->d_name;

Tu as surement fait un malloc pour la variable d_name...

Jc
11/01/2006, 11h50
gege2061

Bonjour,

Citation:

Envoyé par Soum

et pourtant en bidouillant

Un conseil : change de langage !
Le C n'est pas un langage pour bidouilleurs. Essai le KPL ;)

Maintenant si tu souhaite continue le C, il va falloir apprendre qu'à chaque malloc, doit correspondre un free; que certaines fonctions peuvent échouer (fopen, malloc par exemple), il faut vérifier ceci.

:arrow: http://emmanuel-delahaye.developpez.com/
11/01/2006, 13h11
Emmanuel Delahaye

Re: Free() et Problème mémoire

Citation:

Envoyé par Soum

Je vous montre l'algo :

Ne compile pas. Merci de poster un code minimum mais compilable qui montre le problème.
11/01/2006, 15h01
AlBoLeToNo

Citation:

Envoyé par gege2061

Un conseil : change de langage !
Le C n'est pas un langage pour bidouilleurs. Essai le KPL

Roh c'est méchant ça :).

Par contre, entièrement d'accord, les bidouilles rattrapent vite le programmeur (surtout quand on manipule la mémoire en C !).
Tout début de corps de fonction doit comporter les déclarations PUIS les initialisations.
Si tu oublies des initialisations, tu risques de libérer des zones de mémoire qu'il ne faut pas ! Ou d'écraser des zones de mémoire !

/* Déclarations. */
char * maChaine;

long * monTableau;

int i;
int j;

/* Initialisations. */
maChaine = NULL;
monTableau = NULL;

i = 0;
j = 0;
11/01/2006, 17h17
Emmanuel Delahaye
Citation:

Envoyé par AlBoLeToNo

Si tu oublies des initialisations, tu risques de libérer des zones de mémoire qu'il ne faut pas ! Ou d'écraser des zones de mémoire !

/* Déclarations. */
char * maChaine;

long * monTableau;

int i;
int j;

/* Initialisations. */
maChaine = NULL;
monTableau = NULL;

i = 0;
j = 0;

Hou que c'est laid !
Code:

1 2 3 4 char * maChaine = NULL; long * monTableau = NULL; int i = 0; int j = 0;
11/01/2006, 17h23
Soum

lol j'explique l'affaire ...

mon maitre de stage m'a demandé de réaliser un script dans un langage que je n'avais jamais approché lundi de la semaine dernière ... le c
à partir de là ceci explique cela mais bon j'aurais pu faire pire car j'ai fini aujourd'hui ce projet là ... carpe diem ...

et vive la bidouille quand meme quelque fois :p
11/01/2006, 17h42
ggnore

Citation:

Envoyé par Soum

lol j'explique l'affaire ...

mon maitre de stage m'a demandé de réaliser un script dans un langage que je n'avais jamais approché lundi de la semaine dernière ... le c
à partir de là ceci explique cela mais bon j'aurais pu faire pire car j'ai fini aujourd'hui ce projet là ... carpe diem ...

et vive la bidouille quand meme quelque fois :p

Tu devrais peut être poster ton code fini : Ce n'est pas parceque ton code fonctionne 99 fois que la 100ème fois il ne plantera pas.

Bidouiller avec la mémoire en langage C, c'est pas exactement conseillé.

Oh, et tu n'es pas obligé de ponctuer tes phrases pas kikoololmdr, y a rien de drôle (ou alors mon humour est aux antipodes du tien), et ça nuit à la lisibilité.

Ce forum n'est pas un portable ou canal d'irc. Merci.
11/01/2006, 22h07
Patriarch24

Citation:

Bidouiller avec la mémoire en langage C, c'est pas exactement conseillé.

Source d'amusements chez certains, de calvitie chez d'autres.
12/01/2006, 08h57
PRomu@ld

Citation:

mon maitre de stage m'a demandé de réaliser un script dans un langage que je n'avais jamais approché lundi de la semaine dernière ... le c

J'ai jamais entendu dire que le langage C était un langage de script, on m'avait parlé de langage de programmation ...

Citation:

gege2061 a écrit:
Un conseil : change de langage !
Le C n'est pas un langage pour bidouilleurs. Essai le KPL

Roh c'est méchant ça .

Non pas méchant, réaliste ...
12/01/2006, 10h43
Soum

Citation:

Envoyé par PRomu@ld

J'ai jamais entendu dire que le langage C était un langage de script, on m'avait parlé de langage de programmation ...

exact mais vu ce que je fais en ce moment ça ressemble plus à du scripting ... je me comprend ...