diff options
author | David Brooks <dbrooks@digium.com> | 2009-06-17 19:15:07 +0000 |
---|---|---|
committer | David Brooks <dbrooks@digium.com> | 2009-06-17 19:15:07 +0000 |
commit | ecfbab07823c7efb429838f2144fed75eb3c90bc (patch) | |
tree | 78fe1378017459955ba06bab6c8f61b076467ebc | |
parent | 9bf67151c9b2ab78ed74c8629d97b31533599e7d (diff) |
Merged revisions 201380 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4
........
r201380 | dbrooks | 2009-06-17 13:45:50 -0500 (Wed, 17 Jun 2009) | 9 lines
Checks for NULL sip_pvt pointer in chan_sip.c->acf_channel_read()
Zombie channels could be passed, and chan_sip.c wasn't checking for it.
Could crash Asterisk. Now checking for NULL pointer.
(closes issue #15330)
Reported by: okrief
Tested by: dbrooks
........
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@201381 65c4cc65-6c06-0410-ace0-fbb531ad65f3
-rw-r--r-- | channels/chan_sip.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/channels/chan_sip.c b/channels/chan_sip.c index 9e3e238b0..144b6fa1c 100644 --- a/channels/chan_sip.c +++ b/channels/chan_sip.c @@ -21046,6 +21046,10 @@ static int acf_channel_read(struct ast_channel *chan, const char *funcname, char memset(buf, 0, buflen); + if (p == NULL) { + return -1; + } + if (!strcasecmp(args.param, "peerip")) { ast_copy_string(buf, p->sa.sin_addr.s_addr ? ast_inet_ntoa(p->sa.sin_addr) : "", buflen); } else if (!strcasecmp(args.param, "recvip")) { |