Merge "camera: set mNeedConfig after creating ZSL stream"

Merge "AArch64: AString::append for longs and pointers"

Merge "libeffects: 64 bit compile issues"

libeffects: 64 bit compile issues

Change-Id: I32dd13d307572fee91150f8e113df78924ca0067

Merge "media: -Werror re-enabled"

Merge changes I0a744dc7,Id993a70d

* changes:
  media: use size_t for integer iterator to Vector::size()
  media: 64 bit compile issues

media: use size_t for integer iterator to Vector::size()

Change-Id: I0a744dc7815a86a993df9b0623440be620ec8903

media: 64 bit compile issues

- change internal sized types to use stdint.h
- printf & scanf formats
- size_t or unsigned int for iterators

Change-Id: Id993a70d8bf54c667c5d652b34179a2c727ed446

Merge "Remove code markers"

Remove code markers

Change-Id: Iaebfaef2e58e9659d86699d9b7fc68f24cf09bf3

media: -Werror re-enabled

Change-Id: I62fdb78e8acefafd468ce2e5aa9e78eba0dd0f48

Merge "libstagefright: fix 64-bit warnings"

camera: set mNeedConfig after creating ZSL stream

Set mNeedConfig after creating ZSL stream, camera device
will reconfigure for the case when only ZSL stream
is changed.

Change-Id: Ib592817f81125969824a1280349f77973116f375

AArch64: AString::append for longs and pointers

The AString::append methods for long, unsigned long and void *
pointers were using char arrays of 16 elements, which were not long
enough for 64-bit longs and pointers in __LP64__ systems. This
resulted in "FORTIFY_SOURCE: vsprintf: prevented write past end of
buffer. Calling abort()." when the
android.media.cts.DecoderTest#testFlush CTS test was run.

The AString::append methods that were using sprintf have been modifed
to use snprintf instead, taking the sizeof the "s" array (which has
been made 32 char without conditional compilation for __LP64__ where
appropriate), and checking the return value to ensure that the string
has not been truncated.

After this change and changes to the types of OMX_U32 and OMX_S32 in the
frameworks/native/include/media/openmax/OMX_Types.h header file, the
android.media.cts.DecoderTest#testFlush CTS test passes.

Change-Id: I76d897373473c82f52986f43a15b050b844a370a
Signed-off-by: Marcus Oakland <marcus.oakland@arm.com>

libstagefright: fix 64-bit warnings

%lld -> %" PRId64 " for int64_t
%d -> %zu for size_t
Also fixes some casts from void* to integer types, and some comparisons
between signed and unsigned.

(cherry picked from commit b4a7a2df4c28c3f32b5d877b54831d2cc5d78f81)

Change-Id: I76ba94d0b67776fd7abdc83b43d47c61d6c32f4c

Merge "media: memset sizeof argument issues"

Merge "Fix aosp_arm64 mirror-aosp-master build 1119326"

Fix aosp_arm64 mirror-aosp-master build 1119326

Remove -Werror that was set in 'warnings be gone'

Change-Id: I2379dc56f04a6d0ac50f90e6d1cdf05fce3a5c30

media: memset sizeof argument issues

Change-Id: I37ef154eea0f77fb7f22faf6891f3025f6ff346e

Merge "build fix for aosp-idea133 @ 1119318"

Merge "warnings be gone (4/4)"

warnings be gone (4/4)

(cherry picked from commit 84333e0475bc911adc16417f4ca327c975cf6c36)
(squashed with commit b4a7a2df4c28c3f32b5d877b54831d2cc5d78f81)

Change-Id: I55df09714a0e1cb98a44c8fcddac2c646ae5e139

Merge "warnings be gone (3/4)"

warnings be gone (3/4)

(cherry picked from commit 84333e0475bc911adc16417f4ca327c975cf6c36)
(squashed with commit b4a7a2df4c28c3f32b5d877b54831d2cc5d78f81)

Change-Id: I0c5a0c3ea32bb5091d90c7391c67edd02e50397f

Merge "warnings be gone (2/4)"

warnings be gone (2/4)

(cherry picked from commit 84333e0475bc911adc16417f4ca327c975cf6c36)
(squashed with commit b4a7a2df4c28c3f32b5d877b54831d2cc5d78f81)

