kenmirrors
/
freeswitch
mirror of https://github.com/signalwire/freeswitch.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

[mod_sofia] I cannot certify that on SWITCH_MESSAGE_INDICATE_TRANSCODING_NECESSARY 

that we should be following this code path, but it does not look like
there's anything that gets changed when we enter sofia_receive_message()
in this case.  Thus there's no sense in locking tech_pvt->sofia_mutex ,
thus avoiding the deadlock described in issue #2290.
This commit is contained in:
Yossi Neiman 2024-02-14 20:12:04 -06:00
parent 9df3076f29
commit 6f1cc570d5
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/mod/endpoints/mod_sofia/mod_sofia.c
View File

@ -1482,6 +1482,9 @@ static switch_status_t sofia_receive_message(switch_core_session_t *session, swi
sofia_clear_flag(tech_pvt, TFLAG_EARLY_MEDIA);
}
goto end;
case SWITCH_MESSAGE_INDICATE_TRANSCODING_NECESSARY:
// There doesn't seem to be anything needing to be done, so let's skip the mutex deadlock in issue 2290
goto end;
case SWITCH_MESSAGE_INDICATE_ANSWER:
case SWITCH_MESSAGE_INDICATE_PROGRESS:
{