diff options
| author | Tilghman Lesher <tilghman@meg.abyt.es> | 2010-08-04 14:05:11 +0000 |
|---|---|---|
| committer | Tilghman Lesher <tilghman@meg.abyt.es> | 2010-08-04 14:05:11 +0000 |
| commit | ca661f4702b042acc9ec14e5220d895a83f10240 (patch) | |
| tree | 35e8026d79bf3b0800ed1dfc60c6f261415f3f52 /channels/chan_mgcp.c | |
| parent | 42490d744b195a8b630f49ed564b7deea08ccedf (diff) | |
Merged revisions 280879 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8
........
r280879 | tilghman | 2010-08-04 09:04:07 -0500 (Wed, 04 Aug 2010) | 14 lines
Check cur value before attempting a deref.
(closes issue #17775)
Reported by: svinson
Patches:
20100804__issue17775.diff.txt uploaded by tilghman (license 14)
Tested by: svinson
(closes issue #17743)
Reported by: tgruenberg
Patches:
20100804__issue17775.diff.txt uploaded by tilghman (license 14)
Tested by: tgruenberg
........
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@280880 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'channels/chan_mgcp.c')
| -rw-r--r-- | channels/chan_mgcp.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/channels/chan_mgcp.c b/channels/chan_mgcp.c index 485b39aad..a787eab72 100644 --- a/channels/chan_mgcp.c +++ b/channels/chan_mgcp.c @@ -3580,7 +3580,7 @@ static int find_and_retrans(struct mgcp_subchannel *sub, struct mgcp_request *re if (sscanf(req->identifier, "%30d", &seqno) != 1) { seqno = 0; } - for (cur = sub->parent->parent->responses, next = cur->next; cur; cur = next, next = cur->next) { + for (cur = sub->parent->parent->responses, next = cur ? cur->next : NULL; cur; cur = next, next = cur ? cur->next : NULL) { if (now - cur->whensent > RESPONSE_TIMEOUT) { /* Delete this entry */ if (prev) |