commit bf821e2c39ac948d43e3fa19f6f517317e195060
author Phil Burk <philburk@google.com> Fri Apr 17 11:51:43 2020 -0700
committer Phil Burk <philburk@google.com> Fri Apr 17 14:55:24 2020 -0700
tree f6b8d1648690a9773a61fe49873ae7a3de24597d
parent 01d9c845f00cae661e1764162f36d5156871a60b

aaudio: initialize the callback buffer

This can prevent some loud noises during unit tests.

Also use std::unique_ptr

Bug: 154342174
Test: libaaudio/tests/test_various.cpp
Change-Id: I1c1dc0f4a6a29cf7d7a656392a81a325847de22d

media/libaaudio/src/client/AudioStreamInternal.h

diff --git a/media/libaaudio/src/client/AudioStreamInternal.h b/media/libaaudio/src/client/AudioStreamInternal.h
index 095f30c..42f2889 100644
--- a/media/libaaudio/src/client/AudioStreamInternal.h
+++ b/media/libaaudio/src/client/AudioStreamInternal.h
@@ -164,7 +164,7 @@
     // Offset from underlying frame position.
     int64_t                  mFramesOffsetFromService = 0; // offset for timestamps
 
-    uint8_t                 *mCallbackBuffer = nullptr;
+    std::unique_ptr<uint8_t[]> mCallbackBuffer;
     int32_t                  mCallbackFrames = 0;
 
     // The service uses this for SHARED mode.