ApfFilter: systematically use u8, u16, u32 getters

This patch adds a getUint8 getter for ByteBuffers and changes ApfFilter
to make uses of getUint8/16/32 everywhere.

The return types of getUint16 is also changed from long to int, which
will expand gracefully to long as an unsigned int as it is guaranteed to
be positive after getUint16.

Test: ApfTest passes

(cherry picked from commit 995dd94673005b43d32456e2de5fda0090b23576)

Change-Id: I606ebc5aedfcacde400d27cc6bc37145769b122c

Add fuzzing tests to ApfFilter RA processing

Test: added new unit tests

(cherry picked from commit 8acea76a2b7555b3bd5ca1170cca9d09e979fafc)

Change-Id: I847d7e4895766042043c0bba1c9b9a698a705d87

Support timeouts for requestNetwork() invocations.

(cherry-pick of 06c3ef1367e850746eef0a5462bdce8674d74c30)
(cherry picked from commit 57faba9ec73fbb311d5ca1df3278aee9728589d5)

Bug: 21414325
Change-Id: I1a58823a372154589f972b98c4c428eab0e0523e

Silence the obnoxious MTU 0 error message that occur when no MTU is
specified for a given network.

Bug: 10705136

(cherry picked from commit 54f0d9edbf7f6ea6dd805dc77fdbf2c3848188e2)

Change-Id: I89e2fdfbbfa981d435a2974422d9d9c009872d76

Merge "Fix wording ambiguity for Bluetooth LDAC Playback Quality selection"
am: 0645c21163

Change-Id: I66a7a5f8dc423c7999b4e0a39a272a837e66c847

Merge "Fix wording ambiguity for Bluetooth LDAC Playback Quality selection"

Merge "Adding Dynamic ImsService Binding (1/3)"
am: 8e6b13a1c0

Change-Id: I8534273da5579b7662e66f80a60348a4665dd65b

Merge "Adding Dynamic ImsService Binding (1/3)"

Adding Dynamic ImsService Binding (1/3)

Adds support for dynamic ImsService Binding (change 1/3). Included
in this change:
- AIDLs for ImsServiceController
- ImsFeature/ImsServiceBase definitions
- KEY_CONFIG_IMS_PACKAGE_OVERRIDE CarrierConfig option

Test: Unit Tests in opt/telephony
Bug: 30290416
Change-Id: Ic4cb1d85a29681b08a6a525c588a72209862dcc3

Fix wording ambiguity for Bluetooth LDAC Playback Quality selection

"Optimize for" -> "Optimized for"

Bug: 34687041
Test: UI Visual inspection
Change-Id: I112ed6451465750978181e6ee35f96fd76969e9c

Merge "zygote: don't drop CAP_SYS_PTRACE from the bounding set."
am: 273b886c49

Change-Id: I82c58cd03c8db78a1733ba5070d314607475cee7

Merge "zygote: don't drop CAP_SYS_PTRACE from the bounding set."

Merge "Fix a use-of-uninitialized-value warning."
am: 500cf67206

Change-Id: I8b171a90696e09c42a1506255bc4c0056f69f35c

zygote: don't drop CAP_SYS_PTRACE from the bounding set.

crash_dump needs to acquire CAP_SYS_PTRACE to be able to ptrace
processes with capabilities. selinux should hopefully be sufficient for
restricting ptrace to processes that should do it.

Bug: http://b/34694637
Test: debuggerd `pidof system_server`
Change-Id: If46f0b9baa54529780f7767f309f76b102a42ab4

Merge "Fix a use-of-uninitialized-value warning."

Merge "nfc: Make NDEF validation stricter"
am: f3a6269fed

Change-Id: If69e586c81e5926c0795af3689de67b260891eaa

Merge "nfc: Make NDEF validation stricter"

nfc: Make NDEF validation stricter

- Check if the MB is not set for any record apart from the first one.
- If the record is chunked, first record must contain the type

Bug: 20722275
Test: Manual; write raw NDEF messages to a tag
Change-Id: I3e73fc5163bbcd8a1f23417e98f57b665f4f1fad
Signed-off-by: Ruchi Kandoi <kandoiruchi@google.com>

Merge "Bluetooth: fix indentation, formatting in dumpsys"
am: 35c715332f

Change-Id: I7c603ab9f50921afb45e3d7181541260d8939089

Merge "Bluetooth: fix indentation, formatting in dumpsys"

Bluetooth: fix indentation, formatting in dumpsys

Before
======
Enable log:
01-17 09:43:18   Enabled  by com.android.systemui
01-18 07:13:10  Disabled  by com.andorid.systemui

1 BLE Apps registered:
com.google.android.gms

