tools/bit: Avoid leak in error condition

In the unlikely event of fread() failing, we want to avoid leaking
our buffer or the file handle.

Test: Treehugger
Change-Id: I1866d00c74ad38af4559d747fee696a60464c9f0

Merge "Add new trace tag for AIDL."

Merge "Better javadoc and rename several methods in ApnSetting."

Better javadoc and rename several methods in ApnSetting.

Cherrypicked from ag/3709855.

Test: Manually by TestDpc
Bug: 73087996
Merged-In: I5232c92a44ab505946956f81965a205f170aa52b

Add new trace tag for AIDL.

So we can auto-generate tracing code for AIDL interfaces.

Bug: 74416314
Test: inspect atrace output
Change-Id: I42eee24358763305d6b4e954087f3331ad20f9b7
Merged-In: I42eee24358763305d6b4e954087f3331ad20f9b7

Merge "Pass all relevant resolver info to netd"

Merge "Change null behavior of getCellBandwidths"

Merge "Use ABI_STRING."

Change null behavior of getCellBandwidths

Returns empty array instead of null when there are
no cell bandwidths.

Bug: 73751023
Test: make, runtest
Change-Id: Idc3f034bbea8f6c3f11dcd125f760b121d2c91d9
Merged-In: Idc3f034bbea8f6c3f11dcd125f760b121d2c91d9
(cherry picked from commit c119a863501cc21bfe4853ad46987a2021041e2b)

Use ABI_STRING.

Bug: N/A
Test: builds
Change-Id: I5a9d2a6604887f1f0e9583086e012a086042b7df

Merge "Prune files from /data/anr/ by number as well as age."

Merge "Revert "Revert "Logging: Bluetooth Enabled"""

Merge "More flexible API enforcement policy."

More flexible API enforcement policy.

This CL adds the basics to set black, dark gray or light gray list
enforcement, rather than just black as before. It's not possible to
actually set the policy per-package yet.

PackageDexOptimizer still uses a single bit, for API checks on/off, rather
than the new enum. It assumes blacklist enforcement internally. This can
be improved in a follow up CL.

(cherry-picked from commit e52130ae4cf3b046b38a09fc7cc7712f5b7fb83d)

Test: m
Test: Boot device

BUG: 73337509
Change-Id: Ic0d5b8fa631c2bd583b6fc52b2ee3708c8113f59
Merged-In: Idd73c9938592c5c4d67cfb9efefdffed0dd5f262

Merge "Fix mini-debug-info generation for apps with wrap.sh script."

Merge "audioservice: Add a property for default voice call volume"

Merge "Have the status bar listen to all macro-users VPN changes."

Pass all relevant resolver info to netd

Test: runtest frameworks-net passes
Test: manual testing of opportunistic and strict mode works
Bug: 34953048
Bug: 64133961
Bug: 72345192
Change-Id: Ia4f1905e1fa60e7a468c2246fb809191d2ba796b

Merge "Have PermissionMonitor arbiter which app can access background networks"

Merge "new WFC registration failure notification string"

Merge "Add multisim notifications for SST"

Merge "Add method to NetworkStatsService for UID stats."

Merge "Enable Checkstyle for "repo upload" for more projects."

new WFC registration failure notification string

Bug: 37579327
Test: Manual
Change-Id: I5adaf643b2b2c86cac28dabb944765d0a8bcefd0
(cherry picked from commit 6ac589ae665b2190a762d71948de6f41922352d6)
Merged-in: I5adaf643b2b2c86cac28dabb944765d0a8bcefd0

Enable Checkstyle for "repo upload" for more projects.

This CL enables Checkstyle for the following directories:
- telephony/

Bug: 34746646, 32954735
Test: Compilation
Change-Id: Ia67eaf5b871e6b74f21344fef1572835801843a5

Merge "HIDD: Address API Review concerns"

Prune files from /data/anr/ by number as well as age.

