Closed
Bug 1089469
Opened 10 years ago
Closed 7 years ago
MediaCodecReader crashes while parsing media files on Flame.
Categories
(Core :: Audio/Video: Playback, defect)
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: brsun, Unassigned)
References
Details
Sometimes (not always) Flame KK crashes while parsing media files:
- https://www.codeaurora.org/cgit/quic/la/platform/frameworks/av/tree/media/libstagefright/ACodec.cpp?h=b2g_kk_3.5#n2630
============================
ACodec::sendFormatChange() {
...
CHECK_GE(rect.nLeft, 0);
CHECK_GE(rect.nTop, 0);
CHECK_GE(rect.nWidth, 0u);
CHECK_GE(rect.nHeight, 0u);
CHECK_LE(rect.nLeft + rect.nWidth - 1, videoDef->nFrameWidth); // <-- Crash here!!!
CHECK_LE(rect.nTop + rect.nHeight - 1, videoDef->nFrameHeight);
...
}
============================
In this case, rect.* are 0, and videoDef->nFrame* are 0 too. So ACodec crashes due to comparing unsigned -1 with 0.
|
||
Updated•9 years ago
|
Component: Audio/Video → Audio/Video: Playback
|
||
Updated•7 years ago
|
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•