Leanback crashes when minified with R8 included in AGP 8.10.0-alpha04 [395489597]

Fixed

Bug

Status Update

No update yet.

Description

st...@gmail.com

created issue #1

Feb 10, 2025 12:20PM

When minifying a project that uses Leanback (View-based) with the version of R8 that is included with AGP 8.10.0-alpha04, I get the crash that is included below.

This only happens when minifyEnabled is set to true and debuggable is set to false.

If I use AGP 8.8.0 then the app doesn't crash.

I have attached a project that can be used to reproduce the issue (it's basically the Leanback template that's in Android Studio with some small modifications).

Crash log:

FATAL EXCEPTION: main
Process: com.example.myapplication, PID: 18548
java.lang.NullPointerException: Attempt to invoke virtual method 'java.lang.Object android.util.SparseArray.get(int)' on a null object reference
	at android.view.View.dispatchRestoreInstanceState(View.java:22249)
	at android.view.ViewGroup.dispatchRestoreInstanceState(ViewGroup.java:4028)
	at android.view.ViewGroup.dispatchRestoreInstanceState(ViewGroup.java:4034)
	at android.view.View.restoreHierarchyState(View.java:22232)
	at kotlin.properties.NotNullVar.addItem(Unknown Source:95)
	at androidx.leanback.widget.SingleRow.appendVisibleItems(Unknown Source:137)
	at androidx.leanback.widget.Grid.appendOneColumnVisibleItems(Unknown Source:11)
	at androidx.leanback.widget.GridLayoutManager.onLayoutChildren(Unknown Source:1061)
	at androidx.recyclerview.widget.RecyclerView.dispatchLayoutStep2(Unknown Source:73)
	at androidx.recyclerview.widget.RecyclerView.dispatchLayout(Unknown Source:71)
	at androidx.recyclerview.widget.RecyclerView.onLayout(Unknown Source:7)
	at android.view.View.layout(View.java:24421)
	at android.view.ViewGroup.layout(ViewGroup.java:6440)
	at androidx.leanback.widget.ScaleFrameLayout.onLayout(Unknown Source:236)
	at android.view.View.layout(View.java:24421)
	at android.view.ViewGroup.layout(ViewGroup.java:6440)
	at android.widget.FrameLayout.layoutChildren(FrameLayout.java:332)
	at android.widget.FrameLayout.onLayout(FrameLayout.java:270)
	at android.view.View.layout(View.java:24421)
	at android.view.ViewGroup.layout(ViewGroup.java:6440)
	at android.widget.FrameLayout.layoutChildren(FrameLayout.java:332)
	at android.widget.FrameLayout.onLayout(FrameLayout.java:270)
	at android.view.View.layout(View.java:24421)
	at android.view.ViewGroup.layout(ViewGroup.java:6440)
	at android.widget.FrameLayout.layoutChildren(FrameLayout.java:332)
	at android.widget.FrameLayout.onLayout(FrameLayout.java:270)
	at android.view.View.layout(View.java:24421)
	at android.view.ViewGroup.layout(ViewGroup.java:6440)
	at android.widget.FrameLayout.layoutChildren(FrameLayout.java:332)
	at android.widget.FrameLayout.onLayout(FrameLayout.java:270)
	at android.view.View.layout(View.java:24421)
	at android.view.ViewGroup.layout(ViewGroup.java:6440)
	at android.widget.FrameLayout.layoutChildren(FrameLayout.java:332)
	at android.widget.FrameLayout.onLayout(FrameLayout.java:270)
	at android.view.View.layout(View.java:24421)
	at android.view.ViewGroup.layout(ViewGroup.java:6440)
	at android.widget.LinearLayout.setChildFrame(LinearLayout.java:1891)
	at android.widget.LinearLayout.layoutVertical(LinearLayout.java:1729)
	at android.widget.LinearLayout.onLayout(LinearLayout.java:1638)
	at android.view.View.layout(View.java:24421)
	at android.view.ViewGroup.layout(ViewGroup.java:6440)
	at android.widget.FrameLayout.layoutChildren(FrameLayout.java:332)
	at android.widget.FrameLayout.onLayout(FrameLayout.java:270)
	at com.android.internal.policy.DecorView.onLayout(DecorView.java:789)
	at android.view.View.layout(View.java:24421)
	at android.view.ViewGroup.layout(ViewGroup.java:6440)
	at android.view.ViewRootImpl.performLayout(ViewRootImpl.java:4258)
	at android.view.ViewRootImpl.performTraversals(ViewRootImpl.java:3583)
	at android.view.ViewRootImpl.doTraversal(ViewRootImpl.java:2465)
	at android.view.ViewRootImpl$TraversalRunnable.run(ViewRootImpl.java:9305)
	at android.view.Choreographer$CallbackRecord.run(Choreographer.java:1339)
	at android.view.Choreographer$CallbackRecord.run(Choreographer.java:1348)
	at android.view.Choreographer.doCallbacks(Choreographer.java:952)
	at android.view.Choreographer.doFrame(Choreographer.java:882)
	at android.view.Choreographer$FrameDisplayEventReceiver.run(Choreographer.java:1322)
	at android.os.Handler.handleCallback(Handler.java:958)
	at android.os.Handler.dispatchMessage(Handler.java:99)
	at android.os.Looper.loopOnce(Looper.java:205)
	at android.os.Looper.loop(Looper.java:294)
	at android.app.ActivityThread.main(ActivityThread.java:8177)
	at java.lang.reflect.Method.invoke(Native Method)
	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:552)
	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:971)

