Rollback SHELL UID check for CarrierConfigLoader

When a test calls adoptShellPermissionIdentity before
calling a command, it doesn't adopt the shell UID, only
the permissions. So, we can not use
Binder#getCallingIdentity to determine it the API is
being called as part of a test. For now, we must only
use MODIFY_PHONE_STATE until a better pattern is
established.

Test: Manual
Bug: 131185660
Merged-In: I607b7c9afe1c384a41ce12b92ae43747b2ee1bc3
Change-Id: I59d32a2e36950b2485e1a46a25e3724928676219

Merge "Add EPROTONOSUPPORT to IpSecManager SSE map"

Merge "Support adding NATT keepalive packet filter"

Merge "Add Network, NetworkCapabilities, StaticIpConfiguration common test cases"

Merge "Freeze {ipmemorystore,networkstack}-aidl-interfaces as version 2."

Support adding NATT keepalive packet filter

Support adding NATT keepalive packet filter to APF
filter.
Generating APF program will be addressed in another CL.

Bug: 33530442
Test: - atest NetworkStackTests
      - atest FrameworksNetTests

Change-Id: I4961d5da343d8700600269632787c28112f0f9f1

Freeze {ipmemorystore,networkstack}-aidl-interfaces as version 2.

Generated with:
m networkstack-aidl-interfaces-freeze-api \
    ipmemorystore-aidl-interfaces-freeze-api

Test: builds, boots
Bug: 128803828
Change-Id: I079018679766738a97b827bc87a6903655f308dd

Merge "Add AIDL parcelable for NattKeepalivePacketData"

Add Network, NetworkCapabilities, StaticIpConfiguration common test cases

Bug: 129200415
Bug: 129200142
Bug: 129200418
Test: atest FrameworksNetTests
Test: atest CtsNetTestCases: added tests pass
Change-Id: I568d516e1ffc9e32061ca6860224c102bdfd7445

Merge "Revert "Make the network stack a persistent process.""

Merge changes Ia9865c75,I47d9d53d,Idbe10c36

* changes:
  Fix some checkstyle nits in DhcpPacketTest.
  Additional code for server name in DHCP packets.
  Parse the server host name field of the dhcp package

Revert "Make the network stack a persistent process."

This reverts commit 796afddd6c7012dc7f10bc2557eb06767c5d873a.

Reason for revert: breaks updatability because persistent processes are not updatable.

Change-Id: I35ded113b31e742f13ad8523a10af3a83a111374

Fix some checkstyle nits in DhcpPacketTest.

Use the Generated code suppression comment filter instead of the
IndentationCheck suppression comment filter because the latter
complains that '+' should be on a new line.

Bug: 120584519
Bug: 127423755
Test: atest NetworkStackTests
Test: atest FrameworksNetTests
Change-Id: Ia9865c75f940e2fbb3aef374a297339e8ea2fd1b

Additional code for server name in DHCP packets.

- Add the hostname to DhcpResultsParcelable.
- Don't store the server hostname if option overload is in use,
  as this is not valid.
- Add unit tests.

Bug: 120584519
Bug: 127423755
Test: atest NetworkStackTests
Test: atest FrameworksNetTests
Change-Id: I47d9d53d1fb58968322cc8b071a44fbc8f7156e1

Parse the server host name field of the dhcp package

Some hotspot devices will fill in the server host name field of the DHCP
package, such as iphone. Parsing the server host name of the DHCP
package can help identify ios hotspots.

Bug: 120584519
Test: 127423755
Test: builds, boots, wifi works
Change-Id: Idbe10c36332ce421f1bba93eb87e9e1e12463088

Merge "Convert misc tests Android.mk files to Android.bp"

Merge "Add validation type definition to support multiple probe result"

Merge "Make DNS cache lifecycle management explicit"

Merge "Fixed the comment of carrier config for CDNR"

Fixed the comment of carrier config for CDNR

Bug: 122921456
Test: build
Merged-In: If5cc8d37ba1e66d59e5bfe9444fe4108f2263be9
Change-Id: If5cc8d37ba1e66d59e5bfe9444fe4108f2263be9
(cherry picked from commit 7ac9a8f314ad8be530f6745a947b90bfcff9495c)

Add validation type definition to support multiple probe result

In order to allow NetworkMonitor to tell ConnectivityService
if the network is either parital, validated or both.
NetworkMonitor could update the probe result by passing the
bitmask to ConnectivityService.

Bug: 130078798
Bug: 130683832
Test: build
Change-Id: I429ccbd9b9afae3eafaf73b3c9eb835213c1f64d

