summaryrefslogtreecommitdiffstats
path: root/media/libmedia/mediaplayer.cpp
diff options
context:
space:
mode:
authorMarco Nelissen <marcone@google.com>2010-03-10 10:53:16 -0800
committerMarco Nelissen <marcone@google.com>2010-03-10 10:55:09 -0800
commit83ff1438d2d1d5dbf39ca5e6f2e4fa1799e7ba80 (patch)
tree4f1398095d0a69f4aa46cf533137c51ada1599fa /media/libmedia/mediaplayer.cpp
parentb699c595d78e8cd7f60c2c5102d40db07b18926b (diff)
downloadframeworks_av-83ff1438d2d1d5dbf39ca5e6f2e4fa1799e7ba80.zip
frameworks_av-83ff1438d2d1d5dbf39ca5e6f2e4fa1799e7ba80.tar.gz
frameworks_av-83ff1438d2d1d5dbf39ca5e6f2e4fa1799e7ba80.tar.bz2
MEDIA_PLAYER_STATE_ERROR is not a bitmask
(b/2502881) Change-Id: I08e427eb2c36f5d70e40f9aeb8638fa8262de989
Diffstat (limited to 'media/libmedia/mediaplayer.cpp')
-rw-r--r--media/libmedia/mediaplayer.cpp3
1 files changed, 2 insertions, 1 deletions
diff --git a/media/libmedia/mediaplayer.cpp b/media/libmedia/mediaplayer.cpp
index d2cec0c..f623295 100644
--- a/media/libmedia/mediaplayer.cpp
+++ b/media/libmedia/mediaplayer.cpp
@@ -104,7 +104,8 @@ status_t MediaPlayer::setDataSource(const sp<IMediaPlayer>& player)
{ // scope for the lock
Mutex::Autolock _l(mLock);
- if ( !( mCurrentState & ( MEDIA_PLAYER_IDLE | MEDIA_PLAYER_STATE_ERROR ) ) ) {
+ if ( !( (mCurrentState & MEDIA_PLAYER_IDLE) ||
+ (mCurrentState == MEDIA_PLAYER_STATE_ERROR ) ) ) {
LOGE("setDataSource called in state %d", mCurrentState);
return INVALID_OPERATION;
}