summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJames Dong <jdong@google.com>2012-12-20 18:25:12 -0800
committerGerrit Code Review <noreply-gerritcodereview@google.com>2012-12-20 18:25:12 -0800
commitb54cedabdf0261211241e2f3af09c75cffd911ed (patch)
tree4a6ec04010d7c3a7f4b341131ba50612bc36658e
parent2a55b5964c6ae167cc1bc57eca5ca7a1c4aa452c (diff)
parent8af5fe5a2431522a7d30bc546dcd31c0c64db70c (diff)
Merge "Use default values when MPEG4 audio config parsing fails."
-rw-r--r--media/libstagefright/rtsp/AMPEG4AudioAssembler.cpp13
1 files changed, 10 insertions, 3 deletions
diff --git a/media/libstagefright/rtsp/AMPEG4AudioAssembler.cpp b/media/libstagefright/rtsp/AMPEG4AudioAssembler.cpp
index f9a44f02..aa8ffc65 100644
--- a/media/libstagefright/rtsp/AMPEG4AudioAssembler.cpp
+++ b/media/libstagefright/rtsp/AMPEG4AudioAssembler.cpp
@@ -311,9 +311,7 @@ static status_t parseStreamMuxConfig(
case 2:
{
- // reserved
- TRESPASS();
- break;
+ return ERROR_UNSUPPORTED;
}
case 3:
@@ -460,6 +458,15 @@ AMPEG4AudioAssembler::AMPEG4AudioAssembler(
&mFixedFrameLength,
&mOtherDataPresent, &mOtherDataLenBits);
+ if (err == ERROR_UNSUPPORTED) {
+ ALOGW("Failed to parse stream mux config, using default values for playback.");
+ mMuxConfigPresent = false;
+ mNumSubFrames = 0;
+ mFrameLengthType = 0;
+ mOtherDataPresent = false;
+ mOtherDataLenBits = 0;
+ return;
+ }
CHECK_EQ(err, (status_t)NO_ERROR);
}