Merge "Remove Settings constants for NetworkStack"

Merge "Update exception type thrown when NetworkStack is missing"

Remove Settings constants for NetworkStack

The definitions reference of these constants are moved from
Settings to DeviceConfig. These definitions are @hide and the
usage in the Settings are removed. Thus, remove the definitions
from API level.

Bug: 123167629
Test: make system-api-stubs-docs-update-current-api \
      test-api-stubs-docs-update-current-api
Test: atest NetworkStackTests
Change-Id: I28c4bf2c0b72e154cea91d11007a9497c7f21570
Merged-In: I7291cfa86cd6e907df35e8fbc97e2e9a77e56cab
Merged-In: Ic48bea07e79490dd3787068bc88c748525752a61

Merge "Add privapp permissions to network modules."

Merge "Use injected instance of RcsControllerCall"

Merge "Do not populate SLOT_KEY with wrong slot info"

Add EPROTONOSUPPORT to IpSecManager SSE map

This change maps EPROTONOSUPPORT to the list of error codes that map to
UnsupportedOperationException in IpSecManager.

Bug: 80103456
Test: Compiles, CTS tests ran
Change-Id: Iec3d5fc4a9bcad7c104414afefae775232d46558

Do not populate SLOT_KEY with wrong slot info

The slotId is not necessarily the phoneId, so do not broadcast as if it
is. SLOT_KEY is a hidden extra so this should not impact apps in general
(if they need the old extra they can still use the hidden PHONE_KEY).

For select broadcasts like SIM_CARD_STATE_CHANGED, we will manually
populate the slot key. This is a temporary solution to b/130664115 and
we should eventually review and consider making the SLOT_KEY public for
some broadcasts.

Bug: 130664115
Test: manual
Change-Id: I84bd4c6bc4277e32cee9916dd18dd80948a808a0
Merged-In: I84bd4c6bc4277e32cee9916dd18dd80948a808a0

Merge "Add 5G evolution icon"

Merge "Make the network stack a persistent process."

Add 5G evolution icon

This add the 5GE icon to the system UI for specific carriers
LTE CA network.

Bug: 124768697
Test: build & manually test:
fake the operator name with pattern, and check the status bar data icon.

Change-Id: Ife0401219138c7ed22e7bfbd4d5e8648c54a98a8
Merged-In: Ife0401219138c7ed22e7bfbd4d5e8648c54a98a8

Merge "Remove unused methods from INetworkManagementService"

Add privapp permissions to network modules.

When built as part of the system, the network stack and the
captive portal login app should have the BYPASS_PRIVATE_DNS
and CONNECTIVITY_USE_RESTRICTED_NETWORKS permissions.
These are necessary to validate restricted networks, bypass VPNs,
bypass private DNS for captive portal login, etc. Add these
permissions to privapp-permissions-platform.xml.

When installed as mainline modules that cannot use signature
permissions, the modules get this ability by virtue of having the
MAINLINE_NETWORK_STACK permissions.

Additionally, add the CONNECTIVITY_USE_RESTRICTED_NETWORKS
permission to the captive portal login app manifest, which did
not contain it.

Bug: 129789428
Test: builds, boots
Test: dumpsys package shows permissions
Change-Id: I632359f7eff09fed71167733ac75824a5aa57894

Update exception type thrown when NetworkStack is missing

rethrowFromSystemServer is throwing DeadSystemException which
is different from the original log message. Thus, update the
way to rethrow the same RemoteException.

Bug: 130028724
Test: atest FrameworksNetTest
Test: Kill NetworkStack and check the log message
Change-Id: I60862e276dd4e2d143278b272a9ba54219acce26

Make the network stack a persistent process.

The network stack should never be killed, because if it does,
recovering networking requires a reboot. Make the process
persistent to reduce the chances of it being killed by the OOM
killer.

Bug: 131046856
Test: builds, boots
Test: process oom_score_adj is -800
Test: process goes from "Persistent Service" to "Persistent" in dumpsys meminfo
Change-Id: Ic8ebf6356e23489f6cd984d680ff9f1b4cbbe52d

Merge "Switch to HexEncoding from ByteStringUtils"

Merge "Delete unused timeout constant."

Merge "Reinstate new VPN uid filtering unit tests"

Add AIDL parcelable for NattKeepalivePacketData

Bug: 33530442
Test: atest FrameworksNetTests

Change-Id: I9b9a51dc5dc06c90229fb36c34c24258991c4146

Delete unused timeout constant.

Test: m
Bug: 129312219
Change-Id: I5b4c8b42dea515f674ff10d64d0e1d06d849cca1

