java.lang.SecurityException on specific device model [153246136]

Fixed

Bug

Status Update

No update yet.

Description

so...@gmail.com

created issue #1

Apr 6, 2020 04:00AM

Component used: work
Version used: 2.2.0
implementation "androidx.work:work-runtime:2.2.0"
implementation "androidx.work:work-runtime-ktx:2.2.0"
implementation "androidx.work:work-rxjava2:2.2.0"

Devices/Android versions reproduced on:
W1450 / Android 7.0 (NetworkType does not matter.)

It occurs on specific device model only.
And about 600 crashes occur per month.

The followings are stack trace.
Please check if it can be fixed.

--------------------------------------------------------------------------------------------------------
java.lang.SecurityException: {PACKAGE_NAME} was not granted either of these permissions: android.permission.CHANGE_NETWORK_STATE, android.permission.WRITE_SETTINGS.
at android.os.Parcel.readException(Parcel.java:1683)
at android.os.Parcel.readException(Parcel.java:1636)
at android.net.IConnectivityManager$Stub$Proxy.requestNetwork(IConnectivityManager.java:2343)
at android.net.ConnectivityManager.sendRequestForNetwork(ConnectivityManager.java:2917)
at android.net.ConnectivityManager.registerDefaultNetworkCallback(ConnectivityManager.java:3197)
at androidx.work.impl.constraints.trackers.NetworkStateTracker.startTracking(NetworkStateTracker.java:3)
at androidx.work.impl.constraints.trackers.ConstraintTracker.addListener(ConstraintTracker.java:8)
at androidx.work.impl.constraints.controllers.ConstraintController.replace(ConstraintController.java:7)
at androidx.work.impl.constraints.WorkConstraintsTracker.replace(WorkConstraintsTracker.java:5)
at androidx.work.impl.background.greedy.GreedyScheduler.schedule(GreedyScheduler.java:21)
at androidx.work.impl.Schedulers.schedule(Schedulers.java:15)
at androidx.work.impl.WorkManagerImpl.rescheduleEligibleWork(WorkManagerImpl.java:4)
at androidx.work.impl.utils.ForceStopRunnable.run(ForceStopRunnable.java:9)
at androidx.work.impl.utils.SerialExecutor$Task.run(SerialExecutor.java:1)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1133)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:607)
at java.lang.Thread.run(Thread.java:761)
--------------------------------------------------------------------------------------------------------

Comments

su...@google.com <su...@google.com> #2Apr 6, 2020 04:21PM

Status: Won't Fix (Infeasible)

Following CLs are added to AndroidX:
-

https://r.android.com/2193004
-

https://r.android.com/2186213
-

https://r.android.com/2232128

so...@gmail.com <so...@gmail.com> #3Apr 7, 2020 10:16AM

Chromium counterpart:

https://crbug.com/1355297

su...@google.com <su...@google.com> #4Apr 7, 2020 04:26PM

Assigned to su...@google.com.

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

commit a0fc2a2e7b8c8c09e36202915d836e67c3312253
Author: Abhijith Nair <abhijithnair@google.com>
Date: Mon Oct 03 13:11:02 2022

Add test for ProcessGlobalConfigActivity

Test ensures that there exists a data directory present on the device
with the name specified using `setDataDirectorySuffix(String, Context)`.

Bug: 250553687
Test: ./gradlew webkit:integration-tests:testapp:connectedAndroidTest
Change-Id: Ic1391ac6527fa78464ed4c9f685a067d6dcb3dc1

A webkit/integration-tests/testapp/src/androidTest/java/com/example/androidx/webkit/ProcessGlobalConfigActivityTestAppTest.java
M webkit/integration-tests/testapp/src/androidTest/java/com/example/androidx/webkit/WebkitTestHelpers.java
M webkit/integration-tests/testapp/src/main/AndroidManifest.xml
M webkit/integration-tests/testapp/src/main/java/com/example/androidx/webkit/ProcessGlobalConfigActivity.java
A webkit/integration-tests/testapp/src/main/res/layout/activity_process_global_config.xml
M webkit/webkit/src/main/java/androidx/webkit/internal/StartupFeatures.java

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

