diff options
author | Wei Jia <wjia@google.com> | 2015-09-11 13:54:59 +0000 |
---|---|---|
committer | Android (Google) Code Review <android-gerrit@google.com> | 2015-09-11 13:54:59 +0000 |
commit | 741a335126a562ae8f5883cc0b02ba72f40e56f3 (patch) | |
tree | 2df1757115ff30b3f7d7405c5cb94e4ee5e51082 /media | |
parent | b659b4c7e1405056d189b44bd96036b6500dd378 (diff) | |
parent | 5f2a76f02f6d88825f8c164b4d9f7f310583eee1 (diff) | |
download | frameworks_av-741a335126a562ae8f5883cc0b02ba72f40e56f3.zip frameworks_av-741a335126a562ae8f5883cc0b02ba72f40e56f3.tar.gz frameworks_av-741a335126a562ae8f5883cc0b02ba72f40e56f3.tar.bz2 |
Merge "DO NOT MERGE - IAudioFlinger: clear config before reading it from parcel." into klp-dev
Diffstat (limited to 'media')
-rw-r--r-- | media/libmedia/IAudioFlinger.cpp | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/media/libmedia/IAudioFlinger.cpp b/media/libmedia/IAudioFlinger.cpp index acfaea0..adc7186 100644 --- a/media/libmedia/IAudioFlinger.cpp +++ b/media/libmedia/IAudioFlinger.cpp @@ -1080,8 +1080,10 @@ status_t BnAudioFlinger::onTransact( } case CREATE_EFFECT: { CHECK_INTERFACE(IAudioFlinger, data, reply); - effect_descriptor_t desc; - data.read(&desc, sizeof(effect_descriptor_t)); + effect_descriptor_t desc = {}; + if (data.read(&desc, sizeof(effect_descriptor_t)) != NO_ERROR) { + ALOGE("b/23905951"); + } sp<IEffectClient> client = interface_cast<IEffectClient>(data.readStrongBinder()); int32_t priority = data.readInt32(); audio_io_handle_t output = (audio_io_handle_t) data.readInt32(); |