Merge "Create new RcsMessageQueryResultParcelable"

Merge "Move NattKeepalivePacketData out of the framework"

Merge "Update API docs for country ISO APIs."

Merge "Add carrier display name related CarrierConfig"

Merge "Move to android_mallopt for malloc debug calls."

Reinstate new VPN uid filtering unit tests

Mock out PackageManager and returns correct information corresponding
to the test app package itself.

Test: atest --generate-new-metrics 10 com.android.server.ConnectivityServiceTest
Bug: 114231106
Bug: 130397860
Change-Id: Ib921700eda417f411d7a2c77c1140fba9ab50bbb

Merge "Fixed that emergency alert dialog not popped up"

Update API docs for country ISO APIs.

Updating to make it more clear the format of the returned country ISO
codes.

Test: Build
Fixes: 131154060
Change-Id: I10c00c1a48e36e00720f50e85d6bb931756f1085

Merge "Add a timeout for the DNS probe."

Merge "Add tests for DhcpErrorEvent"

Merge "Freeze the networkstack-aidl-interfaces interface"

Merge "[KA11] Verify fd ownership and allocate resource for NattKeepalive"

Freeze the networkstack-aidl-interfaces interface

This freezes the interface as of the latest beta build, not the tip of
tree. IIpClient#setL2KeyAndGroupHint is not in the frozen definition in
particular.

Generated with:
m networkstack-aidl-interfaces-freeze-api \
    ipmemorystore-aidl-interfaces-freeze-api

Test: flashed, booted, WiFi and captive portal working
Bug: 128803828
Change-Id: Ideabe73fc93bbefca2d624ee9ca190cf31419424

[KA11] Verify fd ownership and allocate resource for NattKeepalive

Since socket keepalive APIs for UDP encapsulation sockets are
public to generic app. In order to ensure the given fd is valid,
this change verifies the resource id inside the UDP
encapsulation socket by using methods provided by IpSecService.

Bug: 125517194
Fix: 123968920
Test: 1. atest FrameworksNetTests --generate-new-metrics 10
      2. atestcom.android.server.ConnectivityServiceTest \
         #testNattSocketKeepalives --generate-new-metrics 100
Change-Id: I408aacc19b364683854d15a095c34e72389a6e5b

Merge "Fix urn compare in EmergencyNumber"

Add carrier display name related CarrierConfig

Bug: 122921456
Test: build
Merged-In: I79836b2912be5223a2269e660bfaadf4aa72b930
Change-Id: I79836b2912be5223a2269e660bfaadf4aa72b930
(cherry picked from commit 4ae672f192c009c5e0ae629d1c4e321d9677169b)

Merge "Fix an argument to the legacy broadcast"

Merge "Disable iorap test mapping"

Merge "Add NATT keepalive resources and methods into IpSecService"

Merge "Reduce cbrubaker bus factor by 2"

Disable iorap test mapping

Post submit tests are failing, disable the tests for now.

Bug: 123659771
Bug: 72170747
Bug: 130883339
Test: TH

(cherry picked from commit 6565a34d7ff731180ddb9c4ef05c51bcf6691ec7)

Merged-In: I17befa52ebad92413122253f692168cf93cbab6e
Change-Id: Ida6c6cb2ee30ebb08c959df83ae05fabb112d9c2

Fixed that emergency alert dialog not popped up

Fixed by granting START_ACTIVITIES_FROM_BACKGROUND
to the cell broadcast receiver.

Test: Manual
Bug: 130384940

Merged-In: Ie563a8f11ac991ab9568f65df6a4d14eebe47c5e
Change-Id: Ie563a8f11ac991ab9568f65df6a4d14eebe47c5e
(cherry picked from commit 974607aac84c49dd102a578c7776752216280128)

Fix urn compare in EmergencyNumber

Test: Sanity
Bug: 130979270
Change-Id: I585ecafcf9996e24befec2dbe86465a1d5a0f030

Merge "Add configurability of LinkProperties, meteredness for Test Networks"

Add tests for DhcpErrorEvent

The tests are run both in unit and CTS tests.
Test: atest FrameworksNetTests NetworkStackTestCases
Bug: 129200175

Change-Id: I52976bbbaca26fb317836e8461e372c25df02a22

Add a timeout for the DNS probe.

Bug: 129312219
Test: atest NetworkStackTests
Test: atest --generate-new-metrics 50 NetworkStackTests:com.android.server.connectivity.NetworkMonitorTest
Change-Id: Ib3ab9105d7ae39f551b51e8d5a04b9ec5e549655

