[asterisk-commits] may: branch 10 r369147 - in /branches/10: ./ addons/ooh323c/src/

[SVN commits to the Asterisk project]

Author: may
Date: Wed Jun 20 12:36:27 2012
New Revision: 369147

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=369147
Log:
fix locking issue on empty callList
(issue ASTERISK-19298)
Reported by:
        Dmitry Melekhov
Patches:
        ASTERISK-18322-2.patch
........

Merged revisions 369146 from http://svn.asterisk.org/svn/asterisk/branches/1.8

Modified:
    branches/10/   (props changed)
    branches/10/addons/ooh323c/src/ooCalls.c
    branches/10/addons/ooh323c/src/ooq931.c

Propchange: branches/10/
------------------------------------------------------------------------------
Binary property 'branch-1.8-merged' - no diff available.

Modified: branches/10/addons/ooh323c/src/ooCalls.c
URL: http://svnview.digium.com/svn/asterisk/branches/10/addons/ooh323c/src/ooCalls.c?view=diff&rev=369147&r1=369146&r2=369147
==============================================================================
--- branches/10/addons/ooh323c/src/ooCalls.c (original)
+++ branches/10/addons/ooh323c/src/ooCalls.c Wed Jun 20 12:36:27 2012
@@ -265,14 +265,13 @@
 
 int ooRemoveCallFromList (OOH323CallData *call)
 {
-   if(!call)
+   if(!call || !gH323ep.callList)
       return OO_OK;
 
    ast_mutex_lock(&callListLock);
 
    OOTRACEINFO3("Removing call %lx: %s\n", call, call->callToken);
 
-   if (!gH323ep.callList) return OO_OK;
    if(call == gH323ep.callList)
    {
       if(!call->next)

Modified: branches/10/addons/ooh323c/src/ooq931.c
URL: http://svnview.digium.com/svn/asterisk/branches/10/addons/ooh323c/src/ooq931.c?view=diff&rev=369147&r1=369146&r2=369147
==============================================================================
--- branches/10/addons/ooh323c/src/ooq931.c (original)
+++ branches/10/addons/ooh323c/src/ooq931.c Wed Jun 20 12:36:27 2012
@@ -2373,7 +2373,7 @@
       else
          OO_CLRFLAG(call->flags, OO_M_TUNNELING);
 
-      if(opts->disableGk)
+      if(opts->disableGk || gH323ep.gkClient == NULL)
          OO_SETFLAG(call->flags, OO_M_DISABLEGK);
       else
          OO_CLRFLAG(call->flags, OO_M_DISABLEGK);