commit cb8a2cfb1b642b10db1f3d108201b3b502efdfd7
author Ikjoon Jang <ikjn@google.com> Sun Oct 27 07:25:18 2024 +0000
committer Ikjoon Jang <ikjn@google.com> Thu Nov 28 08:46:50 2024 +0000
tree f882f1d8df96a8607f469ea8538cd52520981208
parent bd5175822e8cf650a7a562223def04d469531de2

SessionTrack: grab a CONCURRENCY lock for Deleting URSK

DeleteUrsk() is called upon receiving SESSION_DEINIT_NTF.
To ensure proper serialization, the function is marked as a critical
section. This prevents concurrent access from upper-layer write requests
and the internal DeleteUrsk() operation.

Bug: 375952239
Test: ccc session start/stop
Change-Id: I2ff40bb46083742402757119a56c921b81704e89
Merged-In: I2ff40bb46083742402757119a56c921b81704e89

halimpl/hal/sessionTrack.cc

diff --git a/halimpl/hal/sessionTrack.cc b/halimpl/hal/sessionTrack.cc
index 56d2697..1f471c9 100644
--- a/halimpl/hal/sessionTrack.cc
+++ b/halimpl/hal/sessionTrack.cc
@@ -651,7 +651,9 @@
         }
         break;
       case SessionTrackWorkType::DELETE_URSK:
+        CONCURRENCY_LOCK();
         DeleteUrsk(msg->session_info_);
+        CONCURRENCY_UNLOCK();
         break;
       case SessionTrackWorkType::STOP:
         stop_thread = true;