Merge "Move more network unit tests to common tests"

Fix an argument to the legacy broadcast

Test: new test for this
Fixes: 62650382
Change-Id: I918b8271d3c3c058553ca888cb54cd36a6efba66

Make DNS cache lifecycle management explicit

1. ConnectivityService calls netd binder to create/destroy network directly.
2. Call dnsresolver binder to create/destroy cache after create/destroy network.
3. Remove unused network create/destroy methods in NetworkManagementService.

Bug: 129453995
Test: atest FrameworksNetTests
Change-Id: I15660d27f735e33d621d4af8972cdf115bf76dfa

Merge "Fix a bug where + before % is ignored"

Merge changes Id598ae1d,I475bd011

* changes:
  Fix a possible crash when the listener is null
  Straighten AIDL interface for the memory store

Merge "Add more documentation for LE CoC"

Fix a bug where + before % is ignored

Achievement unlocked : fix a 10-year-old trivial bug

Bug: 1866121
Test: new CTS in Ib7ee866f65baf99b46a31e2115355a42a829421e
Change-Id: Ic91660d974dce21f2affdcacaeffe9accf8451ac

Fix a possible crash when the listener is null

Test: manual
Bug: 129930804
Change-Id: Id598ae1d98154b19d156e52973cf379669e7060c
Merged-In: I325fd3eda6e89228353342f00a49f50b22b17071
Merged-In: Id8151886ab1d26101e02f9abe9e21c58c7236623
(cherry picked from commit 28671455229a7a1b74b54fa183c6ba4cc225099f)

Straighten AIDL interface for the memory store

Some names were still wrong somehow, and the wrappers were
missing.

Test: NetworkStack & FrameworkNetTests
Change-Id: I475bd011ad9bc714a07021a9dfd85c4876f8e9ad
Merged-In: I475bd011ad9bc714a07021a9dfd85c4876f8e9ad

Merge "Make LegacyTypeTracker testable"

Merge "Remove framework.aidl rules"

Merge "Add nullability annotations to android.bluetooth.le.ScanFilter.Builder setServiceSolicitationUuid"

Merge changes from topic "R"

* changes:
  master designated for R. Update MAX_PLATFORM_VERSION, etc.
  AAPT2: Update SDK development codename to Q

Add more documentation for LE CoC

Added more information into Android SDK for LE Connection-oriented
Channels (CoC) so that it matches the corresponding RFComm
documentation.

Bug: 70683224
Test: Compile
Change-Id: I40abde70a7ca6bcd194ee75bd8367c0ed9e97d05

Add nullability annotations to android.bluetooth.le.ScanFilter.Builder setServiceSolicitationUuid

Test: compilation
Fixes: 126698824
Change-Id: I4873b46a37df3e103ddc62001cc9f0bc01aea4dc

Merge changes from topic "GameDriver3"

* changes:
  Game Driver: process sphal libraries in GPU service
  Game Driver: Add a Settings.Global property for sphal libraries
  [Game Driver] Add global whitelist option.
  [GPU Service] Move blacklists process to GPU service.
  Split blacklist from versioned blacklists.
  Game Driver: clean up GraphicsEnvironment
  [GPU Service] Implement GPU service.

Merge "Return copies for get methods"

Game Driver: process sphal libraries in GPU service

This change adds the sphal libraries text file parsing to the GPU
service. As the result, when the Game Driver apk is updated, the sphal
library list will be read out to the GAME_DRIVER_SPHAL_LIBRARIES
settings global property to be used in the graphics environment to
extend the current linker namespace.

Bug: 124448366
Test: Build, flash and boot. Install the apk to verify settings global.

Change-Id: Ifb4007a1fe7269e0a2857fe7badc8642342b1449
Merged-In: Ifb4007a1fe7269e0a2857fe7badc8642342b1449

Game Driver: Add a Settings.Global property for sphal libraries

This change add the GAME_DRIVER_SPHAL_LIBRARIES property to save the
list of sphal libraries assessible to Game Driver.

Bug: 124448366
Test: Build, flash and boot.
Change-Id: I7cb06df80f19d87f5fd13d7df3c991ceb5bdfb06
Merged-In: I7cb06df80f19d87f5fd13d7df3c991ceb5bdfb06
Exempt-From-Owner-Approval: Change in CoreSettingsObserver.java is too
minor

Merge "Expose android.os.Build.VERSION.ACTIVE_CODENAMES to CTS"

Move to android_mallopt for malloc debug calls.

