Merge change 2468 into donut
* changes: setVolume did not check interface descriptor DO NOT MERGE
This commit is contained in:
@@ -164,6 +164,7 @@ public:
|
|||||||
status_t setVolume(float leftVolume, float rightVolume)
|
status_t setVolume(float leftVolume, float rightVolume)
|
||||||
{
|
{
|
||||||
Parcel data, reply;
|
Parcel data, reply;
|
||||||
|
data.writeInterfaceToken(IMediaPlayer::getInterfaceDescriptor());
|
||||||
data.writeFloat(leftVolume);
|
data.writeFloat(leftVolume);
|
||||||
data.writeFloat(rightVolume);
|
data.writeFloat(rightVolume);
|
||||||
remote()->transact(SET_VOLUME, data, &reply);
|
remote()->transact(SET_VOLUME, data, &reply);
|
||||||
@@ -261,6 +262,7 @@ status_t BnMediaPlayer::onTransact(
|
|||||||
return NO_ERROR;
|
return NO_ERROR;
|
||||||
} break;
|
} break;
|
||||||
case SET_VOLUME: {
|
case SET_VOLUME: {
|
||||||
|
CHECK_INTERFACE(IMediaPlayer, data, reply);
|
||||||
reply->writeInt32(setVolume(data.readFloat(), data.readFloat()));
|
reply->writeInt32(setVolume(data.readFloat(), data.readFloat()));
|
||||||
return NO_ERROR;
|
return NO_ERROR;
|
||||||
} break;
|
} break;
|
||||||
|
|||||||
Reference in New Issue
Block a user