Fixed

Bug

Status Update

No update yet.

Description

su...@twofortyfouram.com

created issue #1

Aug 27, 2022 05:16PM

OVERVIEW

Androidx ProfileInstaller library has an implementation bug that can trigger a NullPointerException. Specifically, sending an Intent action androidx.profileinstaller.action.SKIP_FILE without any extras triggers the issue.

STEPS TO REPRODUCE

Install an app using ProfileInstaller
Replacing ${PACKAGE_NAME} with the correct application package name, run the following ADB command: adb shell am broadcast -a androidx.profileinstaller.action.SKIP_FILE -n ${PACKAGE_NAME}/androidx.profileinstaller.ProfileInstallReceiver

RESULTS

Actual: App crashes

Caused by java.lang.NullPointerException: Attempt to invoke virtual method 'java.lang.String android.os.Bundle.getString(java.lang.String)' on a null object reference
       at androidx.profileinstaller.ProfileInstallReceiver.onReceive(ProfileInstallReceiver.java:75)
       at android.app.ActivityThread.handleReceiver(ActivityThread.java:3187)
       at android.app.ActivityThread.-wrap17(ActivityThread.java)
       at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1672)
       at android.os.Handler.dispatchMessage(Handler.java:106)
       at android.os.Looper.loop(Looper.java:164)
       at android.app.ActivityThread.main(ActivityThread.java:6494)
       at java.lang.reflect.Method.invoke(Method.java)
       at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:438)
       at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:807)

Expected: App shouldn't crash

NOTES

The issue is caused by the Intent extras being null. A null check of the extras is needed.

See https://cs.android.com/androidx/platform/frameworks/support/+/androidx-main:profileinstaller/profileinstaller/src/main/java/androidx/profileinstaller/ProfileInstallReceiver.java;l=74

Component used: ProfileInstaller Version used: 1.2 Devices/Android versions reproduced on: Nexus 5 Android 7, Pixel 6 Android 13

Comments

ap...@google.com <ap...@google.com> #2Aug 30, 2022 02:29AM

Marked as fixed.

> The Transition Kotlin extensions target the Framework Transition class which is a bit confusing

All extensions in 'core-ktx' target either 'core' or the framework types. This is working as expected. It sounds like this is a feature request for a transition-ktx with equivalent extensions for the androidx.transition types.

> as most framework classes are being deprecated in favor of their AndroidX counterparts, Fragments for example.

"Most"? This *dramatically* overstates what is actually happening. Fragments (and loaders) were added to the framework in a time where the Android team did not have external libraries for shipping features. Transition was added 5 years later.

na...@google.com <na...@google.com> #3Oct 24, 2022 01:37PM

Pardon my misunderstanding, I was under the belief that where possible, new features that could be developed outside the framework, would be under Androidx. Also it's a common refrain to use the Androidx libraries over the framework libraries where possible, as those ship update independently of platform releases. I wasn't attempting to be dramatic.

If the Transition api is unlikely to change, there is no need for a transition-ktx for androidx.transition types. If developers should use Androidx transitions over framework transitions where possible however, then it would be much appreciated.

I am well aware of the limitations that existed when Fragments and Loaders were created, and when the Transition api was introduced.

jk...@gmail.com <jk...@gmail.com> #4Oct 29, 2022 02:23AM

Project: platform/frameworks/support
Branch: androidx-master-dev

commit ae42e311ecfbac18cedc90e489d1fdde956a4557
Author: Jake Wharton <jakew@google.com>
Date: Fri Nov 22 11:13:30 2019

Introduce transition-ktx for standalone transition library

These extensions are the same as androidx.core.transition for the platform types, just adapted to the standalone library.

Bug: 138870873
Test: gw :transition:transition-ktx:build :transition:transition-ktx:connectedCheck
Change-Id: Ie80022cc60ed1f482b7593db5ef1ac1c5ea33d8d

M buildSrc/src/main/kotlin/androidx/build/PublishDocsRules.kt
M jetifier/jetifier/migration.config
M settings.gradle
A transition/transition-ktx/OWNERS
A transition/transition-ktx/api/1.4.0-alpha01.txt
A transition/transition-ktx/api/current.txt
A transition/transition-ktx/api/public_plus_experimental_1.4.0-alpha01.txt
A transition/transition-ktx/api/public_plus_experimental_current.txt
A transition/transition-ktx/api/res-1.4.0-alpha01.txt
A transition/transition-ktx/api/restricted_1.4.0-alpha01.txt
A transition/transition-ktx/api/restricted_current.txt
A transition/transition-ktx/build.gradle
A transition/transition-ktx/src/androidTest/AndroidManifest.xml
A transition/transition-ktx/src/androidTest/java/androidx/transition/TestActivity.kt
A transition/transition-ktx/src/androidTest/java/androidx/transition/TransitionTest.kt
A transition/transition-ktx/src/androidTest/res/layout/test_activity.xml
A transition/transition-ktx/src/main/AndroidManifest.xml
A transition/transition-ktx/src/main/java/androidx/transition/Transition.kt