Remove the guardrail/MemoryLeakTrackUtil.* files. They aren't used
and they are a copy of frameworks/av/media/utils/MemoryLeakTrackUtil.*.

Bug: 130028357

Test: Enable backtrace for calendar, run am dumpheap -n <PID> <FILE>
Change-Id: I6ce69465b0c04fae97b3622df2763a2996063d85
Merged-In: I6ce69465b0c04fae97b3622df2763a2996063d85
(cherry picked from commit 98180a344cfd08748dcf5a6aa5aebfdeb7e78b62)

Add NATT keepalive resources and methods into IpSecService

This change adds a new NATTKeepalive resource type, along with the
associated allocation/deallocation. Additionally, this change allows
ReferenceCountedResource(s) to not be binder-linked, to allow the
ConnectivityService to verify ownership and allocate a NattKeepalive
without double-registering for binder-death notifications.

Bug: 125517194
Test: IpSecService frameworks tests ran
Change-Id: I8293f79940ad57dabb6f2b9de5e334d06b869443

[Game Driver] Add global whitelist option.

Allow a '*' at the beginning of the whitelist file to mean whitelist
everything.

BUG: 120869311
Test: Build, flash and boot. Verify with apk
Change-Id: Ia1b772f545a04acb7f5b4ccbe5489e43ecddb9d2
Merged-In: Ia1b772f545a04acb7f5b4ccbe5489e43ecddb9d2

[GPU Service] Move blacklists process to GPU service.

Instead of re-process blacklists proto everytime, we move the processing to GPU
Service, and add SettingsObserver to observe the change of
Settings.Global.GAME_DRIVER_BLACKLISTS such that we only re-process the
blacklists when needed.

As a result, we will have GAME_DRIVER_BLACKLISTS which is used to accept a list
of blacklist from the server, one for each game driver version that needs to
have blacklist, and GAME_DRIVER_BLACKLIST will only contain a list of
blacklisted application package names for the current version of game driver on
the device, separated by comma.

BUG: 120869311
Test: Build, flash and boot. Use adb shell settings put command to verify.
Change-Id: Ic49800cd0b5b82ddbbdf9907d603243415d5bd3b
Merged-In: Ic49800cd0b5b82ddbbdf9907d603243415d5bd3b

Add configurability of LinkProperties, meteredness for Test Networks

This commit adds the ability to pass in LinkProperties and meteredness
for additional testing use cases.

Bug: ?
Test: Compiles. Test utility only.
Change-Id: Idf6c3bcae2dea5e61c0418b72b889496919be8ba

Return copies for get methods

NetworkRegistrationInfo and ServiceState are not
immutable class. Their get methods should return a new
copy of object so the caller won't modify its state.

Bug: 130538118
Test: Unit tests + manual

Merged-In: I51662a92b0b6189a8c8aa017085affedac417190
Change-Id: I51662a92b0b6189a8c8aa017085affedac417190
(cherry picked from commit 37538594e68dea9a1965ffb8307b259d8cf3717b)

Merge changes from topic "128616035"

* changes:
  Adding @hide API to return whether an APN is metered or not.
  Add API to set alwaysAllowMms

Move NattKeepalivePacketData out of the framework

For implementing parcelable interface for NattKeepalivePacketData.
Move this class out of framework.jar and move to services.jar

This class is used in telephony-common.jar and it also loads
service.jar.

Bug: 33530442
Test: - build pass
      - atest NetworkStackTests
      - atest ConnectivityServiceTest

Change-Id: Id2fdec5c4015d2cefd8a93e0cfdcc3c304fc9b64

Merge "Add CallLog provider loggging to check for app-ops denial."

Merge "Don't call ConnectivityService when holding ActivityManagerService lock."

Adding @hide API to return whether an APN is metered or not.

In addition, make isMmsDataEnabled more generic:
isDataEnabledForApn

Bug: 128616035
Test: manual
Change-Id: Icd520c3094dd761a13435fae79b7b6b3ba2ebda9
Merged-In: Icd520c3094dd761a13435fae79b7b6b3ba2ebda9

Make LegacyTypeTracker testable

...and add basic trivial tests for it.

Test: this, and ConnectivityServiceTest
Bug: 62650382
Change-Id: Ie7ca938e6f66f1b15feb6ed93afa0aebb20884ae

Add API to set alwaysAllowMms

Add API in SubscriptionManager setAlwaysAllowMmsData
and isMmsDataAllowed.

Bug: 128616035
Test: unittest
Change-Id: Ie56c2d0619a68b188c35e8e24db6c2012d22538b
Merged-In: Ie56c2d0619a68b188c35e8e24db6c2012d22538b