| commit | 7bae6a0b635b142cfba9d4c4612e1e114f85b842 | [log] [tgz] |
|---|---|---|
| author | Mikhail Naganov <mnaganov@google.com> | Mon Apr 24 10:44:08 2017 -0700 |
| committer | Mikhail Naganov <mnaganov@google.com> | Mon Apr 24 10:45:45 2017 -0700 |
| tree | 9cca53e0895c578fcbaba34fab40c42973b0af36 | |
| parent | a190e88f10454a111e88b9e3b8f2311be919af17 [diff] [blame] |
audiohal: Fix handling of invalid arguments in 'debugDump' methods Add checks for validity of the passed in native handle. Bug: 37590777 Change-Id: If906f0086c3442819e1844cc521215d5b74af015 Test: VTS
diff --git a/audio/effect/2.0/default/EffectsFactory.cpp b/audio/effect/2.0/default/EffectsFactory.cpp index 08d92bd..922a922 100644 --- a/audio/effect/2.0/default/EffectsFactory.cpp +++ b/audio/effect/2.0/default/EffectsFactory.cpp
@@ -186,7 +186,7 @@ } Return<void> EffectsFactory::debugDump(const hidl_handle& fd) { - if (fd->numFds == 1) { + if (fd.getNativeHandle() != nullptr && fd->numFds == 1) { EffectDumpEffects(fd->data[0]); } return Void();