Merge "Tethering: fix using wrong subId problem"

Merge "Notify the caller about I/O error"

Tethering: fix using wrong subId problem

There is the potential bug[1] that default data subId change
intent may lose. So tethering may cache the outdated default
data subId in TetheringConfiguration. Now EntitlementManager
would fetch configuration every time and this would re-exam
whether subId is changed.
Additional passing subId to Settings to avoid default data
subId change right away when launching entitlement check.
Thus, Settings can know whether this is outdated entitlement
check request.
[1] b/129224596

Bug: 129751453
Test: -build, flash, boot
      -atest FrameworksNetTests
      -manual test with carrier SIM

Change-Id: If334dd1cd383cced9db5cf0d0bc9dc60ed198816

Notify the caller about I/O error

Call onError for caller of BugreportManager.start() if any io error
occurs during runtime.

Test: Tested manually by throwing IO exception in the code when
bugreport is being generated, onError of the callback successfully
called.
Change-Id: I9033d85d392b926041fc26a86806a370752d062d

Merge "Fix DnsResolver API could not bypass private DNS correctly"

Merge "Restrict access to dangerous methods to <= P"

Merge "Release keepalive slot after stopped"

Fix DnsResolver API could not bypass private DNS correctly

Bug: 130594022
Test: atest DnsResolverTest
Change-Id: Ibf3edac3075d921d189237c485871efb869b320b

Merge "Add comments in grouping APIs to clarify possible exceptions."

Merge "Add EHPLMN and HPLMN to SubscriptionInfo"

Merge "Fix Emergency Number API doc"

Merge "Get new surface if it was released from setWindowStopped"

Add comments in grouping APIs to clarify possible exceptions.

Bug: 127408608
Test: cts, unittest, TelephonyManagerTestApp
Change-Id: I3671e366be7e8c3a75b5ee23b7cf88a33ecd3a4f
Merged-In: I3671e366be7e8c3a75b5ee23b7cf88a33ecd3a4f

Fix Emergency Number API doc

1) remove documentation about null return
2) Map including the key*s* as the active subscription ID*s*

Bug: 130426238
Test: Treehugger
Change-Id: I4c9b682ef25378b9de91cd6255ed6fc514ea220d
Merged-In: I4c9b682ef25378b9de91cd6255ed6fc514ea220d
(cherry picked from commit aca6eb7dac0a307756d0c3e76b26503618f40df5)

Add EHPLMN and HPLMN to SubscriptionInfo

Add EHPLMN and HPLMN fields to SubscriptionInfo. These fields are read
from the ICC card.

Bug: 130253962
Test: manual
Change-Id: If98c57c14e39b917a5e7e8bbc59c2170989b63a7

Merge "Refine Bluetooth Metadata API"

Merge "Add NonNull in Carriers constant"

Add NonNull in Carriers constant

Add NonNull in Carriers constant

Bug: 126699334
Test: build pass
Change-Id: Ifda6407092847e8863cba79360eba3df12f9c7e1
Merged-In: I428ff623d46d3891b05882df5774343bfe3234c0

Merge "Switch ByteStringUtils over to HexEncoding impl"

Merge "Move some telephony methods to max-sdk-p list."

Merge "Update OWNERS"

Release keepalive slot after stopped

Currntly, keepalive slot is released when stop() is called. Next
starting keepalive can use the same slot number while previous
keepalive is still stopping. When the previous keepalive is
stopped, the incoming as will be processed by the new keepalive.

This change release keepalive slot after the result of stopping
has returned. Thus, newly created keepalive cannot allocate the
same slot number while lower layer is still processing stop event.

This change also disable flaky assertions that are caused by
test port has been occupied by other process.

Bug: 129512753
Test: 1. atest com.android.server.ConnectivityServiceTest \
         #testNattSocketKeepalives --generate-new-metrics 100
      2. atest FrameworksNetTests --generate-new-metrics 10
      3. simulate the fail case manually.

