commit c11ee3f5a2a38fc4a4f27190c7e62e67150bffab
author Arun Mirpuri <quic_amirpuri@quicinc.com> Mon Aug 29 16:32:24 2022 -0700
committer Bruce Levy <quic_blevy@quicinc.com> Sat Sep 10 00:18:26 2022 -0700
tree db5e924e7b548c468c1278f6a54f53dede2fd638
parent 25a3cefee024385d978afaa3c54398035938862a

agm: Fix possible subtraction overflow error

Fix possible subtraction overflow error when computing
the difference in wall clock times between cached and current time

Change-Id: I97eed21bc0fdf303f5301ca4c81dc4e72cd4960f

plugins/tinyalsa/src/agm_pcm_plugin.c

diff --git a/plugins/tinyalsa/src/agm_pcm_plugin.c b/plugins/tinyalsa/src/agm_pcm_plugin.c
index b8f375f..1dac129 100644
--- a/plugins/tinyalsa/src/agm_pcm_plugin.c
+++ b/plugins/tinyalsa/src/agm_pcm_plugin.c
@@ -365,7 +365,7 @@
         new_hw_ptr = hw_base + pos;
 
         // Set delta_wall_clk_us only if cached wall clk is non-zero
-        if (priv->pos_buf->wall_clk_msw || priv->pos_buf->wall_clk_lsw) {
+        if (wall_clk_msw != 0 || wall_clk_lsw != 0) {
                 delta_wall_clk_us = (int64_t)((((uint64_t)wall_clk_msw) << 32 | wall_clk_lsw) -
                                         (((uint64_t)priv->pos_buf->wall_clk_msw) << 32 |
                                          priv->pos_buf->wall_clk_lsw));