Discussion :

[l1086]

Bonsoir,

J'ai un tableau alloué dynamiquement, le problème ce pose lorsque je veux le remplir.Je souhaite le remplir à partir d'un fichier.
La ligne incriminé est la suivante :

Code :

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

while((fgets(tab[i++],50,fp)) != NULL);

tab est le tableau à 2 dimensions allouer précédemment
fp le pointeur de lecture de fichier

et voici le log d'erreur que j'ai par valgrind:

==12058== Invalid write of size 1
==12058== at 0x40243EA: memcpy (mc_replace_strmem.c:406)
==12058== by 0x409506C: _IO_getline_info (in /lib/tls/i686/cmov/libc-2.6.1.so)
==12058== by 0x4094F50: _IO_getline (in /lib/tls/i686/cmov/libc-2.6.1.so)
==12058== by 0x4093EC9: fgets (in /lib/tls/i686/cmov/libc-2.6.1.so)
==12058== by 0x8048C28: main (lecture.c:280)
==12058== Address 0x0 is not stack'd, malloc'd or (recently) free'd
==12058==
==12058== Process terminating with default action of signal 11 (SIGSEGV)
==12058== Access not within mapped region at address 0x0
==12058== at 0x40243EA: memcpy (mc_replace_strmem.c:406)
==12058== by 0x409506C: _IO_getline_info (in /lib/tls/i686/cmov/libc-2.6.1.so)
==12058== by 0x4094F50: _IO_getline (in /lib/tls/i686/cmov/libc-2.6.1.so)
==12058== by 0x4093EC9: fgets (in /lib/tls/i686/cmov/libc-2.6.1.so)
==12058== by 0x8048C28: main (lecture.c:280)

Merci de votre aide

[vinzzzz]

Je pense qu'il faut que tu poste plus de code...

Comment tu as alloué tes variables? est-ce que ton i corresponds à la dimension de ton tableau? Est-ce que la taille de chaque chaine utilisée pour lire les lignes du fichier est suffisante?

Bref poste ta fonction

[l1086]

Voici donc la fonction incréminée

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
int main()
{
	FILE *lire=NULL;
	lire=fopen("texte.txt","r");
	if(lire == NULL)
	{
		printf("fichier inexistant\n");
		return 1;
	}
 
	char **tab;
	int ligne=57;
	int colonne=167570;
	tab=createtab(ligne,colonne);
 
	int i=0;
	/*insertion*/
	while(fgets(tab[i++],50,lire) !=NULL);
	/*fermer le dico*/
	fclose(lire);
        return 0;
}

[Sve@r]

Voici donc la fonction incréminée

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
int main()
{
	FILE *lire=NULL;
	lire=fopen("texte.txt","r");
	if(lire == NULL)
	{
		printf("fichier inexistant\n");
		return 1;
	}
 
	char **tab;
	int ligne=57;
	int colonne=167570;
	tab=createtab(ligne,colonne);
 
	int i=0;
	/*insertion*/
	while(fgets(tab[i++],50,lire) !=NULL);
	/*fermer le dico*/
	fclose(lire);
        return 0;
}

Hum... et le contenu de "createtab" ???

Il manque la libération de "tab". par ailleurs tu ne contrôles absolument pas "i" par rapport à l'allocation de "tab". Maintenant si chaque ligne ne contient que 50 caractères, tu aurais tout avantage à déclarer "char (*tab)[50 + 1]" comme ça tu gagnes une dimension au malloc et au free...

[l1086]

au bah la fonction createtab c'est ceci
Désolé de l'avoir oublié

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
char **createtab(int sizeI,int sizeJ)
{
        int i;
	char **tab; // tab est un pointeur de pointeur
	tab = (char**)malloc(sizeof(char*)*sizeI);
	if(tab == NULL)
		return NULL;
 
	for(i=0;i<sizeI;i++)
	{
		tab[i] = (char*)malloc(sizeof(char)* sizeJ);
		if(tab[i] == NULL)
			return NULL;
	}
	return(tab);
}

[souviron34]

ben lis le diagnostic du compilateur...

Il te dit tout... (enfin pas tout, mais si tu règles ça tu verras mieux)

[corentin59]

Voici donc la fonction incréminée

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
int main()
{
	FILE *lire=NULL;
	lire=fopen("texte.txt","r");
	if(lire == NULL)
	{
		printf("fichier inexistant\n");
		return 1;
	}
 
	char **tab;
	int ligne=57;
	int colonne=167570;
	tab=createtab(ligne,colonne);
 
	int i=0;
	/*insertion*/
	while(fgets(tab[i++],50,lire) !=NULL);
	/*fermer le dico*/
	fclose(lire);
        return 0;
}

Ce n'est certainement pas ça le pb mais pourquoi faire des lignes de 167570 éléments (ce qui fait que tab pointe vers un espace mémoire de 9Mo) alors qu'on ne s'autorise qu'a mettre 50 éléments par ligne ?

Il y a peut-être une inversion entre "ligne" et "colonne" ?