Merge "libeffects: do not use GNU old-style field designators"

libeffects: do not use GNU old-style field designators

Not using a GNU extension improves portability

Change-Id: I4e8a0376c089828ca4fc7f1aaf3914bfd917d4c5

Merge "libvideoeditor: fix M4OSA_Clock header guard"

libvideoeditor: fix M4OSA_Clock header guard

CLOCK was mispelled as CLOCH

Change-Id: Ie7163fc8723d7a5b9df113f55a4a550e5465d9b5

Merge "MediaCodec: Pass the crop rect to SoftwareRenderer"

MediaCodec: Pass the crop rect to SoftwareRenderer

Change-Id: I434411516eb5fafe9d7aeac9882bbc7f1a8415d8

Merge "strtok stores its values in thread local storage. So it can not guarantee works well when multithread environment. AudioFlinger has multithread. so strtok_r is more safe."

strtok stores its values in thread local storage.
So it can not guarantee works well when multithread environment.
AudioFlinger has multithread.
so strtok_r is more safe.

Change-Id: I6d77ef9cc49a4478dd856dcdca14e4920ce955c6

Merge "Uninitialized pointer can cause crash."

Merge "Fix crash in MyHandler when sockets are not set."

Merge "Added support for 3GPP meta data atoms"

Added support for 3GPP meta data atoms

3GPP metadata is not supported, only iTunes Metadata
Format is supported.

Change-Id: I78b41c88788d5a43800c936e0ac0f98424515c62

Merge "Fix SoundPool.play() looping regression in 4.3 DO NOT MERGE"

Fix SoundPool.play() looping regression in 4.3 DO NOT MERGE

The change I7370d6e59a7ef26dfb284a8b058d5ab2e0a42ccf caused a regression
in SoundPool looping when using SoundPool's streaming implementation.
This reverts a portion of that change.

Bug: https://code.google.com/p/android/issues/detail?id=58113
Bug: 10171337
Change-Id: I8af0dc8683a7c7f225c80f0eb4d39770667b52e5

Merge "Handle undefined NAL type for h264 streaming"

Merge "Fix mismatched delete/new couple"

Merge "rtsp handle response line ended with '\n'"

rtsp handle response line ended with '\n'

Change-Id: I5bfafd3fa2c95083e833da2846556282eada2b02
Signed-off-by: Yajun Zeng <beanz@marvell.com>

Fix mismatched delete/new couple

In RecordThread::readInputParameters(), mRsmpInBuffer is allocated
with new[] and should be freed with delete[] instead of delete.

This error has been found by valgrind.

Change-Id: I2f947abb658a21a7d1e61eebb99270cb37d2d147
Signed-off-by: Andrei V. FOMITCHEV <andreix.fomitchev@intel.com>
Signed-off-by: David Wagner <david.wagner@intel.com>
Signed-off-by: Jack Ren <jack.ren@intel.com>
Signed-off-by: Bruce Beare <bruce.j.beare@intel.com>

Handle undefined NAL type for h264 streaming

Packages of undefined NAL type (0) was observed but lead to deleting
the subsequent package due to the current assembler implementation.
Identifying and ignoring this package without returning an error
handles undefined packages without side-effects.

Change-Id: I02e15b8682bee3154b3c4acf82639a28417f0c85

Reconcile with jb-mr2-zeroday-release - do not merge

Change-Id: Ie4a7e41d8d28b46966f6aadfd38af5516c082f04

merge in jb-mr2-zeroday-release history after reset to jb-mr2-dev

Reconcile with jb-mr2-zeroday-release - do not merge

Change-Id: I9a9eea4a01ba78b06cb79a26db9b0bcb52f5fd7b

am a2520db0: camera2/3: set preview callback oneshot flag during still capture

* commit 'a2520db02bead68d4980783c41500ae96511bdf8':
  camera2/3: set preview callback oneshot flag during still capture

merge in jb-mr2-zeroday-release history after reset to jb-mr2-dev