Change-Id: I048a2c6868515716c7cd7319a3d0abe6311dac4e

build fix for aosp-idea133 @ 1119318

Change-Id: I34084ebb48318628d46c2ff7ca567bd7638619e4

Merge "warnings be gone."

warnings be gone.

(cherry picked from commit 84333e0475bc911adc16417f4ca327c975cf6c36)

Modified by Mark Salyzyn <salyzyn@google.com> to keep merge conflicts
or errors downstream to a minimum.

Change-Id: Ic3b272f9cbf3155001aabd2f79728f1bc31de613

Merge "64-bit: Correction to OMX_U32 and OMX_S32"

Merge "Fix aac decoder flush"

Merge "Fix decoder EOS handling"

64-bit: Correction to OMX_U32 and OMX_S32

Change-Id: I9c41bb34c35595ac19a7ab8faaacde8cb4fa9a15

Fix aac decoder flush

b/8543366

Change-Id: I746ffed6289486b802a2292bfc492ea6c780ed0b

Fix decoder EOS handling

Conceptually it should be the same whether EOS is signalled on the last
buffer holding data, or an empty buffer that follows. Make it so that
this actually behaves the same for mp3, AAC and Vorbis.

b/8747869

Change-Id: Idece8ef45689a3ffaf70fb45d19862d7b93b2f92

Merge "CameraService: deadlock fix"

CameraService: deadlock fix

Release lock before return from CameraService::dump()

Change-Id: Ia20b0921fc675a284f3b7f8d26e0e3596b5a8774
Signed-off-by: Kalle Lampila <kallex.lampila@intel.com>

Merge changes I9f28bc42,I135620e0

* changes:
  libcameraservice: build 64-bit
  libcameraservice: fix 64-bit print format warnings

libcameraservice: build 64-bit

libcameraservice builds fine for 64-bit, remove LOCAL_32_BIT_ONLY

Change-Id: I9f28bc42dc424dee9b92e6ab092f86d3aa1419b7

libcameraservice: fix 64-bit print format warnings

Use PRId64 from inttypes.h to print nsecs_t (int64_t)
Use %zu to print size_t

Change-Id: I135620e0388db33587a8a7da393b48a45cb7275a

Merge "libstagefright is no longer 32 bit only."

Merge "Use libaudioresampler instead of libaudioflinger"

Merge "Add libaudioresampler"

libstagefright is no longer 32 bit only.

Change-Id: I88d5fcfc005a2c2acd0246cdd9c08b6c00b6c39b
Signed-off-by: Glenn Kasten <gkasten@android.com>

Use libaudioresampler instead of libaudioflinger

in order to be able to build libvideoeditor as 64-bit also.

libaudioresampler is available in both 32-bit and 64-bit,
unlike libaudioflinger which is currently 32-bit only.

Change-Id: Id63857da877b1e45b7c1f8e35f18dca6600be3f7

Add libaudioresampler

libaudioresampler is available in both 32-bit and 64-bit,
unlike libaudioflinger which is currently 32-bit only.

Bug: 8141282
Change-Id: I839f7b4e6aaed6984012ca6d514323f927669df6

Merge "libstagefright_foundation: fix printf format for 64-bit"

libstagefright_foundation: fix printf format for 64-bit

Fixes build error:
frameworks/av/media/libstagefright/foundation/ANetworkSession.cpp:582:131: error: format '%d' expects argument of type 'int', but argument 4 has type 'size_t {aka long unsigned int}' [-Werror=format=]
                 ALOGI("%d datagrams remain queued.", mOutFragments.size());
                                                                                                                                   ^
cc1plus: all warnings being treated as errors

Change-Id: I69d1f086b8f1e450f107671c8f8f3ed696d170b4

Merge "AudioPolicyService: malloc/delete pair"

Merge "More precise 32-bit only for libstagefright/ subdirectories"

More precise 32-bit only for libstagefright/ subdirectories

Change-Id: Ie4e8f9ccd834b902287d3452cc9fa739809ddb68

Merge "More 32-bit only"

More 32-bit only