tombstoned prunes based on both age and number of files. Until we can
fully switch over to tombstoned, emulate that here too.

Bug: http://b/73140330
Test: ran tests
Change-Id: I824034019e91d541fc7b7ba49d152e1ceaf37621

Merge "Add more comments for the isConfigForIdentifiedCarrier()"

Revert "Revert "Logging: Bluetooth Enabled""

This reverts commit 46850eb3d2180d29751a18056ae094965e840827.

Reason for revert: The dummy StatsLog class was merged into stage-aosp-master and oc-mr1-dev-plus-aosp, which were the branches that broke initially.

Change-Id: Iab9bac487d845db0ba70b2622679523604b4944e

Merge "Revert "Change KEY_CARRIER_VOLTE_TTY_SUPPORTED_BOOL to false""

Fix mini-debug-info generation for apps with wrap.sh script.

Debuggable apps enable mini-debug-info after fork, however, this does not
work with apps with wrap.sh script since they follow different code path.

Enable mini-debug-info generation for those as well.

Bug: 74070426
Test: check that app with wrap.sh generates debug info for JIT now
Change-Id: I489ac3c82bcced8fc0448ed5666f67009cbb043d

Have PermissionMonitor arbiter which app can access background networks

This will let ConnectivityService send the right callbacks to the
relevant apps.

Test: manual with apps
      runtest frameworks-net
      cts
      new tests for this functionality
Bug: 67408339

Change-Id: I6f08efd9e73c7e191f833d7f307a3bf4c9e2f0b4

Add method to NetworkStatsService for UID stats.

Useful for clients such as BatteryStats which currently rely
on NetworkStatsFactory. Data at that stage is incomplete as
it does not account for tethering, VT data and corresponding
464xlat corrections.

Test: runtest frameworks-net, CTS tests pass.
Change-Id: I763b77f601c827fd2963204694fb5b45425cc791

Merge "Finalise STK call control strings"

Have the status bar listen to all macro-users VPN changes.

Bug: 73217368
Test: manual
      Using Datally on work profile. Before this, enabling the VPN
      does not show the key icon. After this it does.

Change-Id: I454eb8f3881a48af1b0187c2b14a2a399d3c2445

Merge "Mark eraseSubscriptions as @SystemApi."

Finalise STK call control strings

Bug: 64685331
Bug: 70352301
Test: manual
Change-Id: I53e9ec5e520f960b7adb3a29770e1f57065eb4c8

Merge "Calls unlinkToDeath in binderDied to deregister"

Merge "Enable Belarusian and Latin languages in fontchain_lint tool."

Merge "Add Telecom debug setting for enabling enhanced call blocking."

Enable Belarusian and Latin languages in fontchain_lint tool.

Bug: 68058172
Merged-In: I747cbb5ea3d36ca63335bfe6b337fad1e75ee79b

Revert "Change KEY_CARRIER_VOLTE_TTY_SUPPORTED_BOOL to false"

This reverts commit 84a8bb80a15ad9386983a619dafdc10748e6c14e.

Change-Id: Ied14258726d71262642f8d72db5ee11ea671836b

Merge "Add alpha Strings to CellIdentityTdscdma"

Merge "Revert "Logging: Bluetooth Enabled""

Merge "Add NonNull Annotations To IpSec API Surface"

Revert "Logging: Bluetooth Enabled"

This reverts commit f336d1e9cf8b48fe73ad8527365d5286d673ffba.

Reason for revert: Reverting this CL as it causes several build breakages in git_stage-aosp-master

Change-Id: I4a29f7313857a561452c01fb331accbf59c64a72

Add NonNull Annotations To IpSec API Surface

This CL adds NonNull annotations to a large
number of method returns and parameters as
part of API council feedback.

Bug: 72473424
Test: compilation (docstring-only change)
Change-Id: I2f865dde56fe12116c461ad98e9460bf1802ce18

