From 814941997f3f4b53064511b658cb510cafffe117 Mon Sep 17 00:00:00 2001 From: Dave Sparks Date: Tue, 26 May 2009 14:39:29 -0700 Subject: [PATCH] setVolume did not check interface descriptor DO NOT MERGE --- media/libmedia/IMediaPlayer.cpp | 2 ++ 1 file changed, 2 insertions(+) diff --git a/media/libmedia/IMediaPlayer.cpp b/media/libmedia/IMediaPlayer.cpp index f37519f42bff7..f18765a10843c 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;