commit 189e148409ea63f4b97c95e9cf795878165daaf7
author Xiaoyu Ye <benyxy@codeaurora.org> Thu May 07 09:59:17 2020 -0700
committer Xiaoyu Ye <benyxy@codeaurora.org> Tue May 12 06:32:17 2020 -0700
tree 50ae7d34f6c503d56d7b1b5904183f3a29140136
parent 2bd38a9ad79c9891dda331dd676f0fbdbdf035ae

st-hal: fix st session potential deadlock

In function hw_sess_cb, st_ses->lock could be left as unlocked
in switch case ST_HW_SESS_EVENT_DETECTED when "!lock_status"
condition is met and the session is not in active state. Due to
the switch case exits prematurely, st_ses->lock will not be
unlocked under this condition. Remove the "break" to avoid this
scenario in function hw_sess_cb.

Change-Id: I68af8bc46334a71325fb25fc101b9950c207b2e3

st_session.c

diff --git a/st_session.c b/st_session.c
index a32b71d..fba3b15 100644
--- a/st_session.c
+++ b/st_session.c
@@ -203,7 +203,6 @@
                 ev.payload.detected.detect_status = 3;
 
             DISPATCH_EVENT(st_ses, ev, status);
-            break;
         }
 
         if (!lock_status)