camera2/3: set preview callback oneshot flag during still capture

Preview callback oneshot flag need set when a still picture is being captured
and preview callback oneshot is enabled, otherwise, the callback enabling status
could be lost when preview is restarted after still catpure.

Bug 9742091

Change-Id: If9562369013e53727edfdd16f3d00d6a9392e70d

am 4a5c009b: Camera2/3: As a fallback, unilaterally free old recording buffers

* commit '4a5c009bc2389304cf078c270bceb656479367d2':
  Camera2/3: As a fallback, unilaterally free old recording buffers

Reconcile with jb-mr2-zeroday-release - do not merge

Change-Id: I13e45e13c3abf52cef3fb58de3f4707fb8fedaac

am c582fde9: resolved conflicts for merge of c158971f to stage-aosp-master

* commit 'c582fde93ded7219107157333a9e46d780adcf9c':
  Store rtsp accessunit until PLAY response parsed

resolved conflicts for merge of c158971f to stage-aosp-master

Change-Id: I3d77b86f7e616af62a826fc37126706ad8ff6158

Camera2/3: As a fallback, unilaterally free old recording buffers

When starting a recording session, check for and clear out recording
buffers still owned by stagefright from a previous recording session,
if any.  There should never be any, but in case they are due to a bug
elsewhere in the system, clearing out the buffers makes recovery
possible.

Bug: 9591080
Change-Id: I3a2a1256860bd174381cd525948419f985c9b415

Merge "Store rtsp accessunit until PLAY response parsed"

am 5de9ef20: am 365ed394: Merge "Typo in CameraParameters::set"

* commit '5de9ef20d596bc20f53cba664f1ac7e358910ea2':
  Typo in CameraParameters::set

am 365ed394: Merge "Typo in CameraParameters::set"

* commit '365ed394c4135d9d42082c85638c076aed4bd156':
  Typo in CameraParameters::set

Merge "Typo in CameraParameters::set"

am 5d2ab2a6: am 8eef093c: Merge "libstagefright: Use LOG_BASIC when use chromium http stack"

* commit '5d2ab2a6d5e90c045d0989ef1be6c66a2295ee2d':
  libstagefright: Use LOG_BASIC when use chromium http stack

am 06197925: am 267363ee: Merge "Prevent Media Server crash while the AwesomePlayer getBitrate return true with mBitrate equals to zero."

* commit '061979253985e59c4fcb82ede232717736e633b1':
  Prevent Media Server crash while the AwesomePlayer getBitrate return true with mBitrate equals to zero.

am 8eef093c: Merge "libstagefright: Use LOG_BASIC when use chromium http stack"

* commit '8eef093c213094c7efb655ef4d9e31439646bd7f':
  libstagefright: Use LOG_BASIC when use chromium http stack

am 7266bc80: am ca7f1609: Merge "Prevent double disconnects in RTSPSource"

* commit '7266bc80850dc327e9f18c27b81613f959399630':
  Prevent double disconnects in RTSPSource

Merge "libstagefright: Use LOG_BASIC when use chromium http stack"

am 267363ee: Merge "Prevent Media Server crash while the AwesomePlayer getBitrate return true with mBitrate equals to zero."

* commit '267363ee2836ff9bc18d7fdd93aaedf19de546ad':
  Prevent Media Server crash while the AwesomePlayer getBitrate return true with mBitrate equals to zero.

am ca7f1609: Merge "Prevent double disconnects in RTSPSource"

* commit 'ca7f160921a4a1a5da32ebe8fab1067bd8998a1f':
  Prevent double disconnects in RTSPSource

Merge "Prevent Media Server crash while the AwesomePlayer getBitrate return true with mBitrate equals to zero."

Merge "Prevent double disconnects in RTSPSource"

merge in jb-mr2-zeroday-release history after reset to jb-mr2-dev

am eced745a: Merge "Part of fix for TEE resource leaks when app crashes" into jb-mr2-dev

