Fuite mémoire ?

Version imprimable

Fuite mémoire ?

Bonjour,

j'ai écrit il y a quelques années un petit programme en C/C++ consititué de plusieurs fonctions. L'une d'entre elles a pour but de retourner un pointeur sur le premier élément d'un tableau d'entier. Est ce que je ne génère pas une fuite mémoire ? :( Si oui, quel est l'impact sur les serveurs où il s'exécute de manière infinie ? :(
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 int *DateHeure() { int Date[6]; date Date_avance; struct time Heure; getdate(&Date_avance); gettime(&Heure); Date[0]=int(Date_avance.da_day); Date[1]=int(Date_avance.da_mon); Date[2]=int(Date_avance.da_year); Date[3]=int(Heure.ti_hour); Date[4]=int(Heure.ti_min); Date[5]=int(Heure.ti_sec); return &Date[0]; }
Je suis novice en programmation et de l'aide est bienvenue.

par avance Merci

12/02/2009, 13h24
souviron34

c'est surtout que tu généres un comportement indéterminé..... :aie:

Date est une variable locale..

Dont tu retournes l'adresse, soit..

MAIS cette varaible n'a plus de sens dès qu'on est sorti de la fonction...
12/02/2009, 14h36
catdor

Fuite mémoire ?

Si je comprends bien, je récupère un pointeur sur une variable locale qui est désalloué lorsque je sors de la fonction et qui peut être réalloué pour une autre utilsation à n'importe quel moment. Est ce que je suis dans le vrai ?

Lorsque je récupère l'adresse de mon tableau (retourné par ma fonction) dans un pointeur d'entier int*Date, je parviens à lire son contenu mais je pourrais très bien retourner quelque chose d'autre.

Puis je faire un accès dans une zone non autorisés dédiée à un autre processus (Oracle par exemple shm .....) ?

Merci de ta réponse
12/02/2009, 14h37
Furikawari

Pour être tout à fait clair :

1- il n'y a pas de fuite mémoire

2. Ton tableau Date est une variable locale à la fonction, ce qui signifie que l'espace mémoire occupée par cette variable est libéré dès que la dite fonction se termine.
Or tu retournes l'adresse de ce tableau. La mémoire étant libérée elle peut être réutilisée à n'importe quel moment par le programme (d'où le caractère non déterministe souligné par Souviron). Du coup il y a de fortes chances pour que les valeurs que tu lises soient fausses...

[Edit] Tu as répondu entre temps :aie:

Dans tout bon OS les processus tournent dans des espaces mémoires séparés et tu ne peux pas allouer dans un espace qui n'appartient pas à ton processus.
12/02/2009, 14h54
souviron34

une manière plus adaptée serait :

Code:

void DateHeure(int *Date)

où Date est declaré (dimensionné) dans la routine appellante

OU

Code:

int *DateHeure()

où Date est déclaré comme pointeur dans la routine, puis alloué (via malloc ou calloc), et où tu retournes le pointeur.

Cependant, avec cette manière, il te faut libérer la mémoire dans la routine appellante une fois que tu n'en as plus besoin.
12/02/2009, 14h58
millie

Citation:

Envoyé par Furikawari

Dans tout bon OS les processus tournent dans des espaces mémoires séparés et tu ne peux pas allouer dans un espace qui n'appartient pas à ton processus.

Par contre, il est possible d'allouer de la mémoire partagée entre plusieurs processus. Mais c'est souvent casse gueule et c'est rarement vraiment utile.

Sinon, je plussoie Furikawari, sur les ordinateurs "normaux", il y a un composant qui s'appelle la MMU qui permet au processus de ne pouvoir lire que dans sa zone de mémoire virtuelle
12/02/2009, 15h53
alex_pi

Citation:

Envoyé par catdor

j'ai écrit il y a quelques années un petit programme en C/C++

Ce serait plus un message pour le forum C/C++ alors non ? Parce que les "langages en général" ne posent absolument pas ce genre de problème, c'est assez spécifique à ces deux là.