Merged revisions 114120 via svnmerge from
authorJason Parker <jparker@digium.com>
Mon, 14 Apr 2008 18:34:17 +0000 (18:34 +0000)
committerJason Parker <jparker@digium.com>
Mon, 14 Apr 2008 18:34:17 +0000 (18:34 +0000)
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r114120 | qwell | 2008-04-14 13:31:57 -0500 (Mon, 14 Apr 2008) | 7 lines

The call_token on the pvt can occasionally be NULL, causing a crash.

If it is NULL, we can skip this channel, since it can't the one we're looking for.

(closes issue #9299)
Reported by: vazir

........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@114121 65c4cc65-6c06-0410-ace0-fbb531ad65f3

channels/chan_h323.c

index 140c06f..a3f6ee6 100644 (file)
@@ -1148,7 +1148,7 @@ static struct oh323_pvt *find_call_locked(int call_reference, const char *token)
        while(pvt) {
                if (!pvt->needdestroy && ((signed int)pvt->cd.call_reference == call_reference)) {
                        /* Found the call */
-                       if ((token != NULL) && (!strcmp(pvt->cd.call_token, token))) {
+                       if ((token != NULL) && (pvt->cd.call_token != NULL) && (!strcmp(pvt->cd.call_token, token))) {
                                ast_mutex_lock(&pvt->lock);
                                ast_mutex_unlock(&iflock);
                                return pvt;