commit 7171da26f8093938ba465a100edfd5616fdf512c
author Andy Hung <hunga@google.com> Tue Mar 08 16:58:42 2016 -0800
committer Andy Hung <hunga@google.com> Fri Mar 11 09:42:31 2016 -0800
tree b2443b4a4e5931d9720a9adaeff84585802340ae
parent 188b6223c8baaba6a44c779783a878b4459a0642

Fix unsigned long cast for 64 bit

Bug: 27479136
Change-Id: I3ef77f829d42ff410752aa3f514337829d01c266

hal/audio_hw.c

diff --git a/hal/audio_hw.c b/hal/audio_hw.c
index 296aaab..eb84183 100644
--- a/hal/audio_hw.c
+++ b/hal/audio_hw.c
@@ -1724,8 +1724,10 @@
     if ((out->usecase == USECASE_AUDIO_PLAYBACK_OFFLOAD) && (dsp_frames != NULL)) {
         lock_output_stream(out);
         if (out->compr != NULL) {
-            compress_get_tstamp(out->compr, (unsigned long *)dsp_frames,
-                    &out->sample_rate);
+            unsigned long frames = 0;
+            // TODO: check return value
+            compress_get_tstamp(out->compr, &frames, &out->sample_rate);
+            *dsp_frames = (uint32_t)frames;
             ALOGVV("%s rendered frames %d sample_rate %d",
                    __func__, *dsp_frames, out->sample_rate);
         }
@@ -1757,13 +1759,14 @@
                                    uint64_t *frames, struct timespec *timestamp)
 {
     struct stream_out *out = (struct stream_out *)stream;
-    int ret = -1;
+    int ret = -EINVAL;
     unsigned long dsp_frames;
 
     lock_output_stream(out);
 
     if (out->usecase == USECASE_AUDIO_PLAYBACK_OFFLOAD) {
         if (out->compr != NULL) {
+            // TODO: check return value
             compress_get_tstamp(out->compr, &dsp_frames,
                     &out->sample_rate);
             ALOGVV("%s rendered frames %ld sample_rate %d",