so...@gmail.com <so...@gmail.com> #5Apr 7, 2020 04:31PM

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

commit e59d4c20268f10741b4bfb0d939884704659043c
Author: Abhijith Nair <abhijithnair@google.com>
Date: Tue Oct 25 13:58:31 2022

Unhide ProcessGlobalConfig APIs

Along with unhiding the APIs, the API files are also updated.

Test: ./gradlew webkit:integration-tests:testapp:connectedAndroidTest
Bug: 250553687
Relnote: "Add ProcessGlobalConfig enabling users to set process global
config prior to loading WebView."

Change-Id: I7c0e0e42e91efe45f16e66afead7b9e52240bfe4

M webkit/integration-tests/testapp/src/androidTest/java/com/example/androidx/webkit/ProcessGlobalConfigActivityTestAppTest.java
M webkit/integration-tests/testapp/src/androidTest/java/com/example/androidx/webkit/WebkitTestHelpers.java
M webkit/integration-tests/testapp/src/main/java/com/example/androidx/webkit/ProcessGlobalConfigActivity.java
M webkit/webkit/api/current.txt
M webkit/webkit/api/public_plus_experimental_current.txt
M webkit/webkit/api/restricted_current.txt
M webkit/webkit/src/main/java/androidx/webkit/ProcessGlobalConfig.java
M webkit/webkit/src/main/java/androidx/webkit/WebViewFeature.java
M webkit/webkit/src/main/java/androidx/webkit/internal/StartupFeatures.java
M webkit/webkit/src/main/java/androidx/webkit/internal/WebViewFeatureInternal.java

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

ap...@google.com <ap...@google.com> #6Apr 7, 2020 05:40PM

Marked as fixed.

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

commit d1b0ecf7a695406a3a88da9a5f6c6b1859b784a9
Author: Abhijith Nair <abhijithnair@google.com>
Date: Tue Nov 01 18:07:17 2022

Rename the WebView manifest key name

Rename SET_DATA_DIRECTORY_SUFFIX:DEV to
STARTUP_FEATURE_SET_DATA_DIRECTORY_SUFFIX

Change-Id: Id2a0a29a3cbc6972bbf285836cd13e8d2af7fd3d
Test: ./gradlew webkit:integration-tests:testapp:connectedAndroidTest
Bug: 250553687

M webkit/webkit/src/main/java/androidx/webkit/internal/StartupFeatures.java

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

la...@gmail.com <la...@gmail.com> #7Mar 21, 2024 03:19PM

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

commit 531fd4f5db9931f8d6899c79082ece40a5cc0e3c
Author: Abhijith Nair <abhijithnair@google.com>
Date: Mon Jan 30 18:15:05 2023

Refactor ProcessGlobalActivity in testapp

Since there are multiple features we would need to test within
Process Global Config, this CL adds a level of nesting where Process
Global Config activity displays a set of feature specific activties.

Test: "./gradlew :webkit:integration-tests:testapp:connectedCheck --info
-Pandroid.testInstrumentationRunnerArguments.class=com.example.androidx.webkit.ProcessGlobalConfigActivityTestAppTest"
Bug: 250553687

Change-Id: I6e4123aefec34fbb24c260a66a2e3dbbc43556bc

M webkit/integration-tests/testapp/src/androidTest/java/com/example/androidx/webkit/ProcessGlobalConfigActivityTestAppTest.java
M webkit/integration-tests/testapp/src/main/AndroidManifest.xml
A webkit/integration-tests/testapp/src/main/java/com/example/androidx/webkit/DataDirectorySuffixActivity.java
M webkit/integration-tests/testapp/src/main/java/com/example/androidx/webkit/ProcessGlobalConfigActivity.java
M webkit/integration-tests/testapp/src/main/res/layout/activity_data_directory_config.xml
M webkit/integration-tests/testapp/src/main/res/values/donottranslate-strings.xml

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

Issue 153246136