diff options
| author | James Dong <jdong@google.com> | 2012-11-16 14:31:15 -0800 |
|---|---|---|
| committer | James Dong <jdong@google.com> | 2012-11-16 14:33:57 -0800 |
| commit | 5834181d3f168acb8ff4bf3eff1fd1186afb0bd4 (patch) | |
| tree | 751942479f88b6a87783b8c54c37721c0c4cdfbc /media/libmediaplayerservice/nuplayer/RTSPSource.cpp | |
| parent | 991012a8b887455cb44df782c49c610bb7cf9ec7 (diff) | |
| download | frameworks_av-5834181d3f168acb8ff4bf3eff1fd1186afb0bd4.zip frameworks_av-5834181d3f168acb8ff4bf3eff1fd1186afb0bd4.tar.gz frameworks_av-5834181d3f168acb8ff4bf3eff1fd1186afb0bd4.tar.bz2 | |
Fix a crash when the stop might be called due to some error before start in RTSPSource
o related-to-bug: 7507224
Change-Id: Ic8bfec13097b824ba337a01c9b00c98af2a33f43
Diffstat (limited to 'media/libmediaplayerservice/nuplayer/RTSPSource.cpp')
| -rw-r--r-- | media/libmediaplayerservice/nuplayer/RTSPSource.cpp | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/media/libmediaplayerservice/nuplayer/RTSPSource.cpp b/media/libmediaplayerservice/nuplayer/RTSPSource.cpp index 5a7a785..6df2ddd 100644 --- a/media/libmediaplayerservice/nuplayer/RTSPSource.cpp +++ b/media/libmediaplayerservice/nuplayer/RTSPSource.cpp @@ -57,9 +57,7 @@ NuPlayer::RTSPSource::RTSPSource( } NuPlayer::RTSPSource::~RTSPSource() { - if (mLooper != NULL) { - mLooper->stop(); - } + mLooper->stop(); } void NuPlayer::RTSPSource::start() { @@ -86,6 +84,9 @@ void NuPlayer::RTSPSource::start() { } void NuPlayer::RTSPSource::stop() { + if (mLooper == NULL) { + return; + } sp<AMessage> msg = new AMessage(kWhatDisconnect, mReflector->id()); sp<AMessage> dummy; |