[C++] Gestione memoria

[Luciano79]

Ciao a tutti,
ho semplice programma (leggi/scrivi) che va a gestire file di diversi giga.
Mi sono preoccupato di allocare sempre il minimo della memoria necessaria ad ogni stringa che vado ad utilizzare e di liberarne sempre lo spazio allocato. Nonostante questo la memoria continua ad aumentare velocemente fino ad arrivare al limite dei 2gb se scrivo un file di circa 3gb.
Mi rimangono quindi alcuni dubbi sull'utilizzo della memoria.
- Oltre a malloc/free che attualmente utilizzo solo su stringhe o su un paio di piccole typedef, devo prendere altre accorgimenti per mantenere bassa la memoria utilizzata?
- Come faccio a liberare la memoria utilizzata da una variabile che deve restituirmi il valore di una funzione? Per esempio...

char* AsciiToHex(char* sAscii,int Lunghezza)
{
	char *StringaEsadecimale, *Hex;
	int i;

	StringaEsadecimale=(char*)malloc(Lunghezza*2+1);
	StringaEsadecimale[0]='\0';
	Hex=(char*)malloc(sizeof(char)*1);
	for(i=0;i<Lunghezza;i++)
	{
		itoa((unsigned char)sAscii[i],Hex,16);
		strcat(StringaEsadecimale,Hex);
	}
	return StringaEsadecimale;
}

... come a liberare lo spazio allocato a cui punta StringaEsadecimale?

[oregon]

Usa la free sul puntatore restituito.

[Luciano79]

Come sempre grazie Oregon,
la free sul puntatore restituito la faccio già...

TipoRecordHex=AsciiToHex(TipoRecord,3);
....
free(TipoRecordHex);

-----------------------------------
In effetti la tua risposta sembra sensata, dal momento che restituisco il puntatore ad un'area di memoria, dovrei essere in grado di accedere a quest'area ed eventualmente a liberarla.
Eppure ho appena fatto una prova, la funzione AsciiToHex l'ho commentata e portata nel main. In questo modo riesco a liberare la memoria puntata da StringaEsadecimale tramite free. Ora la memoria non sale + (almeno, sale molto meno). Ma non riesco a capire perchè...

[oregon]

Se c'e' un memory leak, bisogna studiare TUTTO il codice per capirne la causa ...

[Luciano79]

Originariamente inviato da oregon
Se c'e' un memory leak, bisogna studiare TUTTO il codice per capirne la causa ...

credo di aver capito, come al solito hai sempre ragione!

Di fatto ho due puntatori:
il primo (TipoRecordHex) nel main,
il secondo è quello che ottengo dalla funzione AsciiToHex.
Faccio le seguenti operazioni nell'ordine illustrato

TipoRecordHex=malloc....
TipoRecordHex=AsciiToHex(...)   // qui è l'errore

char* AsciiToHex(...)
{
  return Puntatore
}
free (TipoRecordHex);

dopo aver assegnato un indirizzo a TipoRecordHex tramite la malloc, vado a perdere quest'indirizzo nel passaggio successivo, rimpiazzandolo con quello ottenuto dalla funzione AsciiToHex. Quindi perdo il vecchio indirizzo di TipoRecordHex e rimane riservata un'area di memoria senza essere nemmeno utilizzata.
Al posto di

TipoRecordHex=AsciiToHex(...);

devo mettere

strcpy(TipoRecordHex,AsciiToHex(....));

ciao e grazie ancora