Bonjour,

Voilà, après deux jours de recherche sur ce forum et d'autre, je n'arrive pas à mettre la main sur ce bug bien embêtant.

Mon fichier lit un fichier ppm en entrée, parse quelques valeurs, la taille de l'image, sa profondeur et stocke le tout dans une matrice, mais lors de l'appel à fclose, le programme se plante.

J'ai essayé de comprendre les traces de valgrind, je soupçonne une mauvaise utilisation de ma structure mais je n'arrive pas à trouver d'exemple qui contredise l'utilisation que j'en fait.

Voilà le code source, la fonction incriminée est chargerImage

LirePPM.c
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
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
#include <stdlib.h>
#include <string.h>
 
#include "LirePPM.h"
#define TAILLE_MAX 70 /* spécification fichier ppm */
 
 
int chargerImage(char *nomFichier, Image *img) {
     FILE *fichier = NULL;
     char ligne[TAILLE_MAX];
     int tmp;
     int i, j;
 
     fichier = fopen(nomFichier, "r+");
 
     /* ouverture du fichier */
     if (fichier != NULL) {
 
	  /* récupération du nom de fichier */
	  fscanf(fichier, "%s", ligne);
	  if (strcmp(ligne, "P2") != 0) {
	       printf("Fichier non valide...\n");
	       exit(EXIT_FAILURE);
	  }
	  printf("Fichier valide...\n");
 
	  /* récupération dimension image */
	  while (fscanf(fichier, "%d %d", &img->nbColonnes, &img->nbLignes) != 2) {
	       fscanf(fichier, "%s", ligne);
	  }
	  printf("Chargement d'une image de %dx%d", img->nbLignes, img->nbColonnes);
 
	  /* récupération profondeur */
	  while (fscanf(fichier, "%d", &img->profondeur) != 1) {
	       fscanf(fichier, "%s", ligne);
	  }
	  printf(" de profondeur %d...\n", img->profondeur);
 
	  fclose(fichier);
 
 
 
	  /* Allocation de la matrice */
	  img->image = malloc(img->nbLignes * sizeof(img->image));
 
	  for(i = 0; i < img->nbLignes; i++)
	  {
	       img->image[i] = malloc(img->nbColonnes * sizeof(img->image[i]));
	  }
 
	  printf("Allocation image done...\n");
 
	  i = j = 0;
 
/*      /\* Récupération des valeurs du fichier *\/ */
/*      while((fscanf(fichier, "%d", &tmp)) != EOF)  */
/*      { */
/* 	  img->image[i][j] = tmp; */
/* 	  j++; */
 
/* 	  if (j == img->nbColonnes)  */
/* 	  { */
/* 	       i++; */
/* 	       j = 0; */
/* 	  } */
/*      } */
 
/* 	printf("Pointeur fichier : %x\n", fichier); */
 
/*      fclose(fichier); */
     }
 
     return 1; /* chargement du fichier OK */
}
 
 
void detruireImage(Image *img) {
     int i;
 
     for (i = 0; i < img->nbLignes; i++) {
	  free(img->image[i]);
	  img->image[i] = NULL;
     }
 
     free(img->image);
     img->image = NULL;
}
 
int ecrireImage(const Image *img, char *nomFichier) {
     int i, j;
     FILE *fichier = fopen(nomFichier, "a");
 
     /* vérification de l'existence d'une image */
     if(img == NULL) {
	  return 0;
     }
 
     /* erreur création du fichier */     
     if (!fichier)
     {
	  return 0;
     }
 
     /* écriture identifiant fichier */
     fprintf(fichier, "%s\n", "P2");
 
     /* ériture nombre lignes et colonnes */
     fprintf(fichier, "%d %d\n", img->nbColonnes, img->nbLignes);
 
     /* écriture profondeur fichier */
     fprintf(fichier, "%d\n", img->profondeur);
 
     /* écriture des valeurs du fichier */
     for (i = 0; i < img->nbLignes; i++) {
	  for (j = 0; j < img->nbColonnes; j++) {
	       fprintf(fichier, "%d\n", img->image[i][j]);
	  }
     }
 
     /* fermeture du fichier */
     fclose(fichier);
 
     /* écriture ok */
     return 1;
}
 
int main(int argc, char **argv) {
     Image *img;
     int retourChargerImage;
     if (argc >= 2)
     {
	  img = malloc(sizeof(img));
	  retourChargerImage = chargerImage(argv[1], img);
 
	  if (argc == 3) {
	       printf("Ecriture image dans : %s", argv[2]);
	       ecrireImage(img, argv[2]);
	  }
 
	  printf("Destruction image...");
 
	  detruireImage(img);
	  printf(" done.\n");
     }
 
 
     return 0;
}
LirePPM.h
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
 
#ifndef __LIRE_PPM__
#define __LIRE_PPM__
 
#include <stdio.h>
 
 
typedef struct {
     int **image;
     int profondeur;
     int nbLignes;
     int nbColonnes;
} Image;
 
int chargerImage(char *nomFichier, Image *img);
 
void detruireImage(Image *img);
 
int ecrireImage(const Image *img, char *nomFichier);
#endif
Trace du valgrind
http://pastebin.com/m626a0c45

Merci de votre aide.

Harold