After
=====
Enable log:
  02-06 07:49:07   Enabled  by system boot
  02-06 07:49:35  Disabled  by com.andorid.systemui

1 BLE App registered:
  com.google.android.gms

Test: run bugreport, see differences in dumpsys
Change-Id: I6086f7f97c1fcbdb860d70d9516fd5bf630a9f5e

Merge "API: Update Collections to OpenJDK8u60"
am: 815fb54798

Change-Id: I97f0155ed743c7bc674f0241b2b07955e8d68e56

Merge "API: Update Collections to OpenJDK8u60"

Merge "hotspot2: add support for complete PerProviderSubscription/Credential subtree"
am: 123aa5d8b1

Change-Id: If8164839812728361fcd3d9b2f37c481ccf9b33c

Merge "hotspot2: add support for complete PerProviderSubscription/Credential subtree"

Merge "hotspot2: add support for complete HomeSP subtree"
am: 28c7afe578

Change-Id: If1b097fb5b9453d1e108534c1fb088dfc92b1b6e

Merge "hotspot2: add support for complete HomeSP subtree"

hotspot2: add support for complete PerProviderSubscription/Credential subtree

Added missing fields in Credential subtree for supporting Release 2
and added parsing support for the new fields.

Bug: 34198926
Test: frameworks/base/wifi/test/runtests.sh
Change-Id: Ic3665c963ab77ddc4b9a03262517a3c7a4ec3ffc

hotspot2: add support for complete HomeSP subtree

Added missing fields in HomeSP subtree for supporting Release 2
and added parsing support for the new fields.

Bug: 34198926
Test: frameworks/base/wifi/test/runtests.sh
Change-Id: Id2766b6b48be8bd97b56eb7c57a0e8527f38e06c

Fix a use-of-uninitialized-value warning.

Caught by clang's static analyzer:
frameworks/base/core/jni/android_hardware_SoundTrigger.cpp:660:20:
warning: Assigned value is garbage or undefined
        nHandle[0] = (jint)handle;

Given that we return an error code anyway, we can just set it to 0 and
move on.

Bug: None
Test: Still builds; static analyzer doesn't complain.
Change-Id: I9a5be562a0d96fd6c48bb98eca0867fc754bba63

Merge "unbindBackupAgent and initialize inFullBackup"
am: 36654ffef5

Change-Id: I5ba75075e2fb2dae0dcabbb0cee3b83c3d0c15ea

Merge "unbindBackupAgent and initialize inFullBackup"

Merge "Vodafone SIMs should treat Spain as roaming"
am: a9c27b348c

Change-Id: Ia57e460220a5f8e53f47f2454f5c1a005830a3e6

Merge "Vodafone SIMs should treat Spain as roaming"

Merge "Change HFP Client API to support multi device"
am: 01f2e67648

Change-Id: I887b83939581fe1fd4068f7a4a72a6910c994c74

Merge "Change HFP Client API to support multi device"

Merge "Prepare for removal of legacy-test from default targets (Part 2)"
am: 5a3a3d298a

Change-Id: I14fd4502454a98baf01f5ace83fe0c583005a52d

Merge "Prepare for removal of legacy-test from default targets (Part 2)"

Prepare for removal of legacy-test from default targets (Part 2)

In preparation for removing junit classes from the Android API
the legacy-test target will be removed from the
TARGET_DEFAULT_JAVA_LIBRARIES. This change adds explicit
dependencies on junit and/or legacy-android-test to ensure that
modules will compile properly once it is removed.

Bug: 30188076
Test: make checkbuild
Merged-In: If507b641624a2933e722f61e815890fcdf442265
Change-Id: Icdcad50f1cc32e900bc9e882f1db2867ea170fbf

API: Update Collections to OpenJDK8u60

Bug: 29935305
Bug: 31896176
Test: make update-api
Change-Id: I94f0f8c6915fdc1c8151b816d0a988a07629e984

Merge "Expose more ICU4J APIs."
am: e5f0221f06

Change-Id: I77e300979bdfe781ad9ab7aff5241b92491e5660

Merge "Expose more ICU4J APIs."

Merge "All upstream mobile connection management is now in UpstreamNetworkMonitor"
am: c1987487a2

Change-Id: I2b7e0f63c8198c8fde52396f989c16d795e612a8

Merge "All upstream mobile connection management is now in UpstreamNetworkMonitor"

All upstream mobile connection management is now in UpstreamNetworkMonitor

Test: as follows
    - built (bullhead)
    - flashed
    - booted
    - runtest frameworks-net passes
Bug: 32163131

Change-Id: Ifbd430d5bf2d2191469c9810ad8f073506abde46

Merge "support suw and post-suw cases for default app"
am: 9fc8d99c76