LeanbackCrash.zip

114 KB

Download

Comments

sg...@google.com <sg...@google.com> Feb 10, 2025 12:26PM

Reassigned to sg...@google.com.

sg...@google.com <sg...@google.com> #2Feb 10, 2025 04:08PM

Thank you for reporting this issue. For us to further investigate this issue, please provide the following additional information:

Are you reporting this as a Developer or User?
Please confirm if you are using any third party application as a launcher?
Could you please provide a screen record of the issue for us to further investigate?

Note: Please upload the files to google drive and share the folder to android-bugreport@google.com, then share the link here.

sg...@google.com <sg...@google.com> #3Feb 11, 2025 04:28PM

I am a User
No third party launcher using Default pixel launcher
I cannot for now recreate the bug but will enable screen record and upload when caught

sg...@google.com <sg...@google.com> #4Feb 12, 2025 09:37AM

Thank you for your response.

Once you are able to reproduce this bug, please upload the screen record of the issue.

sg...@google.com <sg...@google.com> #5Feb 12, 2025 10:32AM

Please upgrade to the latest release from the link Beta2.1.

If the issue still persists, please capture the new bug report along with the screen record and share it here.

ap...@google.com <ap...@google.com> #6Feb 12, 2025 11:35AM

Please provide the requested information to proceed further. Unfortunately the issue will be closed within 7 days if there is no further update.

ap...@google.com <ap...@google.com> #7Feb 12, 2025 12:27PM

Marked as fixed.

Project: r8
Branch: main
Author: Søren Gjesse <sgjesse@google.com>
Link: https://r8-review.googlesource.com/101480

Don't use aliased values when comparing values for basic block subsumption

Expand for full commit details

Don't use aliased values when comparing values for basic block subsumption 
 
Aliased values can be defined by semantically different assume instructions  and then the resulting value chosen can have 
information which does not correctly represent both values. 
 
Fixes: b/395489597 
Change-Id: Ieaeb4fbf8d49cdc60d2912791cc2b701d42d2b34

Files:

M src/main/java/com/android/tools/r8/ir/analysis/equivalence/BasicBlockBehavioralSubsumption.java
M src/test/java/com/android/tools/r8/deadcode/DifferentAssumptionsOnReturnValuesTest.java

Hash: 94489e15f0c4d1477fe0b16912f6f3bfa4957f09
Date: Wed Feb 12 13:21:29 2025

ap...@google.com <ap...@google.com> #8Feb 12, 2025 01:59PM

Project: r8
Branch: 8.9
Author: Søren Gjesse <sgjesse@google.com>
Link: https://r8-review.googlesource.com/101501

Version 8.9.27

Expand for full commit details

Version 8.9.27 
 
R=christofferqa@google.com 
 
Bug: b/395489597 
Change-Id: I4a2483c145c93f9370e3af2c59bd3b2be487441d

Files:

M src/main/java/com/android/tools/r8/Version.java

Hash: 1d6cbecc9afd9991f4db5f517bdeba1da9347093
Date: Wed Feb 12 13:57:20 2025

ap...@google.com <ap...@google.com> #9Feb 12, 2025 01:59PM

Project: r8
Branch: 8.9
Author: Søren Gjesse <sgjesse@google.com>
Link: https://r8-review.googlesource.com/101483

Don't use aliased values when comparing values for basic block subsumption

Expand for full commit details

Don't use aliased values when comparing values for basic block subsumption 
 
