You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Hi everyone.
In d7c9ef0 commit few blocks of code was added to janus_videoroom.c if(audiocodec && json_string_value(json_object_get(msg->jsep, "sdp")) != NULL) { ...... }
... for audiocodec and videocodec` (please see the commit for exact diff).
In goto error subsection of this code you only janus_refcount_decrease(&publisher->ref);, while in fact there should be additional janus_mutex_unlock(&publisher->room->mutex); janus_refcount_decrease(&publisher->room->ref);
This provoke deadlock in VideoRoom plugin in such error conditions.
The same apply to if(videocodec ... section few lines below.
The text was updated successfully, but these errors were encountered:
Hi everyone.
In d7c9ef0 commit few blocks of code was added to
janus_videoroom.c
if(audiocodec && json_string_value(json_object_get(msg->jsep, "sdp")) != NULL) { ...... }
... for
audiocodec
and videocodec` (please see the commit for exact diff).In
goto error
subsection of this code you onlyjanus_refcount_decrease(&publisher->ref);
, while in fact there should be additionaljanus_mutex_unlock(&publisher->room->mutex); janus_refcount_decrease(&publisher->room->ref);
This provoke deadlock in VideoRoom plugin in such error conditions.
The same apply to
if(videocodec ...
section few lines below.The text was updated successfully, but these errors were encountered: