commit 978eb819b860bc121477d9ea2cee8af40eaaa1d1
author qctecmdr <qctecmdr@localhost> Wed Sep 13 23:15:07 2023 -0700
committer Gerrit - the friendly Code Review server <code-review@localhost> Wed Sep 13 23:15:07 2023 -0700
tree e46d2345f5b19c4011af078147b27f7184e36209
parent 915f207e8b337ce9d60c3fa1c47be35578cdce56
parent 81a451c0462e6d5bf5966425bc7b89a029a58f0e

Merge "pal: Use increase/decrease StreamUserCounter to protect getTimestamp"

Pal.cpp

diff --git a/Pal.cpp b/Pal.cpp
index 46b3b86..e0fc84f 100644
--- a/Pal.cpp
+++ b/Pal.cpp
@@ -892,13 +892,19 @@
     }
 
     rm->lockActiveStream();
-    if (rm->isActiveStream(stream_handle)) {
-        s =  reinterpret_cast<Stream *>(stream_handle);
-        status = s->getTimestamp(stime);
-    } else {
-        PAL_ERR(LOG_TAG, "stream handle in stale state.\n");
+    s =  reinterpret_cast<Stream *>(stream_handle);
+    status = rm->increaseStreamUserCounter(s);
+    if (0 != status) {
+        rm->unlockActiveStream();
+        PAL_ERR(LOG_TAG, "failed to increase stream user count");
+        return status;
     }
     rm->unlockActiveStream();
+    status = s->getTimestamp(stime);
+
+    rm->lockActiveStream();
+    rm->decreaseStreamUserCounter(s);
+    rm->unlockActiveStream();
 
     if (0 != status) {
         PAL_ERR(LOG_TAG, "pal_get_timestamp failed with status %d\n", status);