Change-Id: I1991627545519ee5cb408a3df3a006f710f4af7b

Merge "Import translations. DO NOT MERGE"

Merge "Import translations. DO NOT MERGE"

Merge "Import translations. DO NOT MERGE"

Merge "Import translations. DO NOT MERGE"

Merge "Import translations. DO NOT MERGE"

Merge "Import translations. DO NOT MERGE"

Merge "Import translations. DO NOT MERGE"

Merge "Import translations. DO NOT MERGE"

Merge "Import translations. DO NOT MERGE"

Merge "Import translations. DO NOT MERGE"

Merge "Import translations. DO NOT MERGE"

Merge "Import translations. DO NOT MERGE"

Import translations. DO NOT MERGE

Change-Id: I3488f6c74138241f29504eafd73569f780ee9a74
Auto-generated-cl: translation import

Import translations. DO NOT MERGE

Change-Id: Ib4bb37b12e29ecd0a1a8a726d4a761eaba4abd1f
Auto-generated-cl: translation import

Import translations. DO NOT MERGE

Change-Id: I492f38a82826682746367dfb5122e22b934712f2
Auto-generated-cl: translation import

Import translations. DO NOT MERGE

Change-Id: I219d0c8ab1cf805a0f32e64b53dee3a94abfa68a
Auto-generated-cl: translation import

Import translations. DO NOT MERGE

Change-Id: I55e7b90aabc7dbc880c72cffe1e5282c8ef27f1d
Auto-generated-cl: translation import

Import translations. DO NOT MERGE

Change-Id: I1f5ac7cc91022343245dad427a0dd40c4ed12594
Auto-generated-cl: translation import

Import translations. DO NOT MERGE

Change-Id: Ia4b5b3d094538e5a3e64613a9b7dc11591aaf809
Auto-generated-cl: translation import

Import translations. DO NOT MERGE

Change-Id: I0f47607801a75f9c98ec454e7902102dc3e39988
Auto-generated-cl: translation import

Import translations. DO NOT MERGE

Change-Id: I6d280118ec178a0e787498bd9e38c816e156e6bb
Auto-generated-cl: translation import

Import translations. DO NOT MERGE

Change-Id: I5b163a3870ef427527470e0443bab579cb88ea36
Auto-generated-cl: translation import

Import translations. DO NOT MERGE

Change-Id: I59be6fef50c3065874ca36a6ed77bfa93242c3ee
Auto-generated-cl: translation import

Import translations. DO NOT MERGE

Change-Id: I271a1675c49cfaabbf3cb74c2d805390efc2161f
Auto-generated-cl: translation import

Merge "Fixed the incorrect function signature"

Merge "Remove key_ prefix from CarrierConfig Strings"

Merge "Ask netd to swap stats map before reading"

Merge "Framework: Lock inversion checker"

Merge "Add MEDIA_TIMEOUT error for RTCP timeout"

Merge "Bluetooth: Make ScanRecord.getServiceSolicitationUuids NonNull"

Framework: Lock inversion checker

Add an agent-based lock inversion checker. The agent will dynamically
rewrite bytecode to inject calls to LockHook, which runs a checker
on these.

Implement a simple on-thread checker that keeps a per-thread stack
of locks and a global map of lock ordering. As-is, transitivity of
checks is not guaranteed, but should be captured in most practical
cases.

To run a process with the lock checker, start the process with the
agent. The helper script start_with_lockagent.sh can be used for this:

  adb shell setprop wrap.pkg-name script start_with_lockagent

(cherry picked from commit aeb6fce5b33680bc538dbd66979a28bcba1329b4)

Bug: 124744938
Test: manual
Merged-In: Idd9a7066a5b8cb8c0de2e995f08759c98d9473e1
Change-Id: Idd9a7066a5b8cb8c0de2e995f08759c98d9473e1

Merge "Preload: Add a simple test for the state of classes"