https://android-review.googlesource.com/1172715

ia...@gmail.com <ia...@gmail.com> #5Mar 14, 2023 02:16AM

Is there a plan to backport this to ProfileInstaller 1.2.X?

It seems that this commit added ProfileInstaller to libraries such as fragment, thus bringing in the version of ProfileInstaller that still contains this bug.

This is causing crashes in my app.

mi...@gmail.com <mi...@gmail.com> #6Mar 14, 2023 01:05PM

I have the same bug on Nexus 5x with Android 8.1.0. The dependency androidx.compose.ui:ui contains androidx.profileinstaller:profileinstaller:1.2.1 with this bug.

ap...@google.com <ap...@google.com> #7Mar 15, 2023 08:05PM

Project: platform/frameworks/support
Branch: androidx-main

commit 3fc05b928c858af7bb318cc5f36a97715c6663a8
Author: Chris Craik <ccraik@google.com>
Date: Wed Mar 15 11:25:45 2023

Bump fragment's profileinstaller dependency to 1.3

Bug: 243851384
Test: None

Change-Id: I57274bd56d10fa38aa363419eb6b5cc30a3e967d

M fragment/fragment/build.gradle

https://android-review.googlesource.com/2490598

ap...@google.com <ap...@google.com> #8Mar 15, 2023 08:46PM

Project: platform/frameworks/support
Branch: androidx-main

commit 47f41950723415b5fd857530298ddd6a7aac952a
Author: Chris Craik <ccraik@google.com>
Date: Wed Mar 15 11:22:56 2023

Bump compose's profileinstaller dependency to 1.3

Test: None
Bug: 243851384
Change-Id: I9dff371686203fd18cb8dc5680c79d8c16989d96

M compose/ui/ui/build.gradle

https://android-review.googlesource.com/2491997

ap...@google.com <ap...@google.com> #9Mar 15, 2023 09:09PM

Project: platform/frameworks/support
Branch: androidx-main

commit 512f0c047e135deb054d7764677f981ffdbb1817
Author: Chris Craik <ccraik@google.com>
Date: Wed Mar 15 11:19:01 2023

Bump profileinstaller dependency to 1.3

Test: None
Bug: 243851384

Change-Id: I373f22c9285abf024c051dc22ff2183d743b7360

M appcompat/appcompat/build.gradle
M benchmark/benchmark-macro/build.gradle
M constraintlayout/constraintlayout/build.gradle
M navigation/navigation-common/build.gradle
M recyclerview/recyclerview/build.gradle
M tv/tv-foundation/build.gradle
M tv/tv-material/build.gradle
M wear/compose/compose-foundation/build.gradle
M wear/compose/compose-material-core/build.gradle
M wear/compose/compose-material/build.gradle
M wear/compose/compose-material3/build.gradle
M wear/compose/compose-navigation/build.gradle

https://android-review.googlesource.com/2491996

ap...@google.com <ap...@google.com> #10Mar 15, 2023 09:10PM

Project: platform/frameworks/support
Branch: androidx-main

commit a8729785427262edc8c694584fb5cbaf323bfb80
Author: Chris Craik <ccraik@google.com>
Date: Wed Mar 15 11:24:39 2023

Bump activity's profileinstaller dependency to 1.3

Bug: 243851384
Test: None

Change-Id: Idc9af745ac5774491215980d26b0169b39195f30

M activity/activity/build.gradle

https://android-review.googlesource.com/2491995

ap...@google.com <ap...@google.com> #11Mar 16, 2023 05:13PM

Project: platform/frameworks/support
Branch: androidx-main

commit 4bb158a35205ce4c0480d89ac325bf8ac86d3526
Author: Chris Craik <ccraik@google.com>
Date: Wed Mar 15 11:20:54 2023

Bump lifecycle's profileinstaller dependency to 1.3

Test: ./gradlew room:integ-test:room-testapp:cC -P android.testInstrumentationRunnerArguments.class=androidx.room.integration.testapp.test.QueryExecutorTest # API 17 emulator
Bug: 243851384

QueryExecutorTest was failing on API 17 due to non-native multidex
issues. Wasn't able to workaround in multidex proguard rules because
issues also occurred in the androidTest, which is also multidex.

Change-Id: I792343d042365e2db5940367a1e9fad377aedc26

