Problème de mémoire

Version imprimable

27/03/2012, 10h29
douze_12

Problème de mémoire
Bonjour,

J'ai codé 2 fonctions. L'une (lectureFichier) qui lit dans un fichier et stocke n caractères dans un tableau pour retourner ce tableau.
Code:

1 2 3 4 5 6 FILE *fichier = NULL; fichier = fopen(fichierATester, "rb"); unsigned char *lecture = NULL; lecture = malloc(n*sizeof(unsigned char));
La seconde appelle cette 1ere fonction pour en manipuler le tableau renvoyé.

Code:

unsigned char *lecture = lectureFichier(<paramètres>);

Lorsque j'appelle lectureFichier dans une boucle, pas de souci, celle-ci s'exécute.

Lorsque j'appelle la 2nde, elle "bloque" à 2 itérations et j'ai l'erreur suivante:

Citation:

L'instruction à "0x......." emploie l'adresse mémoire "0x01010101". La mémoire ne peut pas être "read".

Je ne vois pas d'où peut venir le problème puisque l'allocation se passe très bien dans lectureFichier...

Des idées?

Merci d'avance.
27/03/2012, 10h36
gangsoleil

Bonjour,

Tu accedes a une zone memoire sans y avoir le droit. Sans plus de code, et sans boule de cristale, impossible de te repondre.

Essaie de reduire ton code au minimum pour reproduire ton probleme, et poste-le ici.

Ok.

Voilà ma fonction "condensée" lectureFichier:

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
unsigned char* lectureFichier(char fichierATester[], int n)
{
    FILE *fichier = NULL;
    fichier = fopen(fichierATester, "rb");
 
    int	i = 0;
 
	unsigned char *lecture = NULL;
	lecture = malloc(n*sizeof(unsigned char));
	if (lecture == NULL) // On vérifie si l'allocation a marché
    {
        exit(0);
    }
 
	size_t nbElementsLus = 0;
 
	for (i=0; i<n; i++)
	{
	    nbElementsLus = fread(&lecture[i],sizeof(unsigned char),1,fichier);
        if (nbElementsLus == 0)
        {
        //La fin de fichier a été prématurément atteinte
        exit(0);
        }
    }
	fclose(fichier);
	return lecture;
}

Voici la fonction qui l'utilise:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
double fonction2(char fichierATester[], int n)
{    
    unsigned char *lecture = lectureFichier(fichierATester, n);
 
    int	i = 0;
    double sum = 0.0;
 
	for (i=0; i<n; i++) 
	{
	    sum = sum + 2*lecture[i]-1; 
    }
    free(lecture);
	return sum;
}

Puis dans le main, si j'appelle plusieurs fois (dans une boucle for) lectureFichier, pas de souci. Mais si j'appelle plusieurs fois fonction2, erreur comme détaillée précédemment.

A noter que si j'appelle ma fonction dans une boucle c'est pour lire à différents endroits du fichier, mais je ne rentre pas dans les détails ce n'est pas pertinent ici je crois.

MERCI!

27/03/2012, 11h08
diogene

Le seul point pouvant poser problème que je constate est que tu ne vérifies pas l'ouverture correcte du fichier dans lectureFichier(). Si elle échoue, le programme plante sur le fread().

Si les ouvertures de fichiers sont correctes, le problème est ailleurs dans le code.

De toute façon, du point de vue des traitements d'erreurs, ce code est complètement à revoir :
- ouverture du fichier non vérifiée comme déjà mentionné
- si le malloc échoue, le fichier reste ouvert (si il existe)
- si on atteint la fin de fichier prématurément ou en cas d'erreur de lecture, le fichier reste ouvert et l'allocation dynamique n'est pas récupérée.
27/03/2012, 14h53
douze_12

Merci diogene, le problème venait effectivement de la mauvaise gestion des erreurs! :)