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.

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

commit b90079595f33f58fece04026a97faa0d243acdb1
Author: Yuichi Araki <yaraki@google.com>
Date: Wed Sep 18 16:55:49 2019

Change the way to detect mismatch between POJO and query

This fixes cursor mismatch warnings with expandProjection.

Bug: 140759491
Test: QueryMethodProcessorTest
Change-Id: I7659002e5e0d1ef60fc1af2a625c0c36da0664d8

M room/compiler/src/main/kotlin/androidx/room/processor/QueryMethodProcessor.kt
M room/compiler/src/main/kotlin/androidx/room/solver/TypeAdapterStore.kt
M room/compiler/src/main/kotlin/androidx/room/solver/query/result/PojoRowAdapter.kt
M room/compiler/src/test/kotlin/androidx/room/processor/QueryMethodProcessorTest.kt
M room/compiler/src/test/kotlin/androidx/room/testing/TestProcessor.kt

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

https://goto.google.com/android-sha1/b90079595f33f58fece04026a97faa0d243acdb1

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

The following changes were cherrypicked through Coastguard (

https://android-build.googleplex.com/coastguard/)
Release Track: (

https://android-build.googleplex.com/coastguard/dashboard/6093116828680192/#/cherrypickHistory/all)
Created: 2019-10-14 17:23:42.610596+00:00
Changes: aosp/1123258

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

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

commit bdde5a1a970ddc9007b28de4aa29d60ffa588f08
Author: Yigit Boyar <yboyar@google.com>
Date: Thu Apr 16 16:47:05 2020

Re-factor how errors are dismissed when query is re-written

This CL changes how we handle errors/warnings if query is
re-written.
There was a bug in expandProjection where we would report warnings
for things that Room already fixes automatically (

b/140759491

).
The solution to that problem (I7659002e5e0d1ef60fc1af2a625c0c36da0664d8)
solved it by deferring validating of columns until after re-write
decision is made. Unfortunately, this required changing PojoRowAdapter
to have a dummy mapping until it is validating, make it hard to use
as it does have a non-null mapping which is not useful.

This CL partially reverts that change and instead rely on the log
deferring logic we have in Context. This way, we don't need to break
the stability of PojoRowAdapter while still having the ability to
drop warnings that room fixes. This will also play nicer when we
have different query re-writing options that can use more information
about the query results.

Bug: 153387066
Bug: 140759491
Test: existing tests pass
Change-Id: I2ec967c763d33d7a3ff02c1a13c6953b460d1e5f

M room/compiler/src/main/kotlin/androidx/room/log/RLog.kt
M room/compiler/src/main/kotlin/androidx/room/processor/QueryMethodProcessor.kt
M room/compiler/src/main/kotlin/androidx/room/solver/TypeAdapterStore.kt
M room/compiler/src/main/kotlin/androidx/room/solver/query/result/PojoRowAdapter.kt

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

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