Aliased values can be defined by semantically different assume instructions  and then the resulting value chosen can have 
information which does not correctly represent both values. 
 
R=christofferqa@google.com 
 
Fixes: b/395489597 
Change-Id: Ieaeb4fbf8d49cdc60d2912791cc2b701d42d2b34

Files:

M src/main/java/com/android/tools/r8/ir/analysis/equivalence/BasicBlockBehavioralSubsumption.java
M src/test/java/com/android/tools/r8/deadcode/DifferentAssumptionsOnReturnValuesTest.java

Hash: 3f8eebe693e4d73cfbdaf5e5761733ec47d3b775
Date: Wed Feb 12 13:57:12 2025

ap...@google.com <ap...@google.com> #10Feb 12, 2025 01:59PM

Project: r8
Branch: 8.9
Author: Søren Gjesse <sgjesse@google.com>
Link: https://r8-review.googlesource.com/101482

Add reproduction of b/395489597

Expand for full commit details

Add reproduction of b/395489597 
 
R=christofferqa@google.com 
 
Bug: b/395489597 
Change-Id: If769fae5db93ee8e2cfc1cf9feeddd847b00832b

Files:

A src/test/java/com/android/tools/r8/deadcode/DifferentAssumptionsOnReturnValuesTest.java

Hash: 49cade9501045641cf63bca54ab15728ea95f16d
Date: Wed Feb 12 13:57:05 2025

ap...@google.com <ap...@google.com> #11Feb 12, 2025 03:28PM

Project: r8
Branch: 8.8
Author: Søren Gjesse <sgjesse@google.com>
Link: https://r8-review.googlesource.com/101502

Don't use aliased values when comparing values for basic block subsumption

Expand for full commit details

Don't use aliased values when comparing values for basic block subsumption 
 
Aliased values can be defined by semantically different assume instructions  and then the resulting value chosen can have 
information which does not correctly represent both values. 
 
R=christofferqa@google.com 
 
Fixes: b/395489597 
Change-Id: Ieaeb4fbf8d49cdc60d2912791cc2b701d42d2b34

Files:

M src/main/java/com/android/tools/r8/ir/analysis/equivalence/BasicBlockBehavioralSubsumption.java
M src/test/java/com/android/tools/r8/deadcode/DifferentAssumptionsOnReturnValuesTest.java

Hash: e7d78e09612fe9f67730cddc22cf03798a1ddd38
Date: Wed Feb 12 15:02:14 2025

ap...@google.com <ap...@google.com> #12Feb 12, 2025 03:28PM

Project: r8
Branch: 8.8
Author: Søren Gjesse <sgjesse@google.com>
Link: https://r8-review.googlesource.com/101484

Add reproduction of b/395489597

Expand for full commit details

Add reproduction of b/395489597 
 
R=christofferqa@google.com 
 
Bug: b/395489597 
Change-Id: If769fae5db93ee8e2cfc1cf9feeddd847b00832b

Files:

A src/test/java/com/android/tools/r8/deadcode/DifferentAssumptionsOnReturnValuesTest.java

Hash: 7697806c5d5f04330289dca95a11b755ed05e691
Date: Wed Feb 12 15:01:41 2025

ap...@google.com <ap...@google.com> #13Feb 12, 2025 03:29PM

Project: r8
Branch: 8.8
Author: Søren Gjesse <sgjesse@google.com>
Link: https://r8-review.googlesource.com/101503

Version 8.8.34

Expand for full commit details

Version 8.8.34 
 
R=christofferqa@google.com 
 
Bug: b/395489597 
Change-Id: I2475873aa690ee588f913f12d21dbf16e07c0468

Files:

M src/main/java/com/android/tools/r8/Version.java

Hash: e4c4b8c78485b4b879001e60600d9ef799083345
Date: Wed Feb 12 15:02:22 2025

ap...@google.com <ap...@google.com> #14Feb 12, 2025 05:28PM

Project: r8
Branch: 8.7
Author: Søren Gjesse <sgjesse@google.com>
Link: https://r8-review.googlesource.com/101485

Version 8.7.34

Expand for full commit details

Version 8.7.34 
 
R=christofferqa@google.com 
 
Bug: b/395489597 
Change-Id: I4946bdfc811a0211119532ed2b23f737ff5668c3

Files:

M src/main/java/com/android/tools/r8/Version.java

Hash: 44d960a65196e49153711487c3f24793ded7c057
Date: Wed Feb 12 15:03:02 2025

ap...@google.com <ap...@google.com> #15Feb 12, 2025 05:28PM

