Bug 1253706: Fix leaks/uafs caused by repeated setting of STUN/TURN servers. r=mjf

Differential Revision: https://phabricator.services.mozilla.com/D135374
This commit is contained in:
Byron Campen [:bwc] 2022-02-08 23:37:59 +00:00
Родитель dd8c41c4d2
Коммит a2b7cf4952
1 изменённых файлов: 6 добавлений и 0 удалений

Просмотреть файл

@ -123,6 +123,7 @@ int nr_ice_ctx_set_stun_servers(nr_ice_ctx *ctx,nr_ice_stun_server *servers,int
if(ctx->stun_servers_cfg){
RFREE(ctx->stun_servers_cfg);
ctx->stun_servers_cfg=NULL;
ctx->stun_server_ct_cfg=0;
}
@ -144,7 +145,12 @@ int nr_ice_ctx_set_turn_servers(nr_ice_ctx *ctx,nr_ice_turn_server *servers,int
int _status;
if(ctx->turn_servers_cfg){
for (int i = 0; i < ctx->turn_server_ct_cfg; i++) {
RFREE(ctx->turn_servers_cfg[i].username);
r_data_destroy(&ctx->turn_servers_cfg[i].password);
}
RFREE(ctx->turn_servers_cfg);
ctx->turn_servers_cfg=NULL;
ctx->turn_server_ct_cfg=0;
}