Memory Leak bei TJsonObject

[Uwe Raabe]

Zitat von himitsu:

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.

[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 von Uwe Raabe:

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.

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.