Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 1e6019cb authored by Ray Essick's avatar Ray Essick Committed by android-build-merger
Browse files

Fix free-after-use for MediaHTTP

am: dd81e159

Change-Id: I9e0dbd3a91a343de6d607f1f0e00207fdd871f0e
parents 0c01cb74 dd81e159
Loading
Loading
Loading
Loading
+7 −3
Original line number Diff line number Diff line
@@ -58,15 +58,19 @@ status_t MediaHTTP::connect(
        extHeaders.add(String8("User-Agent"), String8(MakeUserAgent().c_str()));
    }

    bool success = mHTTPConnection->connect(uri, &extHeaders);
    mLastURI = uri;
    // reconnect() calls with uri == old mLastURI.c_str(), which gets zapped
    // as part of the above assignment. Ensure no accidental later use.
    uri = NULL;

    bool success = mHTTPConnection->connect(mLastURI.c_str(), &extHeaders);

    mLastHeaders = extHeaders;
    mLastURI = uri;

    mCachedSizeValid = false;

    if (success) {
        AString sanitized = uriDebugString(uri);
        AString sanitized = uriDebugString(mLastURI);
        mName = String8::format("MediaHTTP(%s)", sanitized.c_str());
    }