Change-Id: Ic333215e39757f91b12d66b8a9ad7f43c8ba8b36

Merge "mediaserver and associated services are 32-bit only"

Merge "Implemented support for RTSP 301 Redirect"

mediaserver and associated services are 32-bit only

also 32-bit only command-line apps

Change-Id: I9ac557a8d02bbf6986a9b5c3cdce23d400b306a3

Merge "Remove streaming URI from default logs"

Remove streaming URI from default logs

Streaming URI should not be visible in default logcat logs

Change-Id: I104cc56b5335f8c5621013e4c5be8028f0379833

Merge "Check for ARCH_ARM_HAVE_NEON flag only for ARM arch"

Merge "LP64 fixes for media/libeffects"

Merge "audioflinger: Fix for a deadlock in track creation"

audioflinger: Fix for a deadlock in track creation

AudioFlinger enters a deadlock (with itself) on trying to free a
RecordTrack or Track object that failed initialization. Clear this
bad object from the caller instead.

Bug: 12423233
Change-Id: I926f2beb922a70f6924e593e2bbf1a5b5df85b16

Check for ARCH_ARM_HAVE_NEON flag only for ARM arch

This patch prevents ARM neon code from being included
for Arm64, where ARM is the 2nd architecture.

Change-Id: I5a283991034f2650a0e7d4227aef967be276fc9c
Signed-off-by: Ashok Bhat <ashok.bhat@arm.com>

Merge "Fix ALOGVs for 64bit"

Merge "Fix preventing from heap overwrite in capture command"

Fix ALOGVs for 64bit

Change-Id: I556f90f9e6ace1a897d24326e5c14e7bf1e9fe29
Signed-off-by: Kévin PETIT <kevin.petit@arm.com>

Merge "AArch64: Correction for LP64"

Merge "Use multilib-friendly LOCAL_MODULE_RELATIVE_PATH"

Use multilib-friendly LOCAL_MODULE_RELATIVE_PATH

Change-Id: Ifef0dd59ab317b1799fc899bb2e898bb09d2e5f6

AArch64: Correction for LP64

In the Intra16x16PlanePrediction function, a u32 (unsigned 32-bit
integer) was being used for the "i" variable, and being used with a
value of 7 in the loop "for (i = 0, b = 0; i < 8; i++)" to access the
"above[6-i]" location where "above" is defined as "u8 *". Because "i"
was unsigned there was no sign extension on use with the __LP64__
64-bit pointer, so rather than 0xFFFFFFFF being treated as -1, it was
treated as 4,292,967,295 and that resulted in a SIGSEGV at an invalid
address. By changing the type of "i" to an i32 (signed 32-bit integer)
the expected sign extension occurs and the value is treated as -1.

This change fixes android.media.cts.DecoderTest#testCodeBasicH264 on
64-bit platforms

Change-Id: I85df58b0dc1d39f89ab421d04ab5481356520f0c
Signed-off-by: Marcus Oakland <marcus.oakland@arm.com>

Merge "Avoid a building error"

Merge "LiveSession: Use the actual, possibly redirected url as base in the M3U"

LP64 fixes for media/libeffects

Changes include:

[x] In get parameter series of functions, replaced size_t*
    formal parameter type with uint32_t* where actual parameter
    passed was uint32_t*.

[x] In set parameter series of functions, changed size_t
    formal parameter to uint32_t where actual parameter was
    uint32_t.

[x] Changed the definition of LVM_UINT32 from unsigned
    long to uint32_t as unsigned long is 64-bit in LP64.

[x] Used other stdint.h types for other LVM_types for
    consistency.

[x] Use of uintptr_t for the pNextMember of the INST_ALLOC
    structure, rather than LVM_UINT32, for portablility.

[x] Use of uintptr_t where pointers are used in arithmetic.

[x] Replaced the use of 0xFFFFFFFC with ~3 in places where
    it was used to clear last two bits.

[x] Removed int casts where cmdSize and *replySize, both
    uint32_t, were being compared with sizeof().

Change-Id: Ibec0b4d8e9b855f44b1cd853be6df84d13cf4186
Signed-off-by: Marcus Oakland <marcus.oakland@arm.com>
Signed-off-by: Ashok Bhat <ashok.bhat@arm.com>

