Discussion :

[Amnael]

Bonjour, j'ai un soucis avec mon realloc qui provoque visiblement une fuite mémoire avec un invalid next size.

Mais je ne vois pas du tout le soucis dans le code..

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(){
	int buffsize=4;
	int init_size=4;
	int16_t * x=(int16_t*)malloc(buffsize*sizeof(int16_t));
	int16_t i;
	int bytesRead;
	char *fi="test.wav";
	FILE *fd_waveFile=NULL;
	wavfile head_waveFile;
	int cpt=0;
	wavRead(&fd_waveFile,fi,&head_waveFile);
 
	while ((bytesRead = fread(&i, sizeof(int16_t),1, fd_waveFile)) > 0){
		if(cpt==(buffsize-1)){
			x=(int16_t*)realloc(x,init_size*sizeof(int16_t));
			buffsize=buffsize+init_size;
		}
		x[cpt]=i,
 		cpt++;
	}
return(0);
}

L'erreur:

*** glibc detected *** ./test:realloc: invalid next size : une adresse mémoire

suivi d'une bonne vingtaine de ligne de bactrace.

[Invité]

C'est du C non?
fd_waveFile c'est un pointeur de fichier? a quoi sert "wavRead(&fd_waveFile,fi,&head_waveFile);", parce que si c'est une fonction qui permet de lire entete wav du fichier pointer par "fd_waveFile", il y-a un soucis puisque fd_waveFile == NULL. Donc il n-'y a aucun fichier ouvert.

[Amnael]

Non c'est bien du c++;

et l'argument qui te dérange est un argument en sortie c'est donc normal qu'il ait cette forme là.

Code :

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

1
2
3
4
5
6
7
8
9
void wavRead(FILE **p_wav, char *filename, wavfile *p_header) {
	//fopen_s(p_wav, filename, "rb"); //Windows version
	*p_wav = fopen(filename, "rb");
	if (*p_wav == NULL) {
		fprintf(stderr, "Can't open input file %s\n", filename);
		exit(1);
 
......
	}

Par contre là mon erreur a évolué et ne vient plus du realloc j'ai une segfault avant

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
int main(){
	int init_size=2000;
	int buffsize=init_size*sizeof(int16_t); //4 000
	int16_t * x=(int16_t*)malloc(init_size*sizeof(int16_t));
	int16_t i;
	int bytesRead;
	char *fi="test.wav";
 
 
	FILE *fd_waveFile=NULL;
	wavfile head_waveFile;
	int cpt=0;
	wavRead(&fd_waveFile,fi,&head_waveFile);
 
	while ((bytesRead = fread(&i, sizeof(i),1, fd_waveFile)) > 0){
		if(cpt==(buffsize)){
			//x=(int16_t*)realloc(x,init_size*sizeof(int16_t)); j'ai commenté ça pour tester
			buffsize=buffsize+init_size*sizeof(int16_t);
		}
		x[cpt]=i,
 		std::cout << "cpt=" << cpt << std::endl;
		std::cout << "buffsize=" << buffsize << std::endl;
		cpt++;
	}
return(0);
}

On sort avec une seg fault avec

buff size=4000
cpt=2013

[Invité]

a la ligne 15, si tu remplacer :

Code :

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

x=(int16_t*)realloc(x,init_size*sizeof(int16_t));

par

Code :

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

x=(int16_t*)realloc(x,buffsize*sizeof(int16_t));

et a quoi te sert:

Code :

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

bytesRead

[Amnael]

a la ligne 15, si tu remplacer :

Code :

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

x=(int16_t*)realloc(x,init_size*sizeof(int16_t));

par

Code :

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

x=(int16_t*)realloc(x,buffsize*sizeof(int16_t));

et a quoi te sert:

Code :

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

bytesRead

Le problème ne vient plus de là étant donné que j'ai commenté le realloc et que je ne le fais pas.

En outre, je préfère augmenter la taille de 4000 à chaque fois que de mettre au carré la taille...

[Invité]

Tu fais x[cpt] = i et si cpt est >= init_size?

int16_t * x=(int16_t*)malloc(init_size*sizeof(int16_t)); avec init_size = 2000

Et sur ton erreur cpt == 2013

[Bktero]

Que te dit valgrind ?

Tu as des problèmes avec realloc et pourtant tu ne vérifies ni le retour de malloc(), ni celui de realloc(). Ce n'est peut-être pas ça, mais ce n'est néanmoins pas une bonne idée.

Pourquoi réallouer avec init_size et non buffer_size ? Question inverse avec malloc().