| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* This has been rearchitected in a better way, as this
feature turns out to be more difficult than it seems.
* Reverting all of this stuff and rolling it into a single commit.
This reverts commit c27a16c33c78a36482336a16199b1b8be794cea4.
This reverts commit 32ef0556ae58ff6b7c6fe6fb0a17d3ff7f01de31.
This reverts commit 489c9fb62f02e1d23d6d6c89b22f7d19c596e65e.
This reverts commit a4123803d0a0e9e0c69faa4207d357cc74a65d58.
This reverts commit e13b58b988ab642d4ae5ca6d0a89013510714956.
This reverts commit 47f8c7303c9e2054f1492b02b6c7472385c52dc9.
This reverts commit 0479d7c79a7fd6f112e8dc7e45c009cf6602dbaa.
Change-Id: Iaed9f198d806aa414c95960713e8187c98db248b
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Change Id0af9c8aed1a983095275ac03f7f59abc31594cc
using boot time for video recording start time
as default, but it is specific on devices using
qcom camera HAL3 and it breaks all devices
using camera HAL1.
Fix by changing the default behaviour when
property media.camera.ts.monotonic is not set.
And newer devices such as 8996 might need to setprop
media.camera.ts.monotonic 0 in order to work
properly when using camera HAL3.
Change-Id: Iee07ed1147713a6397fff43e3bbd4f4ed08b56b7
|
|
|
|
|
|
|
|
|
|
|
| |
If track creation fails on server side, mState/mActive is left
unitialized. After this a crash can happen when track stop()
is called because of a reference to a NULL object.
Initialize mState/mActive to specific value in initialized list
of track constructor.
Change-Id: If8c3611e3229c0c1b14b81285e07b9357fec7658
CRs-Fixed: 992608
|
|
|
|
|
|
| |
Fix invalid type in error log's format string.
Change-Id: I26aabfcdc1a03c0a7d9cd10fb3b39a660a9be03a
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- If flush is called on both A/V tracks and video
decoder is not yet created, In offload case the
first audio buffer will get flush and audio playback
starts from timestamp of second buffer, where as
video playback starts from zero leads to AV sync
issue in start and cause framedrops.
- Call seek to current position to align the Audio
and Video tracks. Seek to zero if playback has not
yet started.
Change-Id: I4f75d0aa908714ad292f348fa8bf7aa5f837f8d7
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Camera HAL3 uses boot time for buffer timestamp, rather
than system monotonic time. This leads to issues as
framework uses system monotonic time as reference start
time for timestamp adjustment.
Add change to use boot time for reference start time.
CRs-Fixed: 946735
Change-Id: Id0af9c8aed1a983095275ac03f7f59abc31594cc
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- AudioSink is reopened on AudioTearDown event as well as on
decoder output format change. These events are from different
contexts, independent of each other and can occur concurrently,
doing set of thread-unsafe operations on audioSink, which
sometimes may result in fatal error.
- Any request to openSink, after teardown event is received but
before its completed, is redundant since once teardown is
completed sink is anyway going to be opened again. Also
ignoring sink reopen while in teardown would avoid concurrent
processing of thread-unsafe operations on audioSink.
- Update nuplayer to prevent reopening of sink while teardown is
still going on.
CRs-Fixed: 978013
Change-Id: Ieb3825f133c2099e07d6e92ef101dd5156af8a20
|
|
|
|
|
|
|
|
|
|
|
| |
In non-offload usecase, mAnchorTmeMediaUs is updated
for every audio FillBufferDone. Added the same logic
for offload case, otherwise video updates max media
duration as ZERO, for clips which have small audio
data, and all the buffers are sent to DSP before first
video buffer is processed at NuPlayerRenderer level
Change-Id: Ib6e0f251207c87ad1525041d07187cef91f4dda5
|
|
|
|
|
|
| |
Fix compilation issues that appear when enabling LOG_NDEBUG.
Change-Id: I87e9e5ac66157759dd6f521fab0dd346089a011a
|
|
|
|
|
|
|
|
| |
LOG_NDEBUG is inherited indirectly from both MyHandler.h & Log.h if
no LOG_NDEBUG is defined locally within AVMediaServiceUtils. Add a
local definition to keep debug usage for this file straightforward.
Change-Id: I878cdc747e8349e937b13dd27c738d45f99dee0a
|
|
|
|
|
|
|
|
|
| |
Without this patch, writing B-frame to MediaMuxer will be blocked and
cannot return a result. This may cause an endless progress bar shown in
gallery when triming or muting some videos.
Change-Id: Ib9f902a57da470c0357ec68aab00a35fbb26fb77
CRs-Fixed: 521005
|
|
|
|
|
|
| |
- Remove Virtual for SetDataSource()
Change-Id: Ie8502297376f6663a38f95195ff8329b7a42dcb9
|
|
|
|
|
|
|
|
|
|
|
| |
Make RTSPSource to accept some video damaged access units to enhance
fault tolerance.
Disable this change as default. add the property to enable it such as:
"adb shell setprop rtsp.video.keep-damaged-au video/3gpp" for H263
"adb shell setprop rtsp.video.keep-damaged-au video/avc" for H264
Change-Id: I3b7fb4b098aba5daf149cf36dab7e9380c6d2f69
|
|
|
|
|
|
| |
Defer MPEG4 muxer creation to AVFactory.
Change-Id: If0918be77ab7f8d82c78203f371df789e3cc29b8
|
|
|
|
|
|
| |
Add support for enhancements in QSSP instrumentation code.
Change-Id: I00aa0b2bee5044d91eeeddaa43d3d28518a5ba93
|
|
|
|
|
|
|
| |
- Extend onResume in-order to implement the Audio Voice Concurrency
changes.
Change-Id: I2b02f644df23f35b6d37d07a99d74ad56470b142
|
|
|
|
|
|
|
|
|
|
| |
Adds support for the HAL_PIXEL_FORMAT_RGBX_8888 format used by
mediatek's hardware.
Fixes several CTS tests where interaction between hardware and software
encoders and decoders is tested.
Change-Id: Ie3267c7d34706c49ad415ebc04ad05f2f2ef3506
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* The edge cases, ZOMG!
* Instead of relying on effects to be automatically attached, let's
just always notify userspace and send a bit more information. This
lets the application decide if effects should be attached rather
than relying on a hard-coded configuration file.
* Perform the setup in getOutputForAttr, but do it on the command
thread so we don't block the client.
OPO-593
Change-Id: I3900b349f2e895d51fa3a3dcc2de0c4bdf6dbc08
|
|\
| |
| |
| |
| | |
Ticket: CYNGNOS-2213
Android 6.0.1 release 24
|
| |
| |
| |
| |
| |
| |
| | |
to prevent a cleared service binder from being used.
Bug: 26040840
Change-Id: Ifb5483c55b172d3553deb80dbe27f2204b86ecdb
|
| |
| |
| |
| |
| |
| |
| | |
Previous fix accidentally only fixed the fragmented read case.
Bug: 27208621
Change-Id: Ie16f1920b84c8aba613842659238fcd5925694ad
|
| |
| |
| |
| |
| | |
Bug: 26914474
Change-Id: Ie1a86e86d78058d041149fe599a4996e7f8185cf
|
| |
| |
| |
| |
| |
| | |
We moved the safteynet logging into storeMetaDataInBuffers_l
Bug: 26324358
Change-Id: I2171742b53192ebb71d546bcb8970bb3c68f366f
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Uninitialized MetadataBufferType pointer in case:
SET_INPUT_SURFACE
STORE_META_DATA_IN_BUFFERS
CREATE_INPUT_SURFACE
Fix: initialize them to kMetadataBufferTypeInvalid
Bug: 26324358
Change-Id: Ib3bcac9bedc98bc65efa39f67cdbffbd90b374c9
|
| |
| |
| |
| |
| | |
bug: 26323455
Change-Id: I25bb30d3666ab38d5150496375ed2f55ecb23ba8
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This can lead to local code execution in media server.
Fix initializes the pointer and checks the error conditions before
returning
Bug: 26403627
Change-Id: I7fa90682060148448dba01d6acbe3471d1ddb500
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- During offload (WH) to non-offload fallback (BT),
anchor time is getting updated incorrectly with
offload renderer position when it tries to start
sink using offload mode only.
- update anchor time post resume only when sink is
started successfully.
Change-Id: I2c2992afa90506e08389226964b0bd5a5be07529
CRs-Fixed: 978596
|
| |
| |
| |
| |
| |
| |
| | |
Break CC SEI parsing when payload size exceeds buffer size
to avoid a CHECK that have been seen in MTBF statistics.
Change-Id: Ifd97648678a935ac815dd616301d46f9bf583838
|
| |
| |
| |
| |
| |
| | |
unsigned integer overflow.
Change-Id: Iad5ae41f0bbfc5e837b4b78e8acaa3f9462329e6
|
| |
| |
| |
| |
| |
| |
| | |
Most of these were encountered while running a mixed sanitized/unsanitized
AMR encoder, so may not be reachable in real conditions.
Change-Id: I85af7d40214133234009323e7e64432fc1be39ca
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
There are integer overflow conditions that could occur on
vsyncsForLastFrame in VideoFrameScheduler::schedule upon a VSYNC
add or removal. Additional checks have been put in place to
remove the overflowable conditions.
Bug: 24980200
Change-Id: I7dfc25ae1d2f3e3b3382e990adb3c56518c64e8d
|
| |
| |
| |
| | |
Change-Id: I89e3a827cf566421e8dd9b6a3c842e73a19c140f
|
| |
| |
| |
| |
| |
| |
| |
| | |
Converting unix epoch time to mpeg4 time requires adding over 2B seconds,
which caused an overflow in a calculation involving time_t, which is signed.
Bug: 23574783
Change-Id: I21bacc9f5a422091f3c903fb8cf1c760fc078953
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
In the case that mNumChannels, bytesPerSample, or mSampleRate are
zero, a divide by zero occurs. None of these parameters of a WAV
file should ever be zero. Check that they aren't and return an error
otherwise.
Bug: 23285883
Change-Id: Id67b8620944405ca59572221f6f1c2b19c363e69
|
| |
| |
| |
| |
| | |
Bug: 23279597
Change-Id: Ibaa3d52e586e65230ec6df3680d9456ce873390c
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
If mFrameSize is less than or equal to getHeaderLength(), an integer underflow
will occur. This typically leads to a crash reading out of bounds in the
following code. Prevent this from happening by validating mFrameSize.
Also add NULL checks after references to ID3::Iterator::getData.
Bug: 23285887
Change-Id: I35eeda3c5349ebbd9ffb3ea49b79af6a940d1395
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
For build-system CFLAGS clean-up.
Bug: 18632512
(cherry picked from commit 765b4172f1ffb2c4e2de89163bb888fa352de664)
Change-Id: I765dc394f39f60801843851760fdf9838958d7a4
|
| |
| |
| |
| |
| | |
Change-Id: I83057565a842da31ee137fac674af496c1fc18af
Signed-off-by: Ashok Bhat <ashok.bhat@arm.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
When processing various FourCC values within MP4 media, mLastTrack is accessed
without first ensuring that a track has been encoutered. Check for NULL and
bail out instead of crashing.
Bug: 20139950
Change-Id: Ie16687024d17348f576a0e13bd60bd4d6898de91
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
After a failed write() or ioctl(), errno wasn't being checked until
after some other function calls that could also modify errno, thus
checking the wrong errno. Make sure to check it prior to doing anything
else that can modify it.
[Preconditions]
1.PC connects with phone(mtp).
[Procedures]
1.Copy a file from PC to phone.
2.Cancel the copying.
3.Recopy the file.
Change-Id: Id772fca7ccb96d3f43bd4beb210bedd8d3ac17fa
|
| |
| |
| |
| |
| |
| |
| | |
A 64-bit SIZE_MAX value cast to off64_t is always negative, causing valid
files to be rejected in 64-bit mode.
Change-Id: I8f61c19951f9c73292fa917081b8b2f3bfc405a0
|
| |
| |
| |
| |
| |
| |
| |
| | |
AOSP has a different fix.
This reverts commit 65890a73d53a25b0809b22e41cdee19e3f4aa68d.
Change-Id: I88fc05c791b9d5cef181e94d1cd3c40f0076f827
|
|\ \ |
|
| |\ \
| | |/
| | |
| | |
| | |
| | | |
https://android.googlesource.com/platform/frameworks/av into cm-13.0
Android 6.0.1 release 22
|
| | |\
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
a8ac8f617b am: e2e46f5fa1 am: 2304c606eb am: 0945199000 am: a39f981875 am: e463301aa4
am: 0dd00d9b19
* commit '0dd00d9b1986cd2cc4f45bdb96266dee01f20d72':
Fix out-of-bounds write
|
| | | |\
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
a8ac8f617b am: e2e46f5fa1 am: 2304c606eb am: 0945199000 am: a39f981875
am: e463301aa4
* commit 'e463301aa44f0ea0ae1a2399bf8800d94cf6ce3e':
Fix out-of-bounds write
|
| | | | |\
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
a8ac8f617b am: e2e46f5fa1 am: 2304c606eb am: 0945199000
am: a39f981875
* commit 'a39f9818759155a2651995d6844f9821d83e3f0d':
Fix out-of-bounds write
|
| | | | | |\
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
a8ac8f617b am: e2e46f5fa1 am: 2304c606eb
am: 0945199000
* commit '0945199000aaa5a628705de69f17d28a5c04cc36':
Fix out-of-bounds write
|
| | | | | | |\
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
a8ac8f617b am: e2e46f5fa1
am: 2304c606eb
* commit '2304c606eb2eb9e6f21fcca0e6dcd4ac8156779a':
Fix out-of-bounds write
|
| | | | | | | |\
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
am: a8ac8f617b
* commit 'a8ac8f617b2df8f07935d3e1e47189dee2381b88':
Fix out-of-bounds write
|