Merge "Pass core platform API policy property to ART"

Merge changes I4f181789,Ia8a0c99b

* changes:
  Revert new tests and PackageManager mock
  Revert "Revert "Block incoming non-VPN packets to apps under fully-routed VPN""

Revert new tests and PackageManager mock

A mocked PackageManager caused test failures in existing tests.
Revert that for now to make tests pass again.

Bug: 114231106
Bug: 130397860
Test: atest FrameworksNetTests
Change-Id: I4f181789152438f18e6cd2d235d76fabe3872ea3

Revert "Revert "Block incoming non-VPN packets to apps under fully-routed VPN""

This reverts commit 3897df2e36d4d015304b398c545c1232ee35a1c9.

Reason for revert: Rolling forward, will fix tests in same CL stack.

Bug: 114231106
Bug: 130397860
Test: FrameworksNetTests
Change-Id: Ia8a0c99b4e1fd5dff26c881715cd876618ca4321

Merge "Revert "Block incoming non-VPN packets to apps under fully-routed VPN""

Pass core platform API policy property to ART

When starting ART, read property
""persist.debug.dalvik.vm.core_platform_api_policy" and pass its value
on the command line. This enables users to enable core platform API
violation reporting on demand. Note that the settings take effect after
a reboot.

Bug: 125701194
Test: compiles, boots, reporting observed
Change-Id: I97507afc85d8e186736700979c5dbf2ad0f79f12

Revert "Block incoming non-VPN packets to apps under fully-routed VPN"

This reverts commit fd8f96d71925cc80f5052365af2e5150cc0ec3ca.
This change does not have any topic: not reverting the other 2 commits in the original topic.

Reason for revert: broke FrameworksNetTests presubmit: b/130397860

Change-Id: Iff41d9fe97fafea44680c8d67d1ce19277548cc0

Merge changes from topic "nnapi_ext_restrict"

* changes:
  Remove PRODUCT_SERVICES from PackageManagerNative.getLocation.
  Add getLocationFlags getter to PackageManagerNative.

Merge "Pass valid file descriptor to binder"

Bluetooth: Make ScanRecord.getServiceSolicitationUuids NonNull

Test: compilation
Bug: 115639878
Merged-In: I5f895d65a7036275ab0faa0afd2ee1eebc416476
Change-Id: I5f895d65a7036275ab0faa0afd2ee1eebc416476

Merge "Add a lock to protect persistent stats snapshot"

Merge "Change the validation URL for Chinese SIMs"

Merge "API council feedbacks for DnsResolver"

Merge "Improve Range Checks on CellIdentity Classes"

Remove key_ prefix from CarrierConfig Strings

Remove the key_ prefix from the resolved string
of:
KEY_SUBSCRIPTION_GROUP_UUID_STRING; and
KEY_IS_OPPORTUNISTIC_SUBSCRIPTION_BOOL

Bug: 130374824
Test: compilation (trivial string change)
Change-Id: I1ae1df9d33229d3f3aba237d865ceb2373761af4

Preload: Add a simple test for the state of classes

Use app_process and reflection to find the class state.

This is expected to break when internals change, so include a test
for expectations.

Bug: 130206915
Test: atest PreloadCheck
Change-Id: I7ecdb88a19efcfcfa1c7aa5c7e67d1990306166e

Merge "Update API for UiSelector."

Update API for UiSelector.

Bug: 130371784
Test: API definition update. no impact to code.
Change-Id: I310b98426f0a8bc2e6c52f266ea669619a775552

Improve Range Checks on CellIdentity Classes

Add range checks to ensure that parameters passed
to CellIdentity are within range; if not then set
them to the UNAVAILABLE value.

Bug: 122834811
Test: atest FrameworksTelephonyTests
Change-Id: I0d3699823be60722a1b741efd60b56f23bbc3e86

Pass valid file descriptor to binder

screenshotFd needs to be optional in Bugreport API. For some bugreports
such as wifi,telephony, interactive etc taking a screenshot is not
required.

Initially, the API was sending invalid file descriptor to the Binder, but
that binder transaction could not be completed as Binder validates the
file descriptor to be valid and not-null.

Adding a tmp.png screenshot file to pass to bugreport API call. In a
separate CL in frameworks/native, added check that the bugreports that
don't require screenshots would not use this file descriptor value.

Bug: 128981582
Test: Tested by taking bugreports using Bugreport API in shell
Merged-In: I3233f5753506ae159c9fa591742e6b99e361039b
Change-Id: I3233f5753506ae159c9fa591742e6b99e361039b
(cherry picked from commit b462e2e49964ed500b49ad07cbe80d96866d61bb)

Merge "Block incoming non-VPN packets to apps under fully-routed VPN"

Merge "Create new RcsParticipantQueryResultParcelable"

API council feedbacks for DnsResolver

To address the API review feedback provided by
the API council.

Bug: 129261432
Test: atest DnsResolverTest
Change-Id: I3de11c913682abf790850b45cd5d50ac28b3fc5c

Merge "Add support for TAP interfaces in TestNetworkManager."

Merge "provide a resource overlay to configure customized DNS Server."

Merge "Address nit in NetworkStackUtils"

Merge "Switch from Byte.toHexString() to HexEncoding"

Switch from Byte.toHexString() to HexEncoding

Switch from Byte.toHexString() to HexEncoding equivalents.

Bug: 124232146
Test: build only
Exempt-From-Owner-Approval: Cherry-picking OWNERS approved internal version
Merged-In: Id244e5247eb87c67446cbd65eb5d320f3617b8cd
Change-Id: Id244e5247eb87c67446cbd65eb5d320f3617b8cd

Change the validation URL for Chinese SIMs

The default HTTPS and fallback validation URLs are not accessible from
China, so users in China never get a validated network on WiFi with the
default AOSP settings. OEMs selling devices targeting China typically
change the check URLs for that reason.

When the device uses a Chinese SIM card, change the default validation
URLs to URLs accessible accessible from China as of now.

Test: inserted China SIM, WiFi off/on, dumpsys network_stack shows new
      URL used
Bug: 111819230
Change-Id: Ib7f41f470ef1a9c7712f2477ebfab44e915cf1b9

provide a resource overlay to configure customized DNS Server.

Bug: 122621524
Test:  atest FrameworksNetTests NetworkStackTests
Change-Id: I123b3d4516cc6ba6d17629eb74daf448ba7121eb

Merge "Remove @hide from builder of CarrierRestrictionRules"

Address nit in NetworkStackUtils

Follow up commit to fix nit in aosp/937891

Bug: 120013793
Test: Build
Change-Id: I61a6306d3c439a2d72323ab56e2b47771cc60d48

Fixed the incorrect function signature

The name source should be an integer instead of long

Test: Manual + unit tests
Bug: 130237749
Merged-In: Ic4447da24073d37401ae0f6e8f8b41bd60d7167d
Change-Id: Ic4447da24073d37401ae0f6e8f8b41bd60d7167d
(cherry picked from commit 5d8f6447a7e5e90d74cda99173adb1535a2770a7)

Merge "Freeze netd AIDL interface (framework)"

Merge "Disable XOM on libraries for apps with targetSdkVerison<Q"

Remove @hide from builder of CarrierRestrictionRules

Remove @hide from builder of CarrierRestrictionRules class.

Bug: 124325937
Test: compilation

Change-Id: If05e7d950b6737a6bb92af962816c7cbc36b00d1

Add MEDIA_TIMEOUT error for RTCP timeout

Bug: 130167535
Test: Build
Change-Id: I7498072a1c6a9ecf084308e91db07f3731969106

Merge "Allow GSM RSSI levels to be customized by CarrierConfig"