Project: r8
Branch: 8.7
Author: Søren Gjesse <sgjesse@google.com>
Link: https://r8-review.googlesource.com/101505

Don't use aliased values when comparing values for basic block subsumption

Expand for full commit details

Don't use aliased values when comparing values for basic block subsumption 
 
Aliased values can be defined by semantically different assume instructions  and then the resulting value chosen can have 
information which does not correctly represent both values. 
 
R=christofferqa@google.com 
 
Fixes: b/395489597 
Change-Id: Ieaeb4fbf8d49cdc60d2912791cc2b701d42d2b34

Files:

M src/main/java/com/android/tools/r8/ir/analysis/equivalence/BasicBlockBehavioralSubsumption.java
M src/test/java/com/android/tools/r8/deadcode/DifferentAssumptionsOnReturnValuesTest.java

Hash: 57e30e3ab2df1593c339d6fafa4b9c5162dad653
Date: Wed Feb 12 15:02:55 2025

ap...@google.com <ap...@google.com> #16Feb 12, 2025 05:28PM

Project: r8
Branch: 8.7
Author: Søren Gjesse <sgjesse@google.com>
Link: https://r8-review.googlesource.com/101504

Add reproduction of b/395489597

Expand for full commit details

Add reproduction of b/395489597 
 
R=christofferqa@google.com 
 
Bug: b/395489597 
Change-Id: If769fae5db93ee8e2cfc1cf9feeddd847b00832b

Files:

A src/test/java/com/android/tools/r8/deadcode/DifferentAssumptionsOnReturnValuesTest.java

Hash: ede31119cc4b72e116e3ddbf3bd6d246ee7baff8
Date: Wed Feb 12 15:02:48 2025

sg...@google.com <sg...@google.com> #17Feb 17, 2025 02:56PM

This fix will make it into one of the next AGP alpha versions (most likely 8.10.0-alpha06 or 8.10.0-alpha06). Until that one can use R8 version 8.10.8-dev, see Replacing R8 in Android Gradle plugin for details.

an...@google.com <an...@google.com> #18Feb 18, 2025 01:59PM

Thank you for your patience while our engineering team worked to resolve this issue. A fix for this issue is now available in:

Android Studio Meerkat | 2024.3.1 RC 2
Android Gradle Plugin 8.9.0-rc02

We encourage you to try the latest update.

If you notice further issues or have questions, please file a new bug report.

Thank you for taking the time to submit feedback — we really appreciate it!

an...@google.com <an...@google.com> #19Feb 26, 2025 05:51PM

The fixes for this issue are now also available in:

Android Studio Ladybug Feature Drop | 2024.2.2 Patch 2
Android Gradle Plugin 8.8.2

We encourage you to try the latest update.

If you notice further issues or have questions, please file a new bug report.

Issue 395489597

Description

Issue summary

Comments

sg...@google.com <sg...@google.com> Feb 10, 2025 12:26PM

sg...@google.com <sg...@google.com> #2Feb 10, 2025 04:08PM

sg...@google.com <sg...@google.com> #3Feb 11, 2025 04:28PM

sg...@google.com <sg...@google.com> #4Feb 12, 2025 09:37AM

sg...@google.com <sg...@google.com> #5Feb 12, 2025 10:32AM

ap...@google.com <ap...@google.com> #6Feb 12, 2025 11:35AM

ap...@google.com <ap...@google.com> #7Feb 12, 2025 12:27PM

ap...@google.com <ap...@google.com> #8Feb 12, 2025 01:59PM

ap...@google.com <ap...@google.com> #9Feb 12, 2025 01:59PM

ap...@google.com <ap...@google.com> #10Feb 12, 2025 01:59PM

ap...@google.com <ap...@google.com> #11Feb 12, 2025 03:28PM

ap...@google.com <ap...@google.com> #12Feb 12, 2025 03:28PM

ap...@google.com <ap...@google.com> #13Feb 12, 2025 03:29PM

ap...@google.com <ap...@google.com> #14Feb 12, 2025 05:28PM

ap...@google.com <ap...@google.com> #15Feb 12, 2025 05:28PM

ap...@google.com <ap...@google.com> #16Feb 12, 2025 05:28PM

sg...@google.com <sg...@google.com> #17Feb 17, 2025 02:56PM

an...@google.com <an...@google.com> #18Feb 18, 2025 01:59PM

an...@google.com <an...@google.com> #19Feb 26, 2025 05:51PM

Add comment

Issue metadata