Discussion :

[douze_12]

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 :

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

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 :

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

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:

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.

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

[douze_12]

Ok.

Voilà ma fonction "condensée" lectureFichier:

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

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

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!

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

[douze_12]

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