kenmirrors/asterisk
1
0
Fork 0
mirror of https://github.com/asterisk/asterisk.git synced 2025-10-27 06:31:54 +00:00
Code Issues Packages Projects Releases Wiki Activity

When using MGCP realtime gateway definitions, random crashes occur.

Browse Source 
Fixed incorrect linked list node removal for realtime gateways.

(closes issue #18291)
Reported by: nahuelgreco
Patches:
      dangling-pointers-when-pruning.patch uploaded by nahuelgreco (license 162)


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@315349 65c4cc65-6c06-0410-ace0-fbb531ad65f3
This commit is contained in:
Richard Mudgett
2011-04-25 21:49:00 +00:00
parent 378e383b5e
commit ced679eef9
1 changed files with 3 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
channels/chan_mgcp.c
View File

@@ -3754,7 +3754,7 @@ static void *do_monitor(void *data)
{
int res;
int reloading;
struct mgcp_gateway *g, *gprev, *gnext;
struct mgcp_gateway *g, *gprev;
/*struct mgcp_gateway *g;*/
/*struct mgcp_endpoint *e;*/
/*time_t thispass = 0, lastpass = 0;*/
@@ -3824,12 +3824,10 @@ static void *do_monitor(void *data)
g = gateways;
gprev = NULL;
while(g) {
gnext = g->next;
if(g->realtime) {
if(mgcp_prune_realtime_gateway(g)) {
if(gprev) {
gprev->next = gnext;
gprev = g;
gprev->next = g->next;
} else {
gateways = g->next;
}
@@ -3843,7 +3841,7 @@ static void *do_monitor(void *data)
} else {
gprev = g;
}
g = gnext;
g = g->next;
}
ast_mutex_unlock(&gatelock);
lastrun = time(NULL);