Array korrekt freigeben

[woodie]

Hallo zusammen ich habe folgenden Code:

Code:

var AString := 'Test;1';
var AArray := AString.Split([';']);

//...

Setlength(AArray,0);
Finalize(AArray);
AArray:=nil;

Warum wird mir bei FastMM5 trotzdem ein Leak angezeigt? Was muss mit dem Array passieren, um den Leak zu unterbinden? oder ist die Split Funktion fehlerhaft?

Code:

A memory block has been leaked. The size is: 78

This block was allocated on 2025-09-23 15:28:03.379 by thread 10096, and the stack trace (return addresses) at the time was:
0096A473 [FastMM5.pas][FastMM5][FastMM_DebugGetMem_GetDebugBlock$qqrio][8039]
0096A53B [FastMM5.pas][FastMM5][FastMM_DebugGetMem$qqri][8075]
009478BA [System.pas][System][GetMem][4965]
0094CF9F [System.pas][System][NewUnicodeString][26083]
0094D1E0 [System.pas][System][UStrFromPWCharLen][26697]
0094E597 [System.pas][System][UStrCopy][32009]
009BADFF [System.SysUtils.pas][System.SysUtils][Sysutils.TStringHelper.InternalSplit][31746]
009BB040 [System.SysUtils.pas][System.SysUtils][Sysutils.TStringHelper.Split][31814]
009BAA52 [System.SysUtils.pas][System.SysUtils][Sysutils.TStringHelper.Split][31580]
01EE8B9C [Test.pas][Test][TTestItem.TestF1][1926]

[Thaddy]

Setlength(AArray,0);
Finalize(AArray); // <---

Dass ist doppel: double free. Beide machen dasselbe.

[Der schöne Günther]

Vielleicht im Debugger noch einen der Strings in Visualizer gehabt? Da kann sowas passieren.

Sowas elementares wie String.Split(..) wird ja kaum kaputt sein.

[TiGü]

Zitat von Der schöne Günther:

Vielleicht im Debugger noch einen der Strings in Visualizer gehabt? Da kann sowas passieren.

Hand hoch, wer auch schon mal einen halben Tag auf der Suche nach derartigen Speicherlecks verbracht hatte und sich immer wunderte, warum der FastMM beim Kollegen (ohne derartiges Untersuchungsobjekt in der Watch List) das Problem nicht anzeigte!

[bernau]

Zitat von Thaddy:

Setlength(AArray,0);
Finalize(AArray); // <---

Dass ist doppel: double free. Beide machen dasselbe.

Warum überhaupt freigeben (beide versionen)? Am Ende der Procedure ist es doch sowiso weg.

Auch würde ich nie auf die Idee kommen "AArray:=nil;" zu verwenden

[himitsu]

Zitat von bernau:

Auch würde ich nie auf die Idee kommen "AArray:=nil;" zu verwenden

Große Arrays, wenn sie nicht mehr gebraucht werden, mitten in der Methode freigeben, um Speicher für was Anderes freizumachen.

[QuickAndDirty]

Ihr verwirrt mich!
Ich dachte immer

Delphi-Quellcode:

Setlength(AArray,0);
AArray:=nil;

sind beides das gleiche.

und von
Finalize(AArray); Habe ich noch nie gehört! Ist das auch das gleiche?

[Der schöne Günther]

Finalize ist eigentlich ein allgemeingültiges "gib das hier vorab schon mal frei" für eine dynamische Variable mit ARC (also wohl alles außer TObject).

https://docwiki.embarcadero.com/Libr...ystem.Finalize

Nehme ich eigentlich immer. Bonus: Der Compiler gibt dir einen Hinweis, wenn das nicht notwendig ist, z.B. wenn du Finalize auf einem Record aufrufst, der gar keine dynamischen Variablen hat.

[himitsu]

Zitat von QuickAndDirty:

sind beides das gleiche.

Im Ergebnis ist es gleich.

:=nil ist einfacher und schnell/optimaler, da hier nur freigegeben wird, während SetLength erst noch bissl prüft und dann was macht und auch anderes machen könnte (z.B. wenn nicht 0 reingegeben)

Finalize wird praktisch automatisch im END; gemacht und man verwendet es, z.B. wenn man mit Pointer rumpfuscht.

Beim Finalize (InitializeArray/FinalizeArray und InitializeRecord/FinalizeRecord, was ein Array mit Länge 1 darstellt) muß man auch teilweise aufpassen,
denn je nach Aufruf kann man sich damit alles zerballern ... z.B. mit den Finalize-Funktionen, welche am Ende nicht den Speicher Nullen
und initialize prüft nicht, ob schon was drin ist (es geht blind davon aus, dass der Speicher neu ist, also noch nie initialisiert oder gar benutzt wurde)

[woodie]

ich habe im Beispiel nur alle 3 Varianten aufgeführt:

Code:

Setlength(AArray,0);
Finalize(AArray);
AArray:=nil;

um 'sicher' zu gehen, normalerweise nutze ich nur Setlength(AArray,0);

Die Frage ist aber immer noch, warum es mir von FastMM5 als Leak angezeigt wird.