Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Skip to content
Commit 16ce588a authored by Jiangli Liu's avatar Jiangli Liu Committed by Tyler Gunn
Browse files

Add synchronized lock when set call state ANSWERED

Root cause:
When user answer an incoming call, call state will be changed from
RINGING to ANSWERED, then changed from ANSWERED to ACTIVE.
CallsManager set call state ANSWERED and ACTIVE in different threads.
If CallsManager set the call state ANSWERED and ACTIVE quickly,
sometimes start to set state ACTIVE when setting ANSWERED is running,
This can lead to confusion in the state of the call,
state ACTIVE maybe not set successfully.

solution:
Add synchronized lock when set call state ANSWERED, this ensures that
setting ANSWERED succeeds first, and then start to set ACTIVE.

Bug: 136984649
Test: ran the CTS telecom test 100 times and it passed 100 times.
Change-Id: I953f269086e0f9370bf834284f7504d78323a1d6
parent 0b800170
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment