Merge "ril: fix a memory leak."

ril: fix a memory leak.

This fixes 'Potential leak of memory pointed to by 'socket'

Bug: None
Test: The warning is gone.
Change-Id: I6afca2b4cf4a027c6fb30f1ad7c74083d70dfdb2

Merge "Remove deprecated RADIO_STATES"

Remove deprecated RADIO_STATES

Remove deprecated radio states which carried the SIM state
information. Vendor RILs are not expected to send one of the 3
RADIO states (ON, OFF or UNAVAILABLE) and support the RIL requests
RIL_REQUEST_VOICE_RADIO_TECH and RIL_REQUEST_CDMA_SUBSCRIPTION_SOURCE.

Bug: 32224323
Test: Ensure device boots and camps on network. Basic voice calls and
mobile data works

Change-Id: I2ae4a71c33c43263c64822c294d43363baeb5ad5

Merge "Fix broken asserts."

Fix broken asserts.

When NDEBUG is forcibly undef'ed, these asserts wouldn't compile:
- p_info->fdCommand and p_info->fdListen are ints, not pointers.
- sapSocket->commandFd and sapSocket->listenFd are protected; you need
  to use their accessors.

This patch makes the asserts compile.

Bug: None.
Test: Building with NDEBUG undef'ed now succeeds.
Change-Id: I51418c955994541ba503e1ca6f601dce87e40b55

Merge "Retire socket_loopback_client."

Merge "Update documentation to clarify RIL_REQUEST_GET_ACTIVITY_INFO"

Update documentation to clarify RIL_REQUEST_GET_ACTIVITY_INFO

The documentation for RIL_REQUEST_GET_ACTIVITY_INFO allowed for
critically disruptive behavior in the implementation of the API.
This is a documentation update to ensure that there will be
exactly one power state reported in each millisecond, which
is important for proper power accounting in FDD.

Bug: 32061813
Test: none
Change-Id: I2835d5ad298eaa8cfea2e1a90342f290183f698d

Retire socket_loopback_client.

Making the world safe for IPv6.

Bug: http://b/31537253
Test: builds
Change-Id: If8478aec17892f49e856d6220b72e4585ee9c078

Merge "Fix memory leak in requestSendSMS"

Merge "Fix RIL_CardStatus_v5 Parceling."

Fix RIL_CardStatus_v5 Parceling.

The difference between a v5 and v6 CardStatus response is the addition
of ims_subscription_app_index. In previous releases libril added a -1
to the Parcel in its place for transparent handling of GET_SIM_STATUS
everywhere else regardless of v5 or v6 responses, but it was erroneously
removed in change I4e53c6125995064b8ac0cdfc9376ef2cefe69ec8

Change-Id: I078b61d2f6dc28f738e8eb87f51193fa0bd94c7d

Merge \"Fix misc-macro-parentheses warnings in hardware/ril.\"
am: 1648a3f50b

Change-Id: Ib880298fc4001e0befe10582e9f6e5c792d1bd69

Merge "Fix misc-macro-parentheses warnings in hardware/ril."

Fix misc-macro-parentheses warnings in hardware/ril.

Add parentheses around negative literals.
Bug: 28705665
Test: build with clang-tidy

Change-Id: I038c7cbf3728a4e9ccc20d8c7317ddae3fa9b571

resolve merge conflicts of ccd27b0 to stage-aosp-master
am: 25d580912d

Change-Id: I7afa0604b98b90256c47e806fbb8e11d9ac98594

resolve merge conflicts of ccd27b0 to stage-aosp-master

Change-Id: Ic5a6bb461c60646d7f9798cb1bf99110c6592f91

Merge "Simplify emulator-specific logic."

Simplify emulator-specific logic.

Remove any qemu-specific code from rild.c, this now handled
by setting rild.libpath properly in the init.<board>.rc files
for emulator-specific build products.

Also, only the qemu pipe method is really supported since ICS
so remove obsolete schemes.

BUG=25875346

Change-Id: I82296c127b90c230ca1446ee2431f06e78def617

Merge \\"[qemu]: Use new <system/qemu_pipe.h> header.\\" am: 47a81105da
am: c963b671f3

Change-Id: Id5a2b977a6867537a51f3f740a7df28aca441a4d

Merge \"[qemu]: Use new <system/qemu_pipe.h> header.\"
am: 47a81105da

Change-Id: I3d5e17d7c9468321273bf9540548ef12d931c91d

Merge "[qemu]: Use new <system/qemu_pipe.h> header."

[qemu]: Use new <system/qemu_pipe.h> header.

