Discussion :

[KORTA]

Salut a tous,
J'ai discuté avec quelqu'un à propos de ce code:

Code :

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

1
2
3
4
5
6
7
8
9
 
#include <stdio.h>
#include <stdlib.h>
 
void func(void* mem,int taille)
{
if ( (mem= (void*) realloc(mem,taille+10)) ==NULL)
{printf("Erreur de réallocation\n");}
}

Il paraitrait que celui ci possède "une erreur de conception" à cet endroit

Code :

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

if ( (mem= (void*) realloc(mem,taille+10)) ==NULL)

car cela empecherait la fonction appelante de libérer la mémoire.

Le code appropriée me dit-on serait le suivant:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
#include <stdio.h>
#include <stdlib.h>
 
void func(void* mem,int taille)
{
void* temp;
if ( (temp= (void*) realloc(mem,taille+10)) ==NULL)
{printf("Erreur de réallocation\n");}
else
{mem=temp;}
}

Alors quel est la vérité sur le sujet ?
Merci d'avance a ceux qui fourniront un avis!

[barthelv]

A mon avis, en cas d'erreur, alors oui il peut y avoir un probleme de libération de la mémoire pour la fonciton appelante. En effet, la zone mémoire allouée aura été modifiée, et la séquence non terminée. Si tu utilises un tmp, alors la mémoire pointée par mem n'aura pas été modifiée, et donc restera libérable malgré l'erreur.

Par contre si tout se passe bien, a mon avis il n'y aura pas de problème.

[KORTA]

A mon avis, en cas d'erreur, alors oui il peut y avoir un probleme de libération de la mémoire pour la fonciton appelante. En effet, la zone mémoire allouée aura été modifiée, et la séquence non terminée. Si tu utilises un tmp, alors la mémoire pointée par mem n'aura pas été modifiée, et donc restera libérable malgré l'erreur.

Par contre si tout se passe bien, a mon avis il n'y aura pas de problème.

Ton message m'a fait prendre conscience de la réponse au problème!!Auto-Réponse:
Lorsqu'il se produit une erreur mem prend la valeur de NULL mais l'espace alloué précedemment n'a pas été libéré, il n'est plus possible d' accéder à la zone mémoire (puisque mem a pour valeur NULL) et puisque plus aucune variable ne pointe sur celle ci il n'y a aucun moyen de la libérer (d'ou l'interet du pointeur temp).
Merci pour ta remarque qui m'a "ouvert l'esprit" !!
A bientot