commit 72197256b85f41ff689505fc260fd82b8b71be76
author Weiyin Jiang <wjiang@codeaurora.org> Wed Oct 09 11:49:32 2019 +0800
committer Weiyin Jiang <wjiang@codeaurora.org> Wed Oct 09 11:49:32 2019 +0800
tree 8915a31ad57a3f8d522d10ddf3c11244ec4573d7
parent 15fa15d8acd81599d735c73a8dc82eaadc9a31c3

hal: fix unsigned int overflow

Fix unsigned int overflow.

CRs-Fixed: 2541532
Change-Id: I4b9448ee99c9160a233fdc8b156be228ceb91f60

hal/audio_hw.c

diff --git a/hal/audio_hw.c b/hal/audio_hw.c
index 22a97ad..3761367 100644
--- a/hal/audio_hw.c
+++ b/hal/audio_hw.c
@@ -3044,10 +3044,11 @@
                     pcm_close(in->pcm);
                     in->pcm = NULL;
                 }
-                if (pcm_open_retry_count-- == 0) {
+                if (pcm_open_retry_count == 0) {
                     ret = -EIO;
                     goto error_open;
                 }
+                pcm_open_retry_count--;
                 usleep(PROXY_OPEN_WAIT_TIME * 1000);
                 continue;
             }
@@ -3536,9 +3537,10 @@
                 pcm_close(pcm);
                 pcm = NULL;
             }
-            if (pcm_open_retry_count-- == 0)
+            if (pcm_open_retry_count == 0)
                 return NULL;
 
+            pcm_open_retry_count--;
             usleep(PROXY_OPEN_WAIT_TIME * 1000);
             continue;
         }