Free() et Problème mémoire

**Soum** · 11/01/2006, 10h05

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 :

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

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

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

Je vous montre l'algo :

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
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 ?

**gangsoleil** · 11/01/2006, 10h30

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é.

**Soum** · 11/01/2006, 11h41

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 ?

**fearyourself** · 11/01/2006, 11h49

Ton free marchait parce que tu as fait cette affectation

Code :

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

string1=tableau->d_name;

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

Jc

**gege2061** · 11/01/2006, 11h50

Bonjour,

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.

http://emmanuel-delahaye.developpez.com/

**Emmanuel Delahaye** · 11/01/2006, 13h11

Envoyé par Soum

Je vous montre l'algo :

Ne compile pas. Merci de poster un code minimum mais compilable qui montre le problème.

**AlBoLeToNo** · 11/01/2006, 15h01

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;

**Emmanuel Delahaye** · 11/01/2006, 17h17

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 :

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

1
2
3
4
char * maChaine   = NULL;
long * monTableau = NULL;
int    i = 0;
int    j = 0;

**Soum** · 11/01/2006, 17h23

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

**ggnore** · 11/01/2006, 17h42

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.

**Patriarch24** · 11/01/2006, 22h07

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

Source d'amusements chez certains, de calvitie chez d'autres.

**PRomu@ld** · 12/01/2006, 08h57

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 ...

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 ...

**Soum** · 12/01/2006, 10h43

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 ...

Free() et Problème mémoire

C

Vue hybride

Discussions similaires

Partager

Partager