| commit | 77f472105ef285e7489cb5c77d25184f9e4b7285 | [log] [tgz] |
|---|---|---|
| author | Cody Kesting <ckesting@google.com> | Fri Dec 04 17:47:45 2020 +0000 |
| committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | Fri Dec 04 17:47:45 2020 +0000 |
| tree | b56a525574fb127db406f424d2d63fcf16dcb9a8 | |
| parent | 223874fbdb29f29d2e40662743df480c02ce2d0a [diff] | |
| parent | 4068bd9259fe32b18c921b26ef212bec24a2846f [diff] |
Merge "Allow the System UID to unregister ConnDiags CBs." am: 4068bd9259 Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1515688 Change-Id: Ia82d68661c99ed5123f2a0668aec4afd5ea48cf5
diff --git a/services/core/java/com/android/server/ConnectivityService.java b/services/core/java/com/android/server/ConnectivityService.java index e0a484f..58f1db3 100644 --- a/services/core/java/com/android/server/ConnectivityService.java +++ b/services/core/java/com/android/server/ConnectivityService.java
@@ -8200,8 +8200,10 @@ final NetworkRequestInfo nri = cbInfo.mRequestInfo; - if (uid != nri.mUid) { - if (VDBG) loge("Different uid than registrant attempting to unregister cb"); + // Caller's UID must either be the registrants (if they are unregistering) or the System's + // (if the Binder died) + if (uid != nri.mUid && uid != Process.SYSTEM_UID) { + if (DBG) loge("Uid(" + uid + ") not registrant's (" + nri.mUid + ") or System's"); return; }