Discussion :

[totoscill]

Bonjour,

J'ai un problème avec ma réallocation de mémoire dans mon programme. Je veux réalloué de la mémoire si nécessaire pour un tableau deux dimensions. Mon tableau est alloué initialement avec une valeur de 5.
Lorsque j'exécute mon programme, j'ai l'impression que ma deuxième dimension n'est pas alloué:

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
 
TabTimePalCh = realloc(TabTimePalCh,NbCharge);
 
if ( TabTimePalCh == NULL )
	{
	MessagePopup("Erreur!!!", "Erreur lors de l'allocation de mémoire!"); // on affiche un message
    	QuitUserInterface (0);	// on sort du programme en renvoyant un code d'erreur
	MessagePopup("FATAL ERROR", "Erreur 001: impossible d'allouer la mémoire.");
	}
 
for(i=5;i<NbPal;i++)
	{
	TabTimePalCh[i] = realloc(TabTimePalCh[i],NbPal);
 
	if ( TabTimePalCh[i] == NULL ) 
		{
        		MessagePopup("Erreur!!!", "Erreur lors de l'allocation de mémoire!"); // on affiche un message
        		for(j=0;j<i;j++) 
			{
          		  	free(TabTimePalCh[j]);
			}
        		free(TabTimePalCh);
		QuitUserInterface (0);	// on sort du programme en renvoyant un code d'erreur
		MessagePopup("FATAL ERROR", "Erreur 001: impossible d'allouer la mémoire.");// on sort du programme en renvoyant un code d'erreur
		}
}

Pourriez vous me dire ce qui ne va pas?
Je vous remercie.

[diogene]

Code :

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

1
2
3
for(i=5;i<NbPal;i++)
	{
	TabTimePalCh[i] = realloc(TabTimePalCh[i],NbPal);

Les TabTimePalCh[i] sont nouvellement alloués par le

Code :

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

TabTimePalCh = realloc(TabTimePalCh,NbCharge);

Leur contenu est quelconque. Tu ne peux pas faire une réallocation sur cette adresse.

Code :

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

1
2
3
for(i=5;i<NbPal;i++)
	{
	TabTimePalCh[i] = realloc(NULL,NbPal);

[totoscill]

Ok, mais il n'y a vraiment aucun moyen d'allouer la memoire sur cette dimension?

[Vincent Rogier]

petite remarque au passage...

Réaffecter le retour d'un realloc() au pointeur qui est passé en paramètre est source de bug potentiel.

Car si la réallocation échoue, tu perds ainsi la référence au bloc mémoire que tu ne peux donc plus libérer !

[diogene]

Ok, mais il n'y a vraiment aucun moyen d'allouer la memoire sur cette dimension?

Si, mais pas une réallocation :

Code :

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

1
2
3
4
for(i=5;i<NbPal;i++)
	{
	TabTimePalCh[i] = realloc(NULL,NbPal);
// ou si on préfère TabTimePalCh[i] = malloc(NbPal);

Par contre, si besoin, tu peux faire une réallocation pour i<5

[Médinoc]

petite remarque au passage...

Réaffecter le retour d'un realloc() au pointeur qui est passé en paramètre est source de bug potentiel.

Car si la réallocation échoue, tu perds ainsi la référence au bloc mémoire que tu ne peux donc plus libérer !

En effet.
Les fonctions dans l'autre thread de totoscill servent justement à contourner ce bug. C'est bizarre qu'elles soient absentes de ce code-ci...