HIDD: Address API Review concerns

 * Replace bare field usage with getter methods;
 * Remove Builder;
 * Move BluetoothHidDeviceCallback to inner class;
 * Remove toArray() and equals();
 * Throw IllegalArgumentException where applicable;
 * Add an Executor parameter before Callback;

Bug: 72168436, 72168126
Test: make update-api, make, make sl4a.Common
Change-Id: I13095458bf3ded7a376e8d20fd13df12ef426693

Merge "Logging: Bluetooth Enabled"

Merge "Check mOwnedByTransform to avoid DELSA on SPI"

Merge "Fingerprint navigation settings is a secure setting"

Merge "Cherry-pick 'Add @Nullable annotations to getOperatorAlphaLong and getOperatorAlphaShort to CellIdentity.'"

Calls unlinkToDeath in binderDied to deregister

When a registered BLE App unexpectedly dies and the binderDied callback
is called, the unlinkToDeath is called to remove linkage. Also, refactor
code to use an existing function.

Test: Manual test by repeatedly killing the gms core process.
Bug: 74076974
Change-Id: If47a534ecafe7fceae14f8cf8526987cabd279cd

Add alpha Strings to CellIdentityTdscdma

Add equivalent alphaLong and alphaShort to
TD-SCDMA so that it's equivalent to LTE, WCDMA,
GSM, etc.

Bug: 75980473
Test: compilation
Merged-In: Ibaa8a7daafb44f7fc94f0fd6aaa193b67793eea4
Change-Id: Ibaa8a7daafb44f7fc94f0fd6aaa193b67793eea4
(cherry picked from commit 0acd7eeed49555f0a07283a6146bbe24c096cd87)

Mark eraseSubscriptions as @SystemApi.

This is a blocker for P.

Bug: 74621500
Test: test on phone
Merged-In: I9455a0b7594cbd2acbdbba99a725c9866c448a07
Change-Id: I9455a0b7594cbd2acbdbba99a725c9866c448a07

Fingerprint navigation settings is a secure setting

Fixes: 76025501

Test: turning on/off the navigation setting actually turns on/off
      the ability to swipe the notification panel

Change-Id: I465fcebfc1e248bd37a609f404462ae8a45cf31b
Merged-In: I465fcebfc1e248bd37a609f404462ae8a45cf31b
Signed-Off-By: Siarhei Vishniakou <svv@google.com>

Merge "Expose add/removeAddress for IpSecInterfaces"

Cherry-pick 'Add @Nullable annotations to getOperatorAlphaLong and
getOperatorAlphaShort to CellIdentity.'

Bug: 75984192
Test: Unit test
Change-Id: I71feb20075559c7b28d4aa346a603f0d11ca69da
Merged-In: I71feb20075559c7b28d4aa346a603f0d11ca69da

Merge "Remove Make side (system/test-)api-stubs droiddoc targets"

Expose add/removeAddress for IpSecInterfaces

When exposing the APIs, these were missed.
The outer structure is exposed, so this exposes
the addAddress and removeAddress methods.

Bug: 75234273
Test: compilation
Change-Id: I79911434f9baa660e4d8564cc59d80da4a710c42

Merge "Remove BLE App entry from mBleApps list when App died"

Merge "Expose remaining READ_PHONE_STATE and SEND_SMS APIs to carrier apps."

Remove Make side (system/test-)api-stubs droiddoc targets

Remove old droiddoc targets for api-stubs, system-api-stubs, and test-api-stubs.

And also set api_tag_name, api_filename, private_api_filename, and etc.
properties in Android.bp for new droiddoc.
Test: manually compare if dependencies change after Soong based droiddoc runs.
Bug: b/70351683

Change-Id: I8d944c83de120ec8148276adc1c9872113d9d29b
Merged-In: I8d944c83de120ec8148276adc1c9872113d9d29b

Merge "Add support for auth-only transforms"

Check mOwnedByTransform to avoid DELSA on SPI

The owned by transform flag prevents the removal
of an SPI from accidentally deleting an associated
SA in the kernel. That flag wasn't actually being
checked, so deleting an SPI would result in the
transform being removed.

