commit 0f2bc444250e92e23e6f5c980b3b3e4bd190e803
author Jordan Liu <jminjie@google.com> Wed Nov 18 16:47:37 2020 -0800
committer Jordan Liu <jminjie@google.com> Thu Nov 19 00:56:47 2020 +0000
tree 4ee4119ff2346060dbe6f48837800b6db83ef2b0
parent df41fb91554ef6d5f6dd778751fb0f3ef39694c7

Move getDataServicePackages to after clearCallingIdentity

getDataServicePackages invokes another API call through
CarrierConfigManager which tries to do another permission check. In
order to avoid a package-name/UID mismatch for the second permission
check, we need to clear the calling identity so the UID matches
com.android.phone.

Bug: 173515503
Test: atest CarrierConfigManagerTest
      atest TelephonyLocationTests
      atest SignalStrengthTest
      atest <any CTS which calls TM#getServiceStateForSubscriber>
Change-Id: I9c627637289205cdc2b88a6f2276ea07f20d0c52