Discussion :

[Funraill.net]

Bonjour

j'ai une fonction qui m permet de lire une ligne dans un fichier,

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
 
void readMem(gchar *ligne, int numLigne){
 
	FILE *fp = NULL;
	gchar buf[81];
	int cpt = 0;
 
	fp = fopen("/proc/meminfo", "r");
	while(fgets (buf, 80, fp)){
      if (cpt == numLigne){
         strcpy(ligne, strstr(buf, ":") +7);
         break;
      }
      cpt++;
   }
	fclose(fp);
}

j'ai cree une structure

Code :

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

1
2
3
4
5
 
typedef struct memInfo{
	gchar	*memTotal;
	gchar	*memFree;
}memInfo;

et cette fonction

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
void getMemInfo(gchar *label){
 
	/* Recupere les informations de la memoire
	*/
	memInfo m;
	readMem(m.memTotal, 0);
	readMem(m.memFree, 1);
 
	strcat(label, "Memoire Total: ");
	strcat(label, m.memTotal);
	strcat(label, "Memoire Libre: ");
	strcat(label, m.memFree);
}

en gros dans ma fonction main():

Code :

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

1
2
3
4
5
 
gchar			*pMemInfo;	
 
getMemInfo(pMemInfo);
contOnglet = g_strdup_printf(pMemInfo);

donc voila j'aimerai pourvoir lui passer mon tableau avec les info du fichier

[granquet]

j'ai pas compris la question ...
mais y me semble que t'as oublié ET la question ET d'allouer de l'espace pour pMemInfo.

que se passe t-il en ce moment? vous prenez tous des cours de rattrapage avec le même prof?

[Pragmateek]

Code :

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

fp = fopen("/proc/meminfo", "r");

"fopen" peut échouer:

Code :

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

1
2
3
if(fp==NULL){
...
}

"fgets" lit n-1 caractères:

Code :

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

while(fgets (buf, sizeof(buf), fp))

[gnto]

A première vu

Code :

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

1
2
3
memInfo m;
readMem(m.memTotal, 0);
readMem(m.memFree, 1);

menTotal et memFree ne sont pas alloué.

et de meme pour pMemInfo

[Pragmateek]

Code :

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

1
2
3
4
typedef struct memInfo{
	gchar	*memTotal;
	gchar	*memFree;
}memInfo;

"memTotal" et "memFree" ne sont que des pointeurs, pas des tableaux.
Donc soit:
-on les garde pointeur et on les fait pointer sur une zone mémoire valide

Code :

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

1
2
3
4
if (cpt == numLigne){
         ligne=strstr(buf, ":") +7;
         break;
      }

-on en fait des tableaux:

Code :

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

1
2
3
4
typedef struct memInfo{
	gchar	memTotal[256];
	gchar	memFree[256];
}memInfo;

et:

Code :

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

1
2
3
4
if (cpt == numLigne){
         strcpy(ligne, strstr(buf, ":") +7);
         break;
      }

[Trap D]

Ceci est un bug vicieux :

Code :

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

1
2
3
4
if (cpt == numLigne){
         ligne=strstr(buf, ":") +7;
         break;
      }

En effet on fournit à ligne l'adresse d'un buffer donc 2 possibilités:
Soit ce buffer est global, mais en cas d'utilisations multiples de ce buffer, ne sera conservée que la dernière information lue, donc on aura une plainte du type : "Je ne comprends pas, j'ai toujours la même information affichée".
Soit ce buffer est local à la fonction de lecture et au sortir de la fonction cette adresse ne sera plus valide et à l'affichage il pourra se passer n'importe quoi.
Cette méthode est donc à proscrire absoulment.
Le mieux dans ce cas est d'utiliser un strdup (ou d'écrire l'équivalent).
D'autre part, il faudrait peut-être tester le retour de strstr, on ne sait jamais.

[Pragmateek]

Ceci est un bug vicieux :

En fait j'ai déclaré "buf" ainsi:

Code :

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

char* buf=malloc(81);

[Trap D]

Tu le réinitialises à chaque lecture ? De toute façon l'adresse de buf n'est pas mémorisé par cette action

Code :

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

ligne=strstr(buf, ":") +7;

Comment fais-tu pour libérer la mémoire ?

[Pragmateek]

Tu le réinitialises à chaque lecture ?

Ou est le problème?
A chaque fois la zone mémoire pointée est différente.
[EDIT]Ah! Pour la libération de la mémoire?[/EDIT]

De toute façon l'adresse de buf n'est pas mémorisé par cette action

Pas celle de buf mais celle de l'élément à "buf"+7

Comment fais-tu pour libérer la mémoire ?

Il faudrait sauvegarder dans un tableau les adresses des zones allouées, ce qui n'est pas pratique.

Ce n'est pas une solution optimale en effet.
Si tu as une idée.

[Trap D]

strdup déjà mentionné

[Pragmateek]

Ca ne va pas poser les mêmes problèmes?

[Trap D]

A la fin de l'utilisation des structures il les libères par des free, si j'ai bien compris le problème, il fait un free(m.memTotal) et un free(m.memFree), mais c'est vrai que je n'ai pas tout suivi.

[Pragmateek]

Le plus simple est de faire une structure de tableaux statiques pour éviter les problèmes d'allocation dynamique.
Et à priori il n'est pas nécessaire de changer le reste du code.