BUG=25875346

Change-Id: Ie8649caa43a77a4ea1399df673cce2682773e4df

Fix build failure which occurs after setting macro RILC_LOG.
am: 626099ec9b

Change-Id: I193a57f676a81a5835d4506550f94278cbc127cc

Fix build failure which occurs after setting macro RILC_LOG.

Bug: 29214796
Change-Id: I5f74981a59344078f900ecf769cc962b1416d5fd

resolve merge conflicts of c9badf1 to nyc-dev-plus-aosp

Change-Id: I59d138a61554a2439577bcfdf400ea663c6bb91d

Merge "Fix misc-macro-parentheses warnings."
am: e8505bf06e

* commit 'e8505bf06e2f17babe479230e62e6e7d1b8c4bca':
  Fix misc-macro-parentheses warnings.

Change-Id: I40b3707b3206aff65312786db1899949a0ed6b53

Merge "Fix misc-macro-parentheses warnings."

Fix misc-macro-parentheses warnings.

Bug: 28705665
Change-Id: Ib7520c64075a652eb0b15483f36383c4632624d1

Merge "RIL: release unsol wakelock" into nyc-dev
am: 22b45625aa

* commit '22b45625aa72ef9311bf23cd785eccce9535efa5':
  RIL: release unsol wakelock

Change-Id: I24114554188c19eac5f3fc005d0ae118c7c3496e

Merge "RIL: release unsol wakelock" into nyc-dev

RIL: release unsol wakelock

Make sure to release unsol wakelock

BUG=28635279
Change-Id: I4bdf0e7b35df4e6aa077b062ab359c6938ce8292

Specify the INVALID flag for BSIC to be 0xFF
am: f3769da0a3

* commit 'f3769da0a39e2d8116d415f6fd1cba4f1648c03b':
  Specify the INVALID flag for BSIC to be 0xFF

Change-Id: I9676bb84fa15d888abda71ca73360d2f9805b106

Specify the INVALID flag for BSIC to be 0xFF

Bug: 28299027
Change-Id: I3786265a266508d6ea842ccc2a2c3dcf99c3654f

Merge "Suppress unused-parameter warnings in libril." am: a22b128851
am: 5eaec8ba33

* commit '5eaec8ba3354075a2aa66b134a93bc851327cb57':
  Suppress unused-parameter warnings in libril.

Change-Id: I859774e38c07c92634567e91ef129e38a4e1998d

Merge "Suppress unused-parameter warnings in libril."
am: a22b128851

* commit 'a22b128851c769f18ab6e1dde38ef5a4c32c28c2':
  Suppress unused-parameter warnings in libril.

Change-Id: I9c6d05fae4dbb4ff29c44146c7cc221fa1a44a50

Merge "Suppress unused-parameter warnings in libril."

Suppress unused-parameter warnings in libril.

Bug: 26936282
Change-Id: I848a9b2e98e31845d29cc84ccea3517d13339dcc

build fix
am: 1067bb6

* commit '1067bb62f81e8c6b5df8fd0c16f7ee5a8179529a':
  build fix

Change-Id: I742d5d9295b7b3ebdd57a68497fb771138844f71

build fix

Change-Id: I52d7255c352f1a8188658d28f609e5028179d0a0

Merge "fail explicitly if setuid() fails" into nyc-dev
am: c23238d

* commit 'c23238d36e15225bade04710e78defadf9d0039e':
  fail explicitly if setuid() fails

Change-Id: I6be3ecc549efc5279d590477fe91b74548707dea

avoid a potential buffer overflow
am: 1973685

* commit '1973685c095e3e15760d3ba4ad9e77997c46594a':
  avoid a potential buffer overflow

Change-Id: Ib3792c7e229f2bda3244d68137e9e620069cd26c

Merge "fail explicitly if setuid() fails" into nyc-dev

fail explicitly if setuid() fails

otherwise, the following code can be executed
with unintended privileges.

bug:28178548
Change-Id: I265f8c0e1693344a675cc2623968db816309b1f4

avoid a potential buffer overflow

strncat can copy upto n bytes if its 3rd argument is n,
while the 1st argument which is the destination has only n bytes
in this case.

Change-Id: I3469abdcf55e64c5878a855700b323335c40c8be

reference-ril: Stop using pthread_cond_timedwait_relative_np()
am: f3769c1

* commit 'f3769c17615bc9c227cff83cad0e55bb9f878197':
  reference-ril: Stop using pthread_cond_timedwait_relative_np()

Change-Id: Id11534397b2360c2dffa00eede02a4447c6e1030

