summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAndreas Huber <andih@google.com>2011-11-11 11:48:00 -0800
committerAndroid Git Automerger <android-git-automerger@android.com>2011-11-11 11:48:00 -0800
commit2e56cfcbc6072342813172c4a1bb9adb3f728c63 (patch)
treee9de870ee19df7fc19a009cf084e4742acf5602a
parent8ae0f5562ae79fed534f12c340fdff7114c12c38 (diff)
parent73b1fd56d99b356b0effe8cf96ecf7446beb207f (diff)
downloadframeworks_av-2e56cfcbc6072342813172c4a1bb9adb3f728c63.zip
frameworks_av-2e56cfcbc6072342813172c4a1bb9adb3f728c63.tar.gz
frameworks_av-2e56cfcbc6072342813172c4a1bb9adb3f728c63.tar.bz2
am cd556b82: Merge "Instead of asserting, signal a runtime error if the session doesn\'t contain" into ics-mr1
* commit 'cd556b82fd12dceb49e9d6543f80212721b220ec': Instead of asserting, signal a runtime error if the session doesn't contain
-rw-r--r--media/libstagefright/rtsp/MyHandler.h17
1 files changed, 15 insertions, 2 deletions
diff --git a/media/libstagefright/rtsp/MyHandler.h b/media/libstagefright/rtsp/MyHandler.h
index 6a5efa4..0dd93be 100644
--- a/media/libstagefright/rtsp/MyHandler.h
+++ b/media/libstagefright/rtsp/MyHandler.h
@@ -463,8 +463,17 @@ struct MyHandler : public AHandler {
mBaseURL = tmp;
}
- CHECK_GT(mSessionDesc->countTracks(), 1u);
- setupTrack(1);
+ if (mSessionDesc->countTracks() < 2) {
+ // There's no actual tracks in this session.
+ // The first "track" is merely session meta
+ // data.
+
+ LOGW("Session doesn't contain any playable "
+ "tracks. Aborting.");
+ result = ERROR_UNSUPPORTED;
+ } else {
+ setupTrack(1);
+ }
}
}
}
@@ -783,9 +792,13 @@ struct MyHandler : public AHandler {
}
if (mNumAccessUnitsReceived == 0) {
+#if 0
LOGI("stream ended? aborting.");
(new AMessage('abor', id()))->post();
break;
+#else
+ LOGI("haven't seen an AU in a looong time.");
+#endif
}
mNumAccessUnitsReceived = 0;