The existing code already guarantees that the SA is
deleted when the transform is deleted

Bug: 73258845
Test: runtest frameworks-net
Change-Id: I4c26aea7af817a5d9e54da5db1cdf4f943bcae06

Merge "Close file descriptors of ProfilerInfo"

Remove BLE App entry from mBleApps list when App died

When a registered BLE App unexpectedly dies, its entry in mBleApps
list needs to be cleanup/removed.

Test: Manual test by repeatedly killing the gms core process.
Bug: 74076974
Change-Id: I2dc86b782dd6b07017a360a0b709504f0a375969

Close file descriptors of ProfilerInfo

When using "am start --start-profiler" command, am doesn't close
instances of related ParcelFileDescriptor of the profiler file path,
but relies on garbage collection, so at the system level the file
kept open for longer than it is needed. In a result, some processes
that monitors the file close event using unix inotify_add_watch
method would wait 20~30 Seconds.

This change closes file descriptors after passing it to ActivityThread
process, which is similar to what "am profile start" does.

Bug: b/73891014
Test: tested manually by flashing into Pixel XL device.
Merged-In: I5b63b96335fdaf86f0cd0caec2c53c32c0e41d72
Change-Id: I5b63b96335fdaf86f0cd0caec2c53c32c0e41d72

Merge "Frameworks: Fix Binder.joinThreadPool"

Add multisim notifications for SST

ServiceStateTracker posts notifications for restricted state and
rejection cause for CS registrations which need to be customzied for
multisim cases.

Bug: 69928579
Test: ServiceStateTrackerTest
Change-Id: Ia7db81da6dd858b99113432a97fb489dbebe2436

Frameworks: Fix Binder.joinThreadPool

Fix the lack of a native implementation.

(cherry picked from commit acd1987950d54afef1f69dad11840996c22cb8fe)

Bug: 75254055
Bug: 75291614
Test: atest BinderTest
Merged-In: Id5b15442a59851e566a24d406a7487f9de66d42f
Change-Id: Id5b15442a59851e566a24d406a7487f9de66d42f

Expose remaining READ_PHONE_STATE and SEND_SMS APIs to carrier apps.

Bug: 70177947
Bug: 70041899
Test: Flash / boot; TreeHugger; Unit/CTS tests in topic
Merged-In: I4061becb1d7ceaa58d84e4e3568c1b6e63cfc8d1
Change-Id: I4061becb1d7ceaa58d84e4e3568c1b6e63cfc8d1

Merge "Added implementation for VTI add/remove address"

Merge "Allow carrier-privileged apps to register telephony listeners."

Merge "Add empty CdmaSmsSubaddress to CDMA PDU parsing"

Add Telecom debug setting for enabling enhanced call blocking.

This feature is only available for certain carriers at the current time.
To facilitate test team productivity, adding a system setting which can
be accessed from the Telecom debug menu to enable this functionality.

Test: Manual
Bug: 28189985
Change-Id: I4c812cecd2832a722e5dc73b7c719a9e0594064d

Merge "Frameworks: Handle exceptions in SystemProperties callbacks"

Merge "Add the AVRCP Target Service (2/2)"

Merge "Rename handover failure constant per API review comments."

Merge "DndTile: Avoid yet another NPE during boot up phase"

Frameworks: Handle exceptions in SystemProperties callbacks

When notifying callbacks, catch thrown exceptions.

Bug: 73058952
Test: m
Test: atest SystemPropertiesTest
Change-Id: I1c61e11b833e8ddd30d80b9859b20a9fb3185a33

DndTile: Avoid yet another NPE during boot up phase

 * Tile state update can only take place once mController
   is properly initialized.

Change-Id: Ia6c1d6dee7d3e396a5bb6910e03859a0c0cc94ae

Add support for auth-only transforms

Kernel limitations prevent auth-only SAs from being created. Explicitly
request a null encryption algorithm instead of omitting the algorithm
to comply with the kernel requirement for ESP.

