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

Commit 23bb1e82 authored by Yurii Zubrytskyi's avatar Yurii Zubrytskyi Committed by Automerger Merge Worker
Browse files

[res] Better native pointer tracking in Java am: b9ef411d am: 5c9055a0 

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/23987679



Change-Id: I021eb63b40cec0424f0147b257547945f7edcde3
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents e1c1bf45 5c9055a0

core/java/android/content/res/AssetManager.java

+5 −0
Original line number Diff line number Diff line
@@ -528,6 +528,10 @@ public final class AssetManager implements AutoCloseable { 
        if (!mOpen) {

            throw new RuntimeException("AssetManager has been closed");

        }

        // Let's still check if the native object exists, given all the memory corruptions.

        if (mObject == 0) {

            throw new RuntimeException("AssetManager is open but the native object is gone");

        }

    }



    /**
@@ -1153,6 +1157,7 @@ public final class AssetManager implements AutoCloseable { 
    int[] getAttributeResolutionStack(long themePtr, @AttrRes int defStyleAttr,

            @StyleRes int defStyleRes, @StyleRes int xmlStyle) {

        synchronized (this) {

            ensureValidLocked();

            return nativeAttributeResolutionStack(

                    mObject, themePtr, xmlStyle, defStyleAttr, defStyleRes);

        }

core/java/android/content/res/StringBlock.java

+2 −1
Original line number Diff line number Diff line
@@ -62,7 +62,7 @@ public final class StringBlock implements Closeable { 
    private static final String TAG = "AssetManager";

    private static final boolean localLOGV = false;



    private final long mNative;

    private long mNative;   // final, but gets modified when closed

    private final boolean mUseSparse;

    private final boolean mOwnsNative;
@@ -207,6 +207,7 @@ public final class StringBlock implements Closeable { 
                if (mOwnsNative) {

                    nativeDestroy(mNative);

                }

                mNative = 0;

            }

        }

    }

core/java/android/content/res/XmlBlock.java

+3 −1
Original line number Diff line number Diff line
@@ -73,7 +73,9 @@ public final class XmlBlock implements AutoCloseable { 
    private void decOpenCountLocked() {

        mOpenCount--;

        if (mOpenCount == 0) {

            mStrings.close();

            nativeDestroy(mNative);

            mNative = 0;

            if (mAssets != null) {

                mAssets.xmlBlockGone(hashCode());

            }
@@ -621,7 +623,7 @@ public final class XmlBlock implements AutoCloseable { 
    }



    private @Nullable final AssetManager mAssets;

    private final long mNative;

    private long mNative;   // final, but gets reset on close

    /*package*/ final StringBlock mStrings;

    private boolean mOpen = true;

    private int mOpenCount = 1;