kenmirrors/freeswitch
1
0
Fork 0
mirror of https://github.com/signalwire/freeswitch.git synced 2025-04-14 08:05:37 +00:00
Code Issues Projects Releases Wiki Activity

FS-10757: [mod_conference] Race condition freeing avatar image #resolve

Browse Source
This commit is contained in:
Anthony Minessale 2017-10-25 14:20:45 -05:00
parent 3300661137
commit 03576e82b8
3 changed files with 12 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/mod/applications/mod_conference/conference_loop.c
View File

@ -832,7 +832,9 @@ void *SWITCH_THREAD_FUNC conference_loop_input(switch_thread_t *thread, void *ob
if (switch_channel_test_flag(channel, CF_VIDEO) && !conference_utils_member_test_flag(member, MFLAG_ACK_VIDEO)) {
conference_utils_member_set_flag_locked(member, MFLAG_ACK_VIDEO);
switch_mutex_lock(member->flag_mutex);
switch_img_free(&member->avatar_png_img);
switch_mutex_unlock(member->flag_mutex);
conference_video_check_avatar(member, SWITCH_FALSE);
switch_core_session_video_reinit(member->session);
conference_video_set_floor_holder(member->conference, member, SWITCH_FALSE);

2
src/mod/applications/mod_conference/conference_member.c
View File

@ -1191,9 +1191,11 @@ switch_status_t conference_member_del(conference_obj_t *conference, conference_m
last = imember;
}
switch_mutex_lock(member->flag_mutex);
switch_img_free(&member->avatar_png_img);
switch_img_free(&member->video_mute_img);
switch_img_free(&member->pcanvas_img);
switch_mutex_unlock(member->flag_mutex);
switch_thread_rwlock_unlock(member->rwlock);

8
src/mod/applications/mod_conference/conference_video.c
View File

@ -2371,8 +2371,10 @@ void conference_video_check_avatar(conference_member_t *member, switch_bool_t fo
avatar = var;
}
switch_mutex_lock(member->flag_mutex);
switch_img_free(&member->avatar_png_img);
if (avatar) {
member->avatar_png_img = switch_img_read_png(avatar, SWITCH_IMG_FMT_I420);
}
@ -2385,6 +2387,8 @@ void conference_video_check_avatar(conference_member_t *member, switch_bool_t fo
member->auto_avatar = 1;
}
switch_mutex_unlock(member->flag_mutex);
if (canvas) {
switch_mutex_unlock(canvas->mutex);
conference_video_release_canvas(&canvas);
@ -2404,7 +2408,9 @@ void conference_video_check_flush(conference_member_t *member, switch_bool_t for
if ((flushed || force) && member->auto_avatar) {
switch_channel_video_sync(member->channel);
switch_mutex_lock(member->flag_mutex);
switch_img_free(&member->avatar_png_img);
switch_mutex_unlock(member->flag_mutex);
member->avatar_patched = 0;
conference_video_reset_video_bitrate_counters(member);
member->blanks = 0;
@ -3329,6 +3335,7 @@ void *SWITCH_THREAD_FUNC conference_video_muxing_thread_run(switch_thread_t *thr
}
}
switch_mutex_lock(imember->flag_mutex);
if (imember->avatar_png_img) {
if (layer) {
if (!imember->avatar_patched || !layer->cur_img) {
@ -3342,6 +3349,7 @@ void *SWITCH_THREAD_FUNC conference_video_muxing_thread_run(switch_thread_t *thr
}
switch_img_free(&img);
}
switch_mutex_unlock(imember->flag_mutex);
if (imember->video_layer_id < 0) {
layer = NULL;