Fix preventing from heap overwrite in capture command

There was heap usage error in case of stop of audio framework while
capturePoint was negative. Pointer to reply data was moved but
final silence write was done using original buffer size.
Now silence set is done at the beginning under the condition
that framework has stopped.

Change-Id: I7dab1e922f1813e5fbfd4a64c8b0d15d9465520c

Merge "libeffects: don't set module path"

libeffects: don't set module path

Setting the module path breaks multilib builds where there may
be two different module paths for 32-bit and 64-bit, and there
is no need to set it to the default.

Change-Id: I95d62b5573a0bb0ab6c97649041dfc4309cbc8b9

Merge "Replace size_t in shared memory by uint32_t"

Replace size_t in shared memory by uint32_t

Eventually we may want to use uint64_t, but will need to confirm atomicity.

Bug: 12381724
Change-Id: Ia2c591d262d22b47b6f7dab4b9d9faa14b86d865

Merge "avcenc: Only do startcode escaping if the next byte requires it"

avcenc: Only do startcode escaping if the next byte requires it

Section 7.4.1 in the H.264 standard says that the only valid bytes
to follow a sequence that starts with 0x000003 are 0x00, 0x01,
0x02 or 0x03.

This makes EncodeDecodeTest pass properly when decoding using
OMX.google.h264.decoder, which is strict about the forbidden
escape sequences.

Change-Id: Ice113d9b934015003ea9cb10d0b21cee4d18d774

Merge "Make frameworks/av 64-bit compatible"

Make frameworks/av 64-bit compatible

Contains the necessary changes to make frameworks/av build and work
on a 64-bit machine.

Signed-off-by: Craig Barber <craig.barber@arm.com>
Signed-off-by: Kévin PETIT <kevin.petit@arm.com>
Signed-off-by: Ashok Bhat <ashok.bhat@arm.com>
Signed-off-by: Marcus Oakland <marcus.oakland@arm.com>
Change-Id: I725feaae50ed8eee25ca2c947cf15aee1f395c43

Merge "frameworks/av: fix errors inside ALOGV"

frameworks/av: fix errors inside ALOGV

Fix errors exposed by adding compile-time checking to disabled ALOGVs.

Change-Id: Ie06db81d422bb4eee7dfc10abb8d03001627af4c

Merge "AudioFlinger: Remove code for supporting resampling in fast tracks"

Merge "AudioTrack: Never try to use the fast path if resampling is required"

Merge "avcenc: Update video port parameters in the base class"

Merge "ChromiumHTTPDataSource: Keep track of the redirected URL"

Merge "SoftVPXEncoder: Set the frame size on the output port as well"

Merge "M3UParser: Fix typo in 8883a38a308"

M3UParser: Fix typo in 8883a38a308

Change-Id: I09f8deb40b8b34efd4bfcfab6866b7780f8bae96

Merge "M3UParser: Skip query strings when looking for the last slash in a URL"

Merge "frameworks/av: fix errors inside ALOGV"

frameworks/av: fix errors inside ALOGV

Fix errors exposed by adding compile-time checking to disabled ALOGVs.

Change-Id: I9602a4a485dffa3caad732c2a19ec0e41a0ac65b

AudioFlinger: Remove code for supporting resampling in fast tracks

This isn't used at the moment.

Change-Id: I4e0fb2af5f7d959dbafd5ddb7defa1c6b8e8636a

M3UParser: Skip query strings when looking for the last slash in a URL

Change-Id: I72d3a5e11fef9bbd75b291bc490c9cab1dce58da

Merge "audioflinger: conform inline ASM to UAL"

audioflinger: conform inline ASM to UAL

Clang requires some inline ASM to conform to the UAL standards (Unified Assembler Language).
This fixes a small issue in this inline asm to allow building.

Change-Id: Ifd9b1814343ab5ade636b9401a21d575559dac16

Merge "correct one logic error in decide whether should render or not"

correct one logic error in decide whether should render or not

Change-Id: Ie41663f6fd5a7d983279f14a2228cb57231771bf
Signed-off-by: Jianzheng Zhou <jianzheng.zhou@freescale.com>