Memory Leak bei TJsonObject

[Kostas]

Hallo Zusammen,

das Problem vorab: beim Zuweisen von

Delphi-Quellcode:

LResponse

entsteht ein Memory Leak.

Über die folgende Methode rufe ich per indy ein JSON response ab. Der Inhalt ist ein neues Token welches abgelaufen ist ab.
Es funktioniert einwandfrei. Wenn ich jedoch die Anwendung beende, wird ein Memory Leak signalisiert da ich

Delphi-Quellcode:

Application.MainFormOnTaskbar := True;

in der Projektdatei aktiviert habe.
Ich habe Schrittweise alles auskommentiert und die Stelle lokalisieren können die dafür verantwortlich ist. Es ist die Zuweisung von

Delphi-Quellcode:

LResponse

Delphi-Quellcode:

procedure TdmCleverReach.RefreshToken;
var LResponse: TJsonObject;
    LResponseStream: TBytesStream;
    FormData: TidMultiPartFormDataStream;
begin

  Formdata := TIdMultiPartFormDataStream.Create;
  Formdata.AddFormField('grant_type', 'refresh_token');
  Formdata.AddFormField('refresh_token', INIRefreshToken);
  Formdata.AddFormField('client_id', INIClientID);
  Formdata.AddFormField('client_secret', INIClientSecret);
  FormData.Position := 0;

  LResponseStream := TBytesStream.Create;
  LResponse := TJsonObject.Create;
  try
    IdhttpToken.Request.CustomHeaders.Clear;
    IdhttpToken.Request.CustomHeaders.AddPair('Content-Type','multipart/form-data');
    try
      IdhttpToken.Post(INITokenURL,FormData,LResponseStream);
      LResponseStream.Position := 0;

      //Hier scheint der Memory Leak zu entstehen, aber warum?
>>>>      LResponse := TJsonObject.ParseJSONValue(LResponseStream.Bytes, 0, LResponseStream.Size) as TJsonObject;


    LResponseStream.SaveToFile(PrgPath + '\ResponseStream.JSON');

  finally
    Formdata.Free;
    LResponse.Free;
    LResponseStream.Free;
  end;
end;

Was mache ich falsch?

Gruß Kostas

[peterbelow]

Zitat:

Zitat von Kostas (Beitrag 1499627)

Hallo Zusammen,

das Problem vorab: beim Zuweisen von

Delphi-Quellcode:

LResponse

entsteht ein Memory Leak.

Ich habe Schrittweise alles auskommentiert und die Stelle lokalisieren können die dafür verantwortlich ist. Es ist die Zuweisung von

Delphi-Quellcode:

LResponse

Delphi-Quellcode:

procedure TdmCleverReach.RefreshToken;
var LResponse: TJsonObject;
    LResponseStream: TBytesStream;
    FormData: TidMultiPartFormDataStream;
begin

  Formdata := TIdMultiPartFormDataStream.Create;
  Formdata.AddFormField('grant_type', 'refresh_token');
  Formdata.AddFormField('refresh_token', INIRefreshToken);
  Formdata.AddFormField('client_id', INIClientID);
  Formdata.AddFormField('client_secret', INIClientSecret);
  FormData.Position := 0;

  LResponseStream := TBytesStream.Create;
  LResponse := TJsonObject.Create;

Die hier erzeugte Instanz wird nie verwendet und auch nie freigegeben.

Weiter unten überrschreibt dein Kode

Delphi-Quellcode:

   LResponse := TJsonObject.ParseJSONValue(LResponseStream.Bytes, 0, LResponseStream.Size) as TJsonObject;

das Objekt.

[Kostas]

oh, genau das habe ich übersehen dass ParseJSONValue ein neues Object erzeugt.

herzlichen Dank.

[himitsu]

Und jetzt sag blos, dass es dir der Compiler nicht vorher gesagt hat?

[Kostas]

Nein, von Compiler habe ich keine Warnung oder Hinweis bekommen.
Ist das einstellbar in den Optionen?

Gruß Kostas

[Uwe Raabe]

Zitat:

Zitat von himitsu (Beitrag 1499634)

Und jetzt sag blos, dass es dir der Compiler nicht vorher gesagt hat?

Das ist gar nicht mal so sicher. Schließlich könnte ja die zuerst erzeugte Instanz bei einer Exception im finally freigegeben werden und würde somit tatsächlich verwendet.:-D

[Incocnito]

In der Tat ... entweder musst du vor der neu-Zuweisung das "alte" Objekt freigeben, oder du ziehst das Free aus dem unteren finally weiter nach oben.

Delphi-Quellcode:

procedure TdmCleverReach.RefreshToken;
var LResponse: TJsonObject;
    LResponseStream: TBytesStream;
    FormData: TidMultiPartFormDataStream;
begin

  Formdata := TIdMultiPartFormDataStream.Create;
  Formdata.AddFormField('grant_type', 'refresh_token');
  Formdata.AddFormField('refresh_token', INIRefreshToken);
  Formdata.AddFormField('client_id', INIClientID);
  Formdata.AddFormField('client_secret', INIClientSecret);
  FormData.Position := 0;

  LResponseStream := TBytesStream.Create;
  try
    IdhttpToken.Request.CustomHeaders.Clear;
    IdhttpToken.Request.CustomHeaders.AddPair('Content-Type','multipart/form-data');
    try
      IdhttpToken.Post(INITokenURL,FormData,LResponseStream);
      LResponseStream.Position := 0;

      //Hier scheint der Memory Leak zu entstehen, aber warum?
      LResponse := TJsonObject.ParseJSONValue(LResponseStream.Bytes, 0, LResponseStream.Size) as TJsonObject;
      try
        // whatever
      finally
        LResponse.Free;
      end;
      LResponseStream.SaveToFile(PrgPath + '\ResponseStream.JSON');
    finally
      // ...
    end;
  finally
    Formdata.Free;
    LResponseStream.Free;
  end;
end;

[DeddyH]

Delphi-Quellcode:

  LResponse := nil;
  try
    ...
    LResponse := TJsonObject.ParseJSONValue(LResponseStream.Bytes, 0, LResponseStream.Size) as TJsonObject;
  finally
    LResponse.Free;
    ...
  end;

Da genügt dann ein Ressourcenschutzblock.

[himitsu]

Zitat:

Zitat von Uwe Raabe (Beitrag 1499641)

Das ist gar nicht mal so sicher. Schließlich könnte ja die zuerst erzeugte Instanz bei einer Exception im finally freigegeben werden und würde somit tatsächlich verwendet.:-D

Aber nur wenn es knallt,
allerdings sollte bei der zweiten Zuweisung eigentlich eine Warnung kommen, denn ohne knallen würde der erste Wert niemals verwendet.

[Kostas]

LResponse wird vorher NICHT erzeugt!

So habe ich es jetzt umgesetzt.

Delphi-Quellcode:

    try  
      LResponse := TJsonObject.ParseJSONValue(LResponseStream.Bytes, 0, LResponseStream.Size) as TJsonObject;

      if Assigned(LResponse) then
      begin
        LResponse.TryGetValue<String>('access_token', access_token);
        LResponse.TryGetValue<String>('refresh_token', refresh_token);
        LResponse.TryGetValue<integer>('expires_in', expires_in);
        LResponse.Free;
      end;
    except
      // Fehlerbehandlung.
    end;

Vielen lieben Dank und frohe Weihnachten an alle.