reference-ril/atchannel.c: Fix time conversion
am: 99f6a4e  -s ours

* commit '99f6a4e8fa12a6d54df5420aaa3eda29d9ba22a6':
  reference-ril/atchannel.c: Fix time conversion

Change-Id: I41ebd943fda8e72aa7eaa19c4159c441e2f1fe64

reference-ril: Stop using pthread_cond_timedwait_relative_np()

pthread_cond_timedwait_relative_np() is deprecated, and is only
available in the 32-bit ABI (see bionic/libc/include/pthread.h). It is
still used by the 32-bit version of reference-ril. Thanks to a recent
bugfix for 64-bit [1], it is now safe to replace the deprecated
function with the standard pthread_cond_timedwait(). Doing so also
simplifies the code, as there is now a unified logic for 32 and 64-bit
targets.

[1] https://android-review.googlesource.com/210301

Signed-off-by: Yu Ning <yu.ning@intel.com>
(cherry picked from commit 3480b1832d3581c708868117c0eb21ba291df165)

Change-Id: I1a15ee0fdfea38b6ee9c9fd49845d10d45f485b1

reference-ril/atchannel.c: Fix time conversion

setTimespecRelative() converts a relative time to an absolute time, by
adding the relative time to current system time. However, it fails to
handle the case where the nanosecond component (tv_nsec) of the
conversion result exceeds 10^9, which can cause a subsequent call to
pthread_cond_timedwait() to return EINVAL.

This bug is the root cause of the "no SIM card" error seen occasionally
on x86_64 Android emulator. In fact, all 64-bit targets use
setTimespecRelative() in conjunction with pthread_cond_timedwait()
during AT handshake, but an EINVAL return value from the latter will
lead to an infinite loop and hang the communication. With this fix,
x86_64 emulator can boot with functional 3G networking every time.

Signed-off-by: Jinhui Li <jinhui.li@intel.com>
[Revised code and commit message]
Signed-off-by: Yu Ning <yu.ning@intel.com>
(cherry picked from commit 11476211584f1a82c870b9486ace8f6f8bb9fc7c)

Change-Id: I5d7396ef7f0af5ef02ccab785046d635fb8f168c

Merge "reference-ril: Stop using pthread_cond_timedwait_relative_np()" am: 2e481da
am: e3546c6

* commit 'e3546c6cb66f2d100de5d0a249f53a783d2d5222':
  reference-ril: Stop using pthread_cond_timedwait_relative_np()

Change-Id: I27b6544423c1ee9fe98c18df452b035f37049b76

Merge "reference-ril: Stop using pthread_cond_timedwait_relative_np()"
am: 2e481da

* commit '2e481dac44a85fbd2e812cd4b7ca06a5a090a6ed':
  reference-ril: Stop using pthread_cond_timedwait_relative_np()

Change-Id: I0c0c35c7839719ec37099d08f0979bb3fef8ed63

Merge "reference-ril: Stop using pthread_cond_timedwait_relative_np()"

Add documentation for RIL_SAP_INIT in ril.h
am: 1be49ba

* commit '1be49ba51de8b4c81132003d12b269f5d0237132':
  Add documentation for RIL_SAP_INIT in ril.h

Change-Id: Ib48fb227e88901c4bac17891d0f09fc459f3ceb2

Add documentation for RIL_SAP_INIT in ril.h

Bug: 27726715
Change-Id: I3c48820f689052a89986191ccb08754197eea229

reference-ril: Stop using pthread_cond_timedwait_relative_np()

pthread_cond_timedwait_relative_np() is deprecated, and is only
available in the 32-bit ABI (see bionic/libc/include/pthread.h). It is
still used by the 32-bit version of reference-ril. Thanks to a recent
bugfix for 64-bit [1], it is now safe to replace the deprecated
function with the standard pthread_cond_timedwait(). Doing so also
simplifies the code, as there is now a unified logic for 32 and 64-bit
targets.

[1] https://android-review.googlesource.com/210301

Change-Id: I0895bbf65ddb71965018d6945dfb8baa786d5d89
Signed-off-by: Yu Ning <yu.ning@intel.com>

Merge "reference-ril/atchannel.c: Fix time conversion" am: 0bbaed7
am: 230df4a

* commit '230df4ab9c69358026634079c1dece849b10d2f0':
  reference-ril/atchannel.c: Fix time conversion

Change-Id: I02f358e240858166c4360b368b23c0dfdee67ff1

Merge "reference-ril/atchannel.c: Fix time conversion"
am: 0bbaed7