* commit 'eced745af7d2d27a8c45e04eb071d4fb30cd3e48':
  Part of fix for TEE resource leaks when app crashes

Merge "Part of fix for TEE resource leaks when app crashes" into jb-mr2-dev

Part of fix for TEE resource leaks when app crashes

bug: 9672606
Change-Id: I98cf65260d5e43d34e3897452f6f26ed2c89294e

am f59a4b39: fix scheduling policy service death detection

* commit 'f59a4b393f4844c5bbc8d6212364bdddea33d232':
  fix scheduling policy service death detection

Prevent Media Server crash while the AwesomePlayer getBitrate return true with mBitrate equals to zero.

[Cause]
   - getBitrate(int64_t *bitrate) will return true if mBitrate = 0.
   - Then, in getCachedDuration_l(int64_t *durationUs, bool *eos),
     we might execute the following function when
     getBitrate(&bitrate) = true and bitrate = 0.
        *durationUs = cachedDataRemaining * 8000000ll / bitrate;
   - Mediaserver will be crashed when divided by zero.
   - The mediaserver crash often occur when DLNA IOP tests on Golden DMS
     with AAC_ADTS_320 audio.
[Solution]
   - Prevent the divide function executing when bitrate is zero.

Change-Id: I4439d92cee5faec95df2109e9186c33b3fff6c66

fix scheduling policy service death detection

Check status of transactions to scheduling policy service
and re-acquire a binder interface in case of DEAD_OBJECT.

Bug: 8875559.
Change-Id: I1e00bd44e2d4723b3ec95d5c31d9652ba08e238a

Reconcile with jb-mr2-release jb-mr2-zeroday-release - do not merge

Change-Id: Idf0f0cdf02e6e94603ff003b6baf869b4414e1cf

Prevent double disconnects in RTSPSource

Prevent a null pointer from occuring by checking that we are not
already disconnected before performing a disconnect, in case
multiple disconnects have been issued.

Change-Id: Ib88800cd246086fd57f77d10748d41b1aefe2084

merge in jb-mr2-zeroday-release history after reset to jb-mr2-dev

am 47a1fe04: am a9aa6baa: Merge "Set u.ext_data to null after it has been freed"

* commit '47a1fe04637793f48d808261b1fac47acc3de1e1':
  Set u.ext_data to null after it has been freed

am a9aa6baa: Merge "Set u.ext_data to null after it has been freed"

* commit 'a9aa6baa5d86fc1a5bd37fc636d766d9e3ad2507':
  Set u.ext_data to null after it has been freed

Merge "Set u.ext_data to null after it has been freed"

Update HTTP proxy configuration for all media playback inside stagefright.

Change-Id: Ie0dd00045aba668d8b49da73224e7a7c9c04f69b
related-to-bug: 8873723
(cherry picked from commit 2704965b8a1ff3b7450ff58ccecf86d8ec688c40)

am 83790fcc: am bdf33c98: Merge "Fix compilation warning: incorrect format specifier for type \'ssize_t\'."

* commit '83790fcce4f5cda1e7c89f1cbed452426502f227':
  Fix compilation warning: incorrect format specifier for type 'ssize_t'.

am bdf33c98: Merge "Fix compilation warning: incorrect format specifier for type \'ssize_t\'."

* commit 'bdf33c98912090176183d3e9b9a9217dbbdca6da':
  Fix compilation warning: incorrect format specifier for type 'ssize_t'.

Merge "Fix compilation warning: incorrect format specifier for type 'ssize_t'."

am a691ff3c: Camera2/3: Don\'t allow recording and callbacks to coexist.

* commit 'a691ff3c03e38e148bbefed35ebb15e552a12613':
  Camera2/3: Don't allow recording and callbacks to coexist.

Set u.ext_data to null after it has been freed

