diff options
author | Dave Sparks <davidsparks@android.com> | 2009-05-26 14:39:29 -0700 |
---|---|---|
committer | Dave Sparks <davidsparks@android.com> | 2009-05-26 14:39:29 -0700 |
commit | 172fb9ab347e146642234f0bd6f223d351e9fd0b (patch) | |
tree | faefb31f15b77dc35f6043a6535317d92a12bc17 | |
parent | f5879c1448cc6aebc51b26d3ec2399d66144f8f4 (diff) | |
download | frameworks_av-172fb9ab347e146642234f0bd6f223d351e9fd0b.zip frameworks_av-172fb9ab347e146642234f0bd6f223d351e9fd0b.tar.gz frameworks_av-172fb9ab347e146642234f0bd6f223d351e9fd0b.tar.bz2 |
setVolume did not check interface descriptor
DO NOT MERGE
-rw-r--r-- | media/libmedia/IMediaPlayer.cpp | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/media/libmedia/IMediaPlayer.cpp b/media/libmedia/IMediaPlayer.cpp index f37519f..f18765a 100644 --- a/media/libmedia/IMediaPlayer.cpp +++ b/media/libmedia/IMediaPlayer.cpp @@ -164,6 +164,7 @@ public: status_t setVolume(float leftVolume, float rightVolume) { Parcel data, reply; + data.writeInterfaceToken(IMediaPlayer::getInterfaceDescriptor()); data.writeFloat(leftVolume); data.writeFloat(rightVolume); remote()->transact(SET_VOLUME, data, &reply); @@ -261,6 +262,7 @@ status_t BnMediaPlayer::onTransact( return NO_ERROR; } break; case SET_VOLUME: { + CHECK_INTERFACE(IMediaPlayer, data, reply); reply->writeInt32(setVolume(data.readFloat(), data.readFloat())); return NO_ERROR; } break; |