* commit '0bbaed79d7054d6c3f085e910e6047dafa11b1ad':
  reference-ril/atchannel.c: Fix time conversion

Change-Id: Ibc7df71b27dc1f8d458cdfc1d3b3b10c57f017a4

Merge "reference-ril/atchannel.c: Fix time conversion"

reference-ril/atchannel.c: Fix time conversion

setTimespecRelative() converts a relative time to an absolute time, by
adding the relative time to current system time. However, it fails to
handle the case where the nanosecond component (tv_nsec) of the
conversion result exceeds 10^9, which can cause a subsequent call to
pthread_cond_timedwait() to return EINVAL.

This bug is the root cause of the "no SIM card" error seen occasionally
on x86_64 Android emulator. In fact, all 64-bit targets use
setTimespecRelative() in conjunction with pthread_cond_timedwait()
during AT handshake, but an EINVAL return value from the latter will
lead to an infinite loop and hang the communication. With this fix,
x86_64 emulator can boot with functional 3G networking every time.

Signed-off-by: Jinhui Li <jinhui.li@intel.com>
[Revised code and commit message]
Signed-off-by: Yu Ning <yu.ning@intel.com>
Change-Id: If04ba560e350bc4e13b22c51f8d215a673230667

Ril Error codes: Voice related messages
am: 1034589

* commit '1034589c37aa83c97f1dcb660209bc6dccb6160a':
  Ril Error codes: Voice related messages

Ril Error codes: Voice related messages

This change adds new error codes for messages
related to voice

Bug: 27038563
Change-Id: I61bcc853f8046e6baf6c8000029f7af0f3f894f0

New error codes for SMS/other messages
am: 68900f5a8e

* commit '68900f5a8eda71d962961ccab56ed3fc99cfd124':
  New error codes for SMS/other messages

New error codes for SMS/other messages

This gerrit adds error codes for messages related
to SMS and set_mute, get_mute requests

Bug: 27038563
Change-Id: I4991e4a38c81501c39e91cfd5df90d8288f542ad

Merge "Add RIL Error code list" into mm-wireless-dev
am: 8dc7f4d003

* commit '8dc7f4d00303f74cbca2d784eaa32e103555f0b5':
  Add RIL Error code list

Merge "Add RIL Error code list" into mm-wireless-dev

Add RIL Error code list

BUG: 27589497

Change-Id: I9cc8c12364676c3c56dbee57fdc01cf055460816

Fix memory leak in atchannel.c

Bug: 27134985
Change-Id: I0273a40f175212bed4aad42e46feb333d9df9a2b

Enable integer sanitization in libril.
am: 2f0b452b4a

* commit '2f0b452b4a234199dcce17dc370aebb126c661c9':
  Enable integer sanitization in libril.

Enable integer sanitization in libril.

Bug: 27486276
Change-Id: I5d440099120567f1ecedb898039b5627346c7089

Fix security vulnerabilities in ril.cpp
am: 0cfc553547

* commit '0cfc553547bb5b0897e0de0c3a38771bed0bf8f9':
  Fix security vulnerabilities in ril.cpp

Fix security vulnerabilities in ril.cpp

Bug: 26522805
Change-Id: I0e29b59a67d571468673b7362ff5adae0f927d7f

Remove log spam in responseCellInfoList in ril.cpp
am: e52a6c9c37

* commit 'e52a6c9c37328aea57d140ddbb0888d92f3ea874':
  Remove log spam in responseCellInfoList in ril.cpp

Remove log spam in responseCellInfoList in ril.cpp

Bug: 27481301
Change-Id: I3a55620d728d579ac50a9c178876e7901340b7af

Fixing security vulnerabilities in ril.cpp
am: 55227b5d7b

* commit '55227b5d7bcd618884b61a5555bd4c2b5dae786c':
  Fixing security vulnerabilities in ril.cpp

Fixing security vulnerabilities in ril.cpp

Bug: 26522805
Change-Id: I8fa19a202b3ebe13dc617b417218b4f40fa8a702

Merge "Use different response type for unsolicited mesg requiring wakelock." into mm-wireless-dev
am: 719d649b12

* commit '719d649b121c4d138651c8286ff57e9ae830d975':
  Use different response type for unsolicited mesg requiring wakelock.

Merge "Use different response type for unsolicited mesg requiring wakelock." into mm-wireless-dev

Create RIL_RadioBandMode Enum
am: 92839f19e3

* commit '92839f19e374cb8377f9f5bddd44692fc2b6e247':
  Create RIL_RadioBandMode Enum