SEGV_MAPPER crash could happen in MetaData::typed_data::freeStorage.
The faulty address could be both random pointers and deadbaad. It
seems like free could be called more than once on u.ext_data. Set
u.ext_data to null after it has been freed as there already is a
null check before trying to free it.

Change-Id: I710c49f9c231bd8852159914c0b2add58a685f0e

Camera2/3: Don't allow recording and callbacks to coexist.

- Tear down conflicting streams when necessary.

- Shut down callbacks if recording starts

- Do not allow callbacks to start if recording is active

Per the current camera API, recording and preview callbacks cannot be
active simultaneously. However, the framework did not explicitly
disallow this, and in fact left the streams configured once they were
created, even if switching between the two operational modes.

In addition, no guards existed for trying to enable both recording and
callbacks at the same time.

Bug: 9423825

Change-Id: I7d6e6114c2e14fcfb5299b4c72ad557895cbf4b8

Fix compilation warning: incorrect format specifier for type 'ssize_t'.

warning: format '%ld' expects argument of type 'long int',
but argument 7 has type 'ssize_t {aka int}' [-Wformat]

Change-Id: I823835a26320a3abb82f8dff9fbe82e5132e7556
Signed-off-by: Scott Fan <fancp2007@gmail.com>

Reconcile with jb-mr2-zeroday-release - do not merge

Change-Id: I4de46f3bfcfb80867bbeb52a76d383f5870e74f4

am 3a4ff0b8: am d74a9ee8: Merge "libstagefright: fix the duration "00:00" for some httplive link"

* commit '3a4ff0b8bf464c9c286d63326bea42609527051e':
  libstagefright: fix the duration "00:00" for some httplive link

am d74a9ee8: Merge "libstagefright: fix the duration "00:00" for some httplive link"

* commit 'd74a9ee83fd6e033fd2260ca3777390d7e5f817a':
  libstagefright: fix the duration "00:00" for some httplive link

Merge "libstagefright: fix the duration "00:00" for some httplive link"

merge in jb-mr2-zeroday-release history after reset to jb-mr2-dev

Merge remote-tracking branch 'origin/jb-mr2-zeroday-release' into HEAD

Reconcile with jb-mr2-zeroday-release - do not merge

Change-Id: Id1c9edbb5b7b398fb863cbbd79e673ab2c10df9a

merge in jb-mr2-zeroday-release history after reset to jb-mr2-dev

am 209bbbcf: Apply EQ and bass boost first in effect chain

* commit '209bbbcf4190231f9dede758cbe77d109919f9f1':
  Apply EQ and bass boost first in effect chain

Apply EQ and bass boost first in effect chain

EQ and bass boost can improperly interact with other effects
 (e.g. virtualizer) outside of the bundle. Apply them first
 in an effect chain, which is equivalent to using them to affect
 the content of a track, rather than how a track is presented,
 when additional effects are cascaded behind EQ and BB.

Bug 8589316

Change-Id: Ifb34e46bc1969f613cdc90ee2a79af677e2fad92

am f3e21c30: am 60f3f102: Merge "Prevent AudioCommands being freed before read"

* commit 'f3e21c301b8fef49eecf9c716373fcbfbaa47670':
  Prevent AudioCommands being freed before read

am 60f3f102: Merge "Prevent AudioCommands being freed before read"

* commit '60f3f102ba2973cc31e5c7ea00d95bbf19968e8b':
  Prevent AudioCommands being freed before read

Merge "Prevent AudioCommands being freed before read"

Prevent AudioCommands being freed before read

When AudioCommandThread::threadLoop process AudioCommands it was
possible for it to delete a command where the posting thread still
hadn't read that status from it.

If a second command signaled the thread loop to continue after it
had inserted a new command while the thread loop was actually waiting
for the first command to report that it had read the status the
thread loop would continue and delete the first command.

Changed the wait condition when waiting for the calling thread to
read status to use command->mCond instead of mWaitWorkCV. This way
it's guaranteed that the signal to continue comes from the correct
thread.