Bug: 75049573
Test: CTS tests for auth-only, crypt-only transforms added for all
combinations of (UDP, TCP, IPv4, IPv6, UDP-encap) Also added unit
tests to ensure correct triggering of NULL_CRYPT usage.
Change-Id: Ia9a5cfee9c7786412846bc039f326420f6211c08

Merge "Docstring Updates for applyTransportModeTransform"

Docstring Updates for applyTransportModeTransform

-Add anotations to usages of PolicyDirection for
 apply...() methods.
-Update the comments on DIRECTION_IN and DIRECTION_OUT
 to better reflect their current usage.
-Add a better explanation to the rekey procedure doc.
-Remove disused createTunnelInterface() stub.

Bug: 73751066
Test: make docs
Change-Id: I9f2ec864466148a18899f1e952c74a525902ccbc

Merge "Renamed getNetworkId and getSystemId"

Merge "NfcAdapter.EXTRA_SE_NAME -> NfcAdapter.EXTRA_SECURE_ELEMENT_NAME"

Rename handover failure constant per API review comments.

Expanding EMERG to EMERGENCY.

Test: Build code
Bug: 74831393
Change-Id: Ib7e6bea84e25b1e1a2ddb154c7ed62f1084a7596

Merge "Define phone process as a constant"

Add empty CdmaSmsSubaddress to CDMA PDU parsing

We were not providing an empty subaddress when
parsing CDMA PDU, which caused an NPE when receiving
CDMA IMS SMS.

Bug: 73994015
Test: Manual, MT/MO SMS
Change-Id: I07be3ac64341eb9235f6da1b6870ac746188f73a

Add the AVRCP Target Service (2/2)

Add the AVRCP Target constant to Bluetooth Profile. Also remove redundant
public static final modifier on the constants.

Bug: 68854188
Test: Compiles and local test with service enabled
Change-Id: If2ec607fc704c225f8903d438fe970dfafac25f1

Merge "Added Locking for MmTelFeature"

Renamed getNetworkId and getSystemId

1. Renamed getNetworkId() to getCdmaNetworkId()
2. Renamed getSystemId() to getCdmaSystemId()

Test: Build
Bug: 74976831
Merged-In: I2a5fb50a59c3239bd3dca5568991a9dff08839fc
Change-Id: I2a5fb50a59c3239bd3dca5568991a9dff08839fc
(cherry picked from commit 2661fac133db49124d2377f49506b12c1a6c9505)

Merge "Added CTS test cases for getNetworkId() and getSystemId()"

Added implementation for VTI add/remove address

This change adds implementation details for add/remove addresses onto a
VTI.

Bug: 73675031
Test: New tests added, passing on Walleye
Change-Id: Idde9d943a5285d2c13c5c6b0f7b8a9faf718e6a5

Merge "Use AsyncTask.SERIAL_EXECUTOR as the default executor for scan"

Add more comments for the isConfigForIdentifiedCarrier()

Bug:74945742
Test: Compile
Change-Id: Ife3fdf797ca8877ec495d61a8aeed56fa20282a7

Merge "Add scan background mode to Settings.Global."

Add scan background mode to Settings.Global.

This enables us to mess with the different scan intervals and shift
clients to a different scan mode in the background based on what scan
interval values we choose for the different power modes.

Bug: 71765044
Test: None. Just adding a key.
Change-Id: Id48ebc521dd3fe8a68c9c4c0bdb1018ea5b3743e

Added CTS test cases for getNetworkId() and getSystemId()

Bug: 32718590
Test: atest CtsTelephonyTestCases:ServiceStateTest#testServiceState
Merged-In: I5826a59d85028f0350ee39c69553f4e488d6456b
Change-Id: I5826a59d85028f0350ee39c69553f4e488d6456b
(cherry picked from commit 1b8b38928b4e20e7e66ab75e39159b158dc5c4c9)