Change-Id: I7a451bf2cfdfd6db65208a46db9d664039297ace

Merge "support suw and post-suw cases for default app"

support suw and post-suw cases for default app

- our default carrier actions might not compatible with suw since data might be disabled
without surfacing users with an option to activate inside SuW.
- instead, skip carrier actions during setupwizard and schedule a job to resume carrier actions if needed when exit suw.
- use jobscheduler to monitor provision bit instead of keep the app running and listen for content change asynchronously.

Bug: 30958215
Test: Manual

Change-Id: Ie4004b46405b914158f585059d332f4e694053f3

Merge "Mark the INetworkScoreCache as oneway."
am: edadb3efc3

Change-Id: I3b71e154f49b354ead2bf734da4a94e332f87a33

Merge "Mark the INetworkScoreCache as oneway."

Merge "Deprecate old TelecomManager actions"
am: d1ab15a1af

Change-Id: Id76db43d6cc4477a328698e049558ef936a4fe1d

Merge "Deprecate old TelecomManager actions"

Mark the INetworkScoreCache as oneway.

Test: built & run
Bug: 34522097
Change-Id: I7d2075febe6c3cdc24b56df255f8ba59e13322bf

Merge "LocalDisplayAdapter: don't crash when no color modes available"
am: 24218db904

Change-Id: I89d080a00fb87c73c2af1604f617706f0f60f32b

Merge "LocalDisplayAdapter: don't crash when no color modes available"

Merge "Make TelephonyManager#get/setAllowedCarriers system api"
am: 6d064d8780

Change-Id: I8f9b1c8e9ce1fda6cd03cd0dd459ead8837bd90d

Merge "Make TelephonyManager#get/setAllowedCarriers system api"

Merge "Disable always-on VPN in factoryReset"
am: 63c14e0ab6

Change-Id: I021b288e3db17ddefbea3645fd282e3ee123a6e8

Merge "Disable always-on VPN in factoryReset"

LocalDisplayAdapter: don't crash when no color modes available

Test: none

Change-Id: Ic11b5b66864fdd97b290722cba534b2aa38ac054

Merge "Move TetheringConfiguration out to its own file"
am: 5a45170f95  -s ours

Change-Id: I6a6bc4b2679432dd539d6b6930d7a32229fab363

Move TetheringConfiguration out to its own file

Test: as follows
    - built (bullhead)
    - flashed
    - booted
    - runtest frameworks-net
    - vanilla tethering works
        - changing from no dun requirement to dun shows dun required
Bug: 32163131

Change-Id: I488449570dff057311b37335923ce6a199b363b7

Merge "Move TetheringConfiguration out to its own file"

Move TetheringConfiguration out to its own file

Test: as follows
    - built (bullhead)
    - flashed
    - booted
    - runtest frameworks-net
    - vanilla tethering works
        - changing from no dun requirement to dun shows dun required
Bug: 32163131

Change-Id: I488449570dff057311b37335923ce6a199b363b7

Merge "Add test for updateMobileRequiresDun()"
am: 7f018500dc

Change-Id: Ic7b984105e2c245a992446e15d11dd19e4671a57

Merge "Add test for updateMobileRequiresDun()"

Add test for updateMobileRequiresDun()

This test validates that when a mobile upstream has been requested
changing the "DUN required" state results in updating the requested
network appropriately.

Test: as follows
    - built
    - flashed
    - booted
    - runtest frameworks-test passes
Bug: 32163131

Change-Id: Ia3a1f6493ea348eb75e334344468a980206f618f

Merge "More concisely export whether we have a "tetherable configuration""
am: 1b2a1b911d

Change-Id: I5f9717c9dab6b39f1fbf2186107b2a1f3f2d4f8f

Merge "More concisely export whether we have a "tetherable configuration""

More concisely export whether we have a "tetherable configuration"

Test: as follows
    - built (bullhead)
    - flashed
    - booted
    - runtest frameworks-net passes
    - vanilla WiFi-to-mobile tethering works
Bug: 32163131
Change-Id: I20dd36b5bf7fc55a639c76ea4bdb55b650654881

Merge "Add Setting for curating saved open networks."
am: b3b665de22

Change-Id: I3c65f39dc4f38f1c6c00a240f519cb0b0cf1a916

Merge "Add Setting for curating saved open networks."

Merge "Move several config elements into TetheringConfiguration class"
am: 8d35276910

Change-Id: I0eb196a4b61e460db24271d6805177641ad3fc9b

Move several config elements into TetheringConfiguration class
am: 3e7566572e

Change-Id: Ic053965b54eed0afc9a4785ce80ef1fab281493a

Merge "Move several config elements into TetheringConfiguration class"

Merge "Address a few unittest comments."
am: 9c3a298748