Change-Id: Ia69b48cb4fdfaf8b4c83b56a197fb9f2058a92d1

am e6478de7: Camera2/3: Avoid shutdown race in callback processor.

* commit 'e6478de72a8ae1d3abb3e8c20898977c45354a4f':
  Camera2/3: Avoid shutdown race in callback processor.

Camera2/3: Avoid shutdown race in callback processor.

It's possible, during shutdown, for callback processor's heap to be
destroyed when it's about to send the callback back to the user.
Properly copy the heap reference to a local variable before unlocking
the mutex.

Bug: 9485959
Change-Id: I301347b77145f19c7ac721b9127dc74f122acce2

am 241b5279: Camera3: Power: Properly shut down unused thread.

* commit '241b52798809d8db3d369af05ace1f73f723f29b':
  Camera3: Power: Properly shut down unused thread.

Camera3: Power: Properly shut down unused thread.

Bug: 9466221
Change-Id: I9ed0740561b2e04954eebc1b8aec89cfab40a3a0

Reconcile with jb-mr2-zeroday-release - do not merge

Change-Id: I2ab98a5fccea49e0f4e6d22fddca485ad50b63b8

merge in jb-mr2-zeroday-release history after reset to jb-mr2-dev

am 26bc908b: Camera2/3: Make sure to wait for idle after still capture.

* commit '26bc908b3dafb5d80f092a22c5c71116526eae06':
  Camera2/3: Make sure to wait for idle after still capture.

am 214a17fd: Camera3: Always close HAL device even in face of errors.

* commit '214a17fd37ef85fc841d3157b1e9096e1aa1b42f':
  Camera3: Always close HAL device even in face of errors.

am 82fc2ca6: am d5018116: Merge "Fix typo."

* commit '82fc2ca6905978b4f49f2c6bf948a6872575f2cc':
  Fix typo.

am d5018116: Merge "Fix typo."

* commit 'd5018116252a3ecb491e2eaddad12180b85d1e78':
  Fix typo.

Merge "Fix typo."

Camera2/3: Make sure to wait for idle after still capture.

While this wait should be immediate, it's important to guarantee
that preview is stopped when takePicture completes, and that the
device knows itself to be idle.

Bug: 9413025
Change-Id: I1c151ba920e226107ee048918187364990f444c0

Camera3: Always close HAL device even in face of errors.

To improve robustness, make sure the HAL device instance is always
closed, even if it cannot be put into the IDLE state.

Bug: 9414167
Change-Id: I8e7035ff0afe7214951f9824e561b340cb37592e

am 1191454a: Merge "Camera3: Skipping no-ops good, not updating state machine bad." into jb-mr2-dev

* commit '1191454a3a960a1972b0f36e625796c6c1b2c9ca':
  Camera3: Skipping no-ops good, not updating state machine bad.

am c4a8d2dd: Merge "Camera2/3: Create JPEG stream unconditionally." into jb-mr2-dev

* commit 'c4a8d2ddfff6d91e493692251733f20ece3e713d':
  Camera2/3: Create JPEG stream unconditionally.

Merge "Camera3: Skipping no-ops good, not updating state machine bad." into jb-mr2-dev

Merge "Camera2/3: Create JPEG stream unconditionally." into jb-mr2-dev

Reconcile with jb-mr2-zeroday-release - do not merge

Change-Id: Id9419bf573d550f1ebebcc96fe615e1aaf8f36ef

merge in jb-mr2-zeroday-release history after reset to jb-mr2-dev

Reconcile with jb-mr2-release - do not merge

Change-Id: I024b04bf1e47cc7c10209e83f640cc6f12e03274

Camera3: Skipping no-ops good, not updating state machine bad.

If a HAL-level stream reconfigure is not needed, still need to
transition camera device to ACTIVE state in configureStreams.
Otherwise, shutdown functions will not wait for device to drain
properly.

Bug: 9413025
Change-Id: I0b15792097c7e5771015f2a362735d5f88b8c6ea