commit b2178b9248827b87a65441ebc37525e8cbbbb010
author Mikhail Naganov <mnaganov@google.com> Wed Dec 11 15:04:43 2019 -0800
committer Mikhail Naganov <mnaganov@google.com> Wed Dec 11 15:07:47 2019 -0800
tree 949ce592c432179d48d23ecbbd70a4f67445a7a9
parent b524bbb92242c23421bb723b64a4f65266146bea

audio: Add check for MMap buffer size to the default implementation

Add documentation and check that 'minSizeFrames' parameter
is a positive value. Previously a similar check was added
to Pixel HAL.

Bug: 141989952
Test: atest VtsHalAudioV6_0TargetTest
Change-Id: I526431fc214f7f5bc424f0c9e4c890cf918ca2b2

audio/6.0/IStream.hal

diff --git a/audio/6.0/IStream.hal b/audio/6.0/IStream.hal
index d7d3c84..2ea1ab3 100644
--- a/audio/6.0/IStream.hal
+++ b/audio/6.0/IStream.hal
@@ -274,6 +274,7 @@
      *
      * @param minSizeFrames minimum buffer size requested. The actual buffer
      *                     size returned in struct MmapBufferInfo can be larger.
+     *                     The size must be a positive value.
      * @return retval OK in case the success.
      *                NOT_SUPPORTED on non mmap mode streams
      *                NOT_INITIALIZED in case of memory allocation error