Create RIL_RadioBandMode Enum

Bug: 27145805
Change-Id: Iccdcf1153c9ad389a2927428440963960ff5a80b

Add OEM specific error code space which can by used to replace Generic errors.
am: 0106aedb55

* commit '0106aedb55be31e6197515d0e968584b0a168bef':
  Add OEM specific error code space which can by used to replace Generic errors.

Add OEM specific error code space which can by used to replace Generic errors.

Bug: 27038563
Change-Id: I4052505ed684067cb9145d165f0e335ad52ed404

Convert newly added error code to string values in ril.cpp.
am: b39e5c9c98

* commit 'b39e5c9c98890fa34ff5b8fd591fc16ece1e56a5':
  Convert newly added error code to string values in ril.cpp.

Convert newly added error code to string values in ril.cpp.

Bug: 23216719
Change-Id: Ie2202124cb460872a9f5c406e8f1ae94465d9d0b

Use different response type for unsolicited mesg requiring wakelock.

+ New type is added as ack from RIL java should be sent only for unsolicited mesgs
which hold wakelock in ril.cpp.

Bug: 22127517

Change-Id: Ib7e920d168ab92665283f80476bad3f5ede033e4

Add Error to RIL.h for Rate Limited Requests
am: c97b2849c5

* commit 'c97b2849c57f46d6e2c0f603bf65a05da2c9cb71':
  Add Error to RIL.h for Rate Limited Requests

Add Error to RIL.h for Rate Limited Requests

Bug: 26767732
Change-Id: I7289f53d0d16b6f066d8e469644c81bbb7d4566c

Add support for cdma dynamic address length.
am: 605c78444b

* commit '605c78444bacdb328105f14d55ecc8c046bd96cb':
  Add support for cdma dynamic address length.

Add support for cdma dynamic address length.

Get the address,sub address,Bearer with dynamic length in the data parcel,
to avoid the OutOfBoundsException with the MAX macro.

BUG=26856156

Change-Id: I1d7437143c1369d77791136b68e06088acd72937

Merge "Add error codes to replace generic failure" into mm-wireless-dev
am: 09fd3f75b5

* commit '09fd3f75b530389ff54b6be1f5faab7729f629bb':
  Add error codes to replace generic failure

Merge "Add error codes to replace generic failure" into mm-wireless-dev

Merge "Subject: [PATCH] Fix heap-use-after-free in reference-ril.c found by AddressSaniziter."
am: 99b6d09faf

* commit '99b6d09fafa21ebb1b2de493eb1f7cc315449662':
  Subject: [PATCH] Fix heap-use-after-free in reference-ril.c found by AddressSaniziter.

Merge "Subject: [PATCH] Fix heap-use-after-free in reference-ril.c found by AddressSaniziter."

Improve wakelock symantics in RIL.
am: 6ff9a87b25

* commit '6ff9a87b250598ea9e383f06a4257512bd0c8046':
  Improve wakelock symantics in RIL.

Improve wakelock symantics in RIL.

Bug: 22127517
Change-Id: I13fce25c8c6b27987e576b29a2ca5cd813f5b06f

Add error codes to replace generic failure

Add error codes to be used instead of RIL_E_GENERIC_FAILURE.

Change-Id: I997da9a15225716786c53b7457b498d4628e76be

Subject: [PATCH] Fix heap-use-after-free in reference-ril.c found by
AddressSaniziter.

BUG=26012424

(cherry picked from commit 7c0165ee3b769ed6a67b58b7c6e2f60bd6f8fbf6)

Change-Id: I108f01229b5fe6a368129e608195ba5833f2ea91

[DO NOT MERGE] Added fields to CELL_INFO_LIST. Gsm: ta, arfcn, bsic Wcdma: uarfcn Lte: earfcn
am: f53c5fa963  -s ours

* commit 'f53c5fa96381915e20353cfedc798c0e3c52289b':
  [DO NOT MERGE] Added fields to CELL_INFO_LIST. Gsm: ta, arfcn, bsic Wcdma: uarfcn Lte: earfcn

[DO NOT MERGE] Added fields to CELL_INFO_LIST.
Gsm: ta, arfcn, bsic
Wcdma: uarfcn
Lte: earfcn

Bug: 23528586
Change-Id: I435dd9e945bd8d0553491184ae2919bb5a126a32

Added fields to CELL_INFO_LIST.
Gsm: ta, arfcn, bsic
Wcdma: uarfcn
Lte: earfcn

Bug: 23528586
Change-Id: Iaf7bfaac96bda92d980c566878b9fe481a6d2c21