M lifecycle/lifecycle-runtime/build.gradle
M room/integration-tests/testapp/build.gradle
D room/integration-tests/testapp/multidex-config.txt
A room/integration-tests/testapp/

multidex-rules.pro

https://android-review.googlesource.com/2490599

pr...@google.com <pr...@google.com> #12Mar 22, 2023 08:22PM

The following release(s) address this bug.It is possible this bug has only been partially addressed:

androidx.activity:activity:1.7.0
androidx.benchmark:benchmark-macro:1.2.0-alpha12
androidx.compose.ui:ui:1.4.0
androidx.constraintlayout:constraintlayout:2.2.0-alpha09
androidx.fragment:fragment:1.6.0-alpha08
androidx.lifecycle:lifecycle-runtime:2.6.1
androidx.navigation:navigation-common:2.6.0-alpha08
androidx.tv:tv-foundation:1.0.0-alpha05
androidx.tv:tv-material:1.0.0-alpha05
androidx.wear.compose:compose-foundation:1.2.0-alpha07
androidx.wear.compose:compose-material:1.2.0-alpha07
androidx.wear.compose:compose-material-core:1.2.0-alpha07
androidx.wear.compose:compose-navigation:1.2.0-alpha07

na...@google.com <na...@google.com> #13Jul 26, 2023 05:21PM

The following release(s) address this bug.It is possible this bug has only been partially addressed:

androidx.appcompat:appcompat:1.7.0-alpha03
androidx.lifecycle:lifecycle-runtime:2.7.0-alpha01

pr...@google.com <pr...@google.com> #14Oct 18, 2023 08:42PM

The following release(s) address this bug.It is possible this bug has only been partially addressed:

androidx.recyclerview:recyclerview:1.4.0-alpha01

ro...@gmail.com <ro...@gmail.com> #15Sep 10, 2024 08:56AM

Profile Installer

IssueTracker

NPE in ProfileInstallReceiver

Status Update

Description

OVERVIEW

STEPS TO REPRODUCE

RESULTS

NOTES

Comments

ap...@google.com <ap...@google.com> #2Aug 30, 2022 02:29AM

na...@google.com <na...@google.com> #3Oct 24, 2022 01:37PM

jk...@gmail.com <jk...@gmail.com> #4Oct 29, 2022 02:23AM

ia...@gmail.com <ia...@gmail.com> #5Mar 14, 2023 02:16AM

mi...@gmail.com <mi...@gmail.com> #6Mar 14, 2023 01:05PM

ap...@google.com <ap...@google.com> #7Mar 15, 2023 08:05PM

ap...@google.com <ap...@google.com> #8Mar 15, 2023 08:46PM

ap...@google.com <ap...@google.com> #9Mar 15, 2023 09:09PM

ap...@google.com <ap...@google.com> #10Mar 15, 2023 09:10PM

ap...@google.com <ap...@google.com> #11Mar 16, 2023 05:13PM

pr...@google.com <pr...@google.com> #12Mar 22, 2023 08:22PM

na...@google.com <na...@google.com> #13Jul 26, 2023 05:21PM

pr...@google.com <pr...@google.com> #14Oct 18, 2023 08:42PM

ro...@gmail.com <ro...@gmail.com> #15Sep 10, 2024 08:56AM

Issue 243851384

Description

OVERVIEW

STEPS TO REPRODUCE

RESULTS

NOTES

Issue summary

Comments

ap...@google.com <ap...@google.com> #2Aug 30, 2022 02:29AM

na...@google.com <na...@google.com> #3Oct 24, 2022 01:37PM

jk...@gmail.com <jk...@gmail.com> #4Oct 29, 2022 02:23AM

ia...@gmail.com <ia...@gmail.com> #5Mar 14, 2023 02:16AM

mi...@gmail.com <mi...@gmail.com> #6Mar 14, 2023 01:05PM

ap...@google.com <ap...@google.com> #7Mar 15, 2023 08:05PM

ap...@google.com <ap...@google.com> #8Mar 15, 2023 08:46PM

ap...@google.com <ap...@google.com> #9Mar 15, 2023 09:09PM

ap...@google.com <ap...@google.com> #10Mar 15, 2023 09:10PM

ap...@google.com <ap...@google.com> #11Mar 16, 2023 05:13PM

pr...@google.com <pr...@google.com> #12Mar 22, 2023 08:22PM

na...@google.com <na...@google.com> #13Jul 26, 2023 05:21PM

pr...@google.com <pr...@google.com> #14Oct 18, 2023 08:42PM

ro...@gmail.com <ro...@gmail.com> #15Sep 10, 2024 08:56AM

Add comment

Issue metadata