Change-Id: I2d41cdbe05653bcbed9d3b38b440c06b98006809

Merge "Address a few unittest comments."

Change HFP Client API to support multi device

- Adds BluetoothDevice as a parameter where required
- Gets rid of device management APIs that can be done via
  BluetoothProfiles instead

Test: Manual sanity tests
Bug: b/33554547
Bug: b/30984220
Change-Id: I3485ac5bfe1fcb29c774ad040fdd608e1cacb8df
(cherry picked from commit 039eeb81b5ceb5a7141ae852346e054348ac2a06)

Add Setting for curating saved open networks.

Bug: 34392914
Test: make
Change-Id: Icd17753af6b0efdb35d1060b16029a53fdf4e605
Merged-In: Icd17753af6b0efdb35d1060b16029a53fdf4e605

Merge "Add additional fields to RecommendationRequest."
am: b8f0e69ef0

Change-Id: I9a50c8aca36cf740abbc62af6879b4e6ea736855

Merge "Add additional fields to RecommendationRequest."

Merge "SystemServer: reorder wifi services start"
am: 10c514d9de

Change-Id: I3a918b707932084759701103c683fffed0e16cee

Merge "SystemServer: reorder wifi services start"

Merge "Add Binder support for Parcelable exceptions."
am: a2ef6b5741

Change-Id: Ide766c0220e86b7661af83551354b3a498297ed8

Merge "Add Binder support for Parcelable exceptions."

Add additional fields to RecommendationRequest.

- Add networkId and timestamp for the last user selected network

Bug: 34467596
Test: runtest --path core/tests/coretests/src/android/net/RecommendationRequestTest.java
Change-Id: Ic81c1beede1a643cd428b6e5c57ed430a6b1ccc0
Merged-In: Ic81c1beede1a643cd428b6e5c57ed430a6b1ccc0

Expose more ICU4J APIs.

Expose more classes/methods, mostly those added in ICU4J 57.

Additionally EthiopicCalendar is now un-hidden which was hidden
accidentally and should have been in N.

Bug: 30825380
Test: m checkapi
Change-Id: Ib3bea26a9b246401d33cc0ccc0f91768d1fe1286

Address a few unittest comments.

Test: as follows
    - built (bullhead)
    - flashed
    - booted
    - runtest frameworks-net passes
Bug: 32163131
Change-Id: I8f6002243031dceff60b3d6d377b9e25f239eb23

Merge "Refactor to SimChangeListener class"
am: a12217012e

Change-Id: I863f0b23251a0ebd18a76cf08e1ff0f2cee9cda3

Move several config elements into TetheringConfiguration class

Encapsulate the following config elements:
    - all interface name regular expressions
    - specification of preferred upstreams
    - downstream DHCPv4 ranges
    - default IPv4 (and IPv6) DNS servers

Test: as follows
    - builds
    - flashed (bullhead)
    - runtest frameworks-net passes
Bug: 32163131
Change-Id: Ife4871c0b89d2d52e2cc9238e0606b428287521c

Merge "Refactor to SimChangeListener class"

Merge "Address comments from previous change (320592)"
am: e7cb6c3761

Change-Id: I3266a02a4a2fd76c4bb2f1a744103add63307e26

Merge "Address comments from previous change (320592)"

Merge "Don't loop forever on zero-length ICMPv6 ND options."
am: e7128b10a8

Change-Id: If2260dd2b1bdaf2619cee668f8c2ef81e9dcfdce

Merge "Don't loop forever on zero-length ICMPv6 ND options."

Refactor to SimChangeListener class

Test: as follows
    - built (bullhead)
    - flashed
    - booted
    - runtest frameswork-net passes
    - tested wifi-to-mobile tethering, including SIM remove/insert
Bug: 32163131
Change-Id: I6aac9a8f34cfd0079a4433471ec08b9b1ec0d754

Merge "DO NOT MERGE Unit tests for NetworkNotificationManager"
am: 216a4e7655  -s ours

Change-Id: I05fe9095395df00b96df3cbe3a6e4f31a160e0b5

Merge "DO NOT MERGE Unit tests for NetworkNotificationManager"

Address comments from previous change (320592)

Test: as follows
    - built (bullhead)
    - flashed
    - booted
    - runtest frameworks-net passes
    - vanilla wifi-to-mobile tethering works
Bug: 32163131

Change-Id: I8788cb0d93606a2893c3dbc4f22e72f450f319b8

Don't loop forever on zero-length ICMPv6 ND options.

Bug: 33828433

(cherry picked from commit 2f157c421a9fa15660b8f1bc0c64b1ebb45fa64a)

Change-Id: Icc03f0fcbf40d87e0b9c69e1172802ffcc85531b