Merge "Update AppComponentFactory.instantiateClassLoader docs"

Disable XOM on libraries for apps with targetSdkVerison<Q

Apps may (and do) assume that libraries are readable. To avoid app
breakage, mark execute-only sections of as read+execute
for apps with targetSdkVersion<Q.

Bug: 128907672
Test: Check libc for app with targetSdk==current
cat /proc/25950/maps | grep libc.so
77c01e3000-77c028b000 --xp 00041000 07:20 106 /apex/com.android.runtime/lib64/bionic/libc.so
Test: Check libc for app with targetSdk<current
cat /proc/26355/maps | grep libc.so
77c01e3000-77c028b000 r-xp 00041000 07:20 106
/apex/com.android.runtime/lib64/bionic/libc.so

Change-Id: I90b5c91923c8008ae4b4818985842fe3e354a850
Merged-In: I90b5c91923c8008ae4b4818985842fe3e354a850
(cherry picked from commit 739c0b5193345abfce0373502c795448cc94a3ab)

Ask netd to swap stats map before reading

To avoid protentail race problem between netd and system_server when
reading the network stats map. Always inform netd before reading the
stats and let netd to do a swap between active stats map and inactive
stats map. So the system_server can safely remove the stats after
reading.

Bug: 126620214
Test: android.app.usage.cts.NetworkUsageStatsTest
      android.net.cts.TrafficStatsTest

Change-Id: I8fa37c26bec23ffca0b29b679e72ba1189f557f1

Add a lock to protect persistent stats snapshot

Since the network stats could be polled from multiple services at
runtime, it is not thread safe for networkStatsFactory to hold a
persistent stats snapshot without any protection. Use a internal lock to
prevent concurrent modification on mPersistentSnapshot to fix the
problem.

Bug: 124764595
Test: android.app.usage.cts.NetworkUsageStatsTest
      android.net.cts.TrafficStatsTest

Change-Id: I22afb46f17697e8b6359d4f593802e0f4b95db8b

Update AppComponentFactory.instantiateClassLoader docs

Clarify the intended use and properties of the default class loader.

Bug: 128524313
Test: n/a
Change-Id: Iae82554f9294d5248b98f1fa72fc1a47993e86fd

Merge "Update Javadocs for CellInfo Classes"

Merge changes I98573a5c,Ic7394f24

* changes:
  Enable checks for MANAGE_TEST_NETWORKS in TestNetworkService
  Cleanup of Test Network service

Block incoming non-VPN packets to apps under fully-routed VPN

When a fully-routed VPN is running, we want to prevent normal apps
under the VPN from receiving packets originating from any local non-VPN
interfaces. This is achieved by using eBPF to create a per-UID input
interface whitelist and populate the whitelist such that all
non-bypassable apps under a VPN can only receive packets from the VPN's
TUN interface (and loopback implicitly)

This is the framework part of the change that build the whitelist.
The whitelist needs to be updated in the following cases:
* When a VPN is connected and disconnected
    This will cover the change to allowBypass bit, since that can't be
    changed without reconnecting.
* When a VPN's NetworkCapabilites is changed (whitelist/blacklist app changes)
* When a new app is installed
* When an existing app is removed
* When a VPN becomes fully-routed or is no longer fully-routed

New user/profile creation will automatically result in a whitelist app change
transition so it doesn't need to be handled specially here.

Due to the limitation of the kernel IPSec interacting with eBPF (sk_buf->ifindex
does not point to the virtual tunnel interface for kernel IPSec), the whitelist
will only apply to app VPNs but not legacy VPN connections, to prevent breaking
connectivity with kernel IPSec entirely.

Test: atest PermissionMonitorTest
Test: atest android.net.RouteInfoTest
Test: atest com.android.server.ConnectivityServiceTest
Test: atest HostsideVpnTests
Bug: 114231106
Change-Id: I143b03d60e46cb1b04732b4a4034f5847b4d1b1a