[asterisk-addons-commits] trunk - r188 in /trunk/asterisk-ooh323c:
./ ooh323c/src/ src/
asterisk-addons-commits at lists.digium.com
asterisk-addons-commits at lists.digium.com
Wed Jan 18 16:11:15 MST 2006
Author: objsys
Date: Wed Jan 18 17:11:09 2006
New Revision: 188
URL: http://svn.digium.com/view/asterisk-addons?rev=188&view=rev
Log:
G.729 codec support update, well formatted code
Modified:
trunk/asterisk-ooh323c/ChangeLog
trunk/asterisk-ooh323c/README
trunk/asterisk-ooh323c/config.h
trunk/asterisk-ooh323c/configure
trunk/asterisk-ooh323c/configure.in
trunk/asterisk-ooh323c/ooh323c/src/eventHandler.c
trunk/asterisk-ooh323c/ooh323c/src/ooCalls.c
trunk/asterisk-ooh323c/ooh323c/src/ooCalls.h
trunk/asterisk-ooh323c/ooh323c/src/ooCapability.c
trunk/asterisk-ooh323c/ooh323c/src/ooCapability.h
trunk/asterisk-ooh323c/ooh323c/src/ooCmdChannel.c
trunk/asterisk-ooh323c/ooh323c/src/ooDateTime.c
trunk/asterisk-ooh323c/ooh323c/src/ooGkClient.c
trunk/asterisk-ooh323c/ooh323c/src/ooGkClient.h
trunk/asterisk-ooh323c/ooh323c/src/ooLogChan.c
trunk/asterisk-ooh323c/ooh323c/src/ooSocket.c
trunk/asterisk-ooh323c/ooh323c/src/ooSocket.h
trunk/asterisk-ooh323c/ooh323c/src/ooStackCmds.c
trunk/asterisk-ooh323c/ooh323c/src/ooTimer.c
trunk/asterisk-ooh323c/ooh323c/src/ooTimer.h
trunk/asterisk-ooh323c/ooh323c/src/oochannels.c
trunk/asterisk-ooh323c/ooh323c/src/ooh245.c
trunk/asterisk-ooh323c/ooh323c/src/ooh245.h
trunk/asterisk-ooh323c/ooh323c/src/ooh323.c
trunk/asterisk-ooh323c/ooh323c/src/ooh323ep.c
trunk/asterisk-ooh323c/ooh323c/src/ooh323ep.h
trunk/asterisk-ooh323c/ooh323c/src/ooports.c
trunk/asterisk-ooh323c/ooh323c/src/ooq931.c
trunk/asterisk-ooh323c/ooh323c/src/ootrace.c
trunk/asterisk-ooh323c/ooh323c/src/ootypes.h
trunk/asterisk-ooh323c/src/chan_h323.c
trunk/asterisk-ooh323c/src/ooh323cDriver.c
trunk/asterisk-ooh323c/src/ooh323cDriver.h
Modified: trunk/asterisk-ooh323c/ChangeLog
URL: http://svn.digium.com/view/asterisk-addons/trunk/asterisk-ooh323c/ChangeLog?rev=188&r1=187&r2=188&view=diff
==============================================================================
--- trunk/asterisk-ooh323c/ChangeLog (original)
+++ trunk/asterisk-ooh323c/ChangeLog Wed Jan 18 17:11:09 2006
@@ -1,3 +1,10 @@
+Release 0.4
+* Fixed 4 deadlocks conditions in onAlerting, onReceivedDigit and
+ onCallExstablished funtions
+* Added Thread/Mutex debugging option, for debug build
+* Fixed bug causing dialed digits alias being set as a char string
+
+
Release 0.2
Added support for gsm, g729a, g723.1
Added call end cause conversion logic from H323 cause to asterisk cause
Modified: trunk/asterisk-ooh323c/README
URL: http://svn.digium.com/view/asterisk-addons/trunk/asterisk-ooh323c/README?rev=188&r1=187&r2=188&view=diff
==============================================================================
--- trunk/asterisk-ooh323c/README (original)
+++ trunk/asterisk-ooh323c/README Wed Jan 18 17:11:09 2006
@@ -12,7 +12,7 @@
Package Contents:
- asterisk-ooh323c-0.3
+ asterisk-ooh323c-0.4
|
|-src [H323 channel driver code]
|
@@ -49,7 +49,7 @@
mode and then run:
>make install
- 3. Open h323.conf.sample in asterisk-ooh323c-0.3 directory. Under [general]
+ 3. Open h323.conf.sample in asterisk-ooh323c-0.4 directory. Under [general]
you will see global configuration setting. Modify IP addresses of asterisk
server "bindaddr" to match your configuration.
@@ -82,7 +82,7 @@
CLI>stop now
Now to run tests, you will have to copy extensions.conf.sample in
- asterisk-ooh323c-0.3 directory to /etc/asterisk as extensions.conf. Make
+ asterisk-ooh323c-0.4 directory to /etc/asterisk as extensions.conf. Make
sure to save a backup copy of your existing extensions.conf. Now restart
the pbx as follows:
>/usr/sbin/asterisk -vvvc
Modified: trunk/asterisk-ooh323c/config.h
URL: http://svn.digium.com/view/asterisk-addons/trunk/asterisk-ooh323c/config.h?rev=188&r1=187&r2=188&view=diff
==============================================================================
--- trunk/asterisk-ooh323c/config.h (original)
+++ trunk/asterisk-ooh323c/config.h Wed Jan 18 17:11:09 2006
@@ -108,13 +108,13 @@
#define PACKAGE_NAME "ASTERISK-OOH323C"
/* Define to the full name and version of this package. */
-#define PACKAGE_STRING "ASTERISK-OOH323C 0.3"
+#define PACKAGE_STRING "ASTERISK-OOH323C 0.4"
/* Define to the one symbol short name of this package. */
#define PACKAGE_TARNAME "asterisk-ooh323c"
/* Define to the version of this package. */
-#define PACKAGE_VERSION "0.3"
+#define PACKAGE_VERSION "0.4"
/* Define to the type of arg 1 for `select'. */
#define SELECT_TYPE_ARG1 int
@@ -135,7 +135,7 @@
/* #undef TM_IN_SYS_TIME */
/* Version number of package */
-#define VERSION "0.3"
+#define VERSION "0.4"
/* Define to empty if `const' does not conform to ANSI C. */
/* #undef const */
Modified: trunk/asterisk-ooh323c/configure
URL: http://svn.digium.com/view/asterisk-addons/trunk/asterisk-ooh323c/configure?rev=188&r1=187&r2=188&view=diff
==============================================================================
--- trunk/asterisk-ooh323c/configure (original)
+++ trunk/asterisk-ooh323c/configure Wed Jan 18 17:11:09 2006
@@ -1,6 +1,6 @@
#! /bin/sh
# Guess values for system-dependent variables and create Makefiles.
-# Generated by GNU Autoconf 2.57 for ASTERISK-OOH323C 0.3.
+# Generated by GNU Autoconf 2.57 for ASTERISK-OOH323C 0.4.
#
# Report bugs to <Objective Systems <support at obj-sys.com>>.
#
@@ -427,8 +427,8 @@
# Identity of this package.
PACKAGE_NAME='ASTERISK-OOH323C'
PACKAGE_TARNAME='asterisk-ooh323c'
-PACKAGE_VERSION='0.3'
-PACKAGE_STRING='ASTERISK-OOH323C 0.3'
+PACKAGE_VERSION='0.4'
+PACKAGE_STRING='ASTERISK-OOH323C 0.4'
PACKAGE_BUGREPORT='Objective Systems <support at obj-sys.com>'
ac_unique_file="src/chan_h323.c"
@@ -938,7 +938,7 @@
# Omit some internal or obsolete options to make the list less imposing.
# This message is too long to be a string in the A/UX 3.1 sh.
cat <<_ACEOF
-\`configure' configures ASTERISK-OOH323C 0.3 to adapt to many kinds of systems.
+\`configure' configures ASTERISK-OOH323C 0.4 to adapt to many kinds of systems.
Usage: $0 [OPTION]... [VAR=VALUE]...
@@ -1004,7 +1004,7 @@
if test -n "$ac_init_help"; then
case $ac_init_help in
- short | recursive ) echo "Configuration of ASTERISK-OOH323C 0.3:";;
+ short | recursive ) echo "Configuration of ASTERISK-OOH323C 0.4:";;
esac
cat <<\_ACEOF
@@ -1099,7 +1099,7 @@
test -n "$ac_init_help" && exit 0
if $ac_init_version; then
cat <<\_ACEOF
-ASTERISK-OOH323C configure 0.3
+ASTERISK-OOH323C configure 0.4
generated by GNU Autoconf 2.57
Copyright 1992, 1993, 1994, 1995, 1996, 1998, 1999, 2000, 2001, 2002
@@ -1114,7 +1114,7 @@
This file contains any messages produced by compilers while
running configure, to aid debugging if configure makes a mistake.
-It was created by ASTERISK-OOH323C $as_me 0.3, which was
+It was created by ASTERISK-OOH323C $as_me 0.4, which was
generated by GNU Autoconf 2.57. Invocation command line was
$ $0 $@
@@ -1451,7 +1451,7 @@
PACKAGE=asterisk-ooh323c
-VERSION=0.3
+VERSION=0.4
ac_aux_dir=
for ac_dir in config $srcdir/config; do
@@ -9612,7 +9612,7 @@
} >&5
cat >&5 <<_CSEOF
-This file was extended by ASTERISK-OOH323C $as_me 0.3, which was
+This file was extended by ASTERISK-OOH323C $as_me 0.4, which was
generated by GNU Autoconf 2.57. Invocation command line was
CONFIG_FILES = $CONFIG_FILES
@@ -9675,7 +9675,7 @@
cat >>$CONFIG_STATUS <<_ACEOF
ac_cs_version="\\
-ASTERISK-OOH323C config.status 0.3
+ASTERISK-OOH323C config.status 0.4
configured by $0, generated by GNU Autoconf 2.57,
with options \\"`echo "$ac_configure_args" | sed 's/[\\""\`\$]/\\\\&/g'`\\"
Modified: trunk/asterisk-ooh323c/configure.in
URL: http://svn.digium.com/view/asterisk-addons/trunk/asterisk-ooh323c/configure.in?rev=188&r1=187&r2=188&view=diff
==============================================================================
--- trunk/asterisk-ooh323c/configure.in (original)
+++ trunk/asterisk-ooh323c/configure.in Wed Jan 18 17:11:09 2006
@@ -1,7 +1,7 @@
AC_PREREQ(2.57)
-AC_INIT([ASTERISK-OOH323C],[0.3],[Objective Systems <support at obj-sys.com>],[asterisk-ooh323c])
+AC_INIT([ASTERISK-OOH323C],[0.4],[Objective Systems <support at obj-sys.com>],[asterisk-ooh323c])
PACKAGE=asterisk-ooh323c
-VERSION=0.3
+VERSION=0.4
AC_CONFIG_AUX_DIR(config)
AM_INIT_AUTOMAKE($PACKAGE, $VERSION)
Modified: trunk/asterisk-ooh323c/ooh323c/src/eventHandler.c
URL: http://svn.digium.com/view/asterisk-addons/trunk/asterisk-ooh323c/ooh323c/src/eventHandler.c?rev=188&r1=187&r2=188&view=diff
==============================================================================
--- trunk/asterisk-ooh323c/ooh323c/src/eventHandler.c (original)
+++ trunk/asterisk-ooh323c/ooh323c/src/eventHandler.c Wed Jan 18 17:11:09 2006
@@ -62,7 +62,7 @@
}
void invokeBitStrValue (OOCTXT* pctxt, ASN1UINT numbits,
- const ASN1OCTET* data)
+ const ASN1OCTET* data)
{
if (0 != pctxt->pEventHandler) {
pctxt->pEventHandler->bitStrValue (numbits, data);
@@ -70,7 +70,7 @@
}
void invokeOctStrValue (OOCTXT* pctxt, ASN1UINT numocts,
- const ASN1OCTET* data)
+ const ASN1OCTET* data)
{
if (0 != pctxt->pEventHandler) {
pctxt->pEventHandler->octStrValue (numocts, data);
@@ -85,7 +85,7 @@
}
void invokeCharStr16BitValue (OOCTXT* pctxt, ASN1UINT nchars,
- ASN116BITCHAR* data)
+ ASN116BITCHAR* data)
{
if (0 != pctxt->pEventHandler) {
pctxt->pEventHandler->charStr16BitValue (nchars, data);
@@ -114,7 +114,7 @@
}
void invokeOpenTypeValue (OOCTXT* pctxt, ASN1UINT numocts,
- const ASN1OCTET* data)
+ const ASN1OCTET* data)
{
if (0 != pctxt->pEventHandler) {
pctxt->pEventHandler->openTypeValue (numocts, data);
Modified: trunk/asterisk-ooh323c/ooh323c/src/ooCalls.c
URL: http://svn.digium.com/view/asterisk-addons/trunk/asterisk-ooh323c/ooh323c/src/ooCalls.c?rev=188&r1=187&r2=188&view=diff
==============================================================================
--- trunk/asterisk-ooh323c/ooh323c/src/ooCalls.c (original)
+++ trunk/asterisk-ooh323c/ooh323c/src/ooCalls.c Wed Jan 18 17:11:09 2006
@@ -52,11 +52,13 @@
sprintf(call->callToken, "%s", callToken);
sprintf(call->callType, "%s", type);
call->callReference = 0;
- if(gH323ep.callerid){
+ if(gH323ep.callerid) {
strncpy(call->ourCallerId, gH323ep.callerid, sizeof(call->ourCallerId)-1);
call->ourCallerId[sizeof(call->ourCallerId)-1] = '\0';
- }else
+ }
+ else {
call->ourCallerId[0] = '\0';
+ }
memset(&call->callIdentifier, 0, sizeof(H225CallIdentifier));
memset(&call->confIdentifier, 0, sizeof(H225ConferenceIdentifier));
@@ -84,17 +86,20 @@
if(!strcmp(call->callType, "incoming"))
{
call->callingPartyNumber = NULL;
- }else{
+ }
+ else{
if(ooUtilsIsStrEmpty(gH323ep.callingPartyNumber))
{
call->callingPartyNumber = NULL;
- }else{
+ }
+ else{
call->callingPartyNumber = (char*) memAlloc(call->pctxt,
strlen(gH323ep.callingPartyNumber)+1);
if(call->callingPartyNumber)
{
strcpy(call->callingPartyNumber, gH323ep.callingPartyNumber);
- }else{
+ }
+ else{
OOTRACEERR3("Error:Memory - ooCreateCall - callingPartyNumber"
".(%s, %s)\n", call->callType, call->callToken);
freeContext(pctxt);
@@ -199,7 +204,8 @@
if(!call->pH225Channel || call->pH225Channel->sock ==0)
{
call->callState = OO_CALL_CLEARED;
- }else{
+ }
+ else{
if(!OO_TESTFLAG(call->flags, OO_M_RELEASE_BUILT))
{
if(call->callState == OO_CALL_CLEAR ||
@@ -243,7 +249,7 @@
OOCTXT *pctxt;
OOTRACEWARN4 ("Cleaning Call (%s, %s)- reason:%s\n",
- call->callType, call->callToken,
+ call->callType, call->callToken,
ooGetReasonCodeText (call->callEndReason));
/* First clean all the logical channels, if not already cleaned. */
@@ -299,7 +305,8 @@
OOTRACEERR3("Error:Failed to forward call (%s, %s)\n", call->callType,
call->callToken);
}
- }else {
+ }
+ else {
if(gH323ep.h323Callbacks.onCallCleared)
gH323ep.h323Callbacks.onCallCleared(call);
}
@@ -328,7 +335,8 @@
if(call->callingPartyNumber)
{
strcpy(call->callingPartyNumber, number);
- }else{
+ }
+ else{
OOTRACEERR3("Error:Memory - ooCallSetCallingPartyNumber - "
"callingPartyNumber.(%s, %s)\n", call->callType,
call->callToken);
@@ -366,7 +374,8 @@
if(call->calledPartyNumber)
{
strcpy(call->calledPartyNumber, number);
- }else{
+ }
+ else{
OOTRACEERR3("Error:Memory - ooCallSetCalledPartyNumber - "
"calledPartyNumber.(%s, %s)\n", call->callType,
call->callToken);
@@ -424,7 +433,8 @@
{
psNewAlias->next = call->ourAliases;
call->ourAliases = psNewAlias;
- }else {
+ }
+ else {
psNewAlias->next = call->remoteAliases;
call->remoteAliases = psNewAlias;
}
@@ -686,7 +696,7 @@
if(call->masterSlaveState == OO_MasterSlave_Master)
sessionID = call->nextSessionID++;
else{
- OOTRACEDBGC4("Session id for %s channel of type audio has to be "
+ OOTRACEDBGC4("Session id for %s channel of type audio has to be "
"provided by remote.(%s, %s)\n", dir, call->callType,
call->callToken);
sessionID = 0; /* Will be assigned by remote */
Modified: trunk/asterisk-ooh323c/ooh323c/src/ooCalls.h
URL: http://svn.digium.com/view/asterisk-addons/trunk/asterisk-ooh323c/ooh323c/src/ooCalls.h?rev=188&r1=187&r2=188&view=diff
==============================================================================
--- trunk/asterisk-ooh323c/ooh323c/src/ooCalls.h (original)
+++ trunk/asterisk-ooh323c/ooh323c/src/ooCalls.h Wed Jan 18 17:11:09 2006
@@ -111,9 +111,9 @@
* a particular call.
*/
typedef struct OOH323Channel {
- OOSOCKET sock; /*!< Socket connection for the channel */
- int port; /*!< Port assigned to the channel */
- DList outQueue; /*!< Output message queue */
+ OOSOCKET sock; /*!< Socket connection for the channel */
+ int port; /*!< Port assigned to the channel */
+ DList outQueue; /*!< Output message queue */
} OOH323Channel;
/**
@@ -142,8 +142,8 @@
OOMediaInfo *mediaInfo;
OOCallFwdData *pCallFwdData;
char localIP[20];/* Local IP address */
- OOH323Channel* pH225Channel;
- OOH323Channel* pH245Channel;
+ OOH323Channel* pH225Channel;
+ OOH323Channel* pH245Channel;
OOSOCKET *h245listener;
int *h245listenport;
char remoteIP[20];/* Remote IP address */
@@ -170,7 +170,7 @@
int logicalChanNoMax;
int logicalChanNoCur;
unsigned nextSessionID; /* Note by default 1 is audio session, 2 is video and 3 is data, from 3 onwards master decides*/
- DList timerList;
+ DList timerList;
ASN1UINT msdRetries;
void *usrData; /*!<User can set this to user specific data*/
struct OOH323CallData* next;
@@ -530,7 +530,7 @@
cb_StartReceiveChannel startReceiveChannel,
cb_StartTransmitChannel startTransmitChannel,
cb_StopReceiveChannel stopReceiveChannel,
- cb_StopTransmitChannel stopTransmitChannel);
+ cb_StopTransmitChannel stopTransmitChannel);
Modified: trunk/asterisk-ooh323c/ooh323c/src/ooCapability.c
URL: http://svn.digium.com/view/asterisk-addons/trunk/asterisk-ooh323c/ooh323c/src/ooCapability.c?rev=188&r1=187&r2=188&view=diff
==============================================================================
--- trunk/asterisk-ooh323c/ooh323c/src/ooCapability.c (original)
+++ trunk/asterisk-ooh323c/ooh323c/src/ooCapability.c Wed Jan 18 17:11:09 2006
@@ -30,7 +30,8 @@
{
gH323ep.dtmfmode |= OO_CAP_DTMF_RFC2833;
OOTRACEINFO1("Enabled RFC2833 DTMF capability for end-point\n");
- }else{
+ }
+ else{
call->dtmfmode |= OO_CAP_DTMF_RFC2833;
OOTRACEINFO3("Enabled RFC2833 DTMF capability for (%s, %s) \n",
call->callType, call->callToken);
@@ -50,7 +51,8 @@
if(!call){
gH323ep.dtmfmode ^= OO_CAP_DTMF_RFC2833;
OOTRACEINFO1("Disabled RFC2833 DTMF capability for end-point\n");
- }else{
+ }
+ else{
call->dtmfmode ^= OO_CAP_DTMF_RFC2833;
OOTRACEINFO3("Disabled RFC2833 DTMF capability for (%s, %s)\n",
call->callType, call->callToken);
@@ -64,7 +66,8 @@
if(!call){
gH323ep.dtmfmode |= OO_CAP_DTMF_H245_alphanumeric;
OOTRACEINFO1("Dtmf mode set to H.245(alphanumeric) for endpoint\n");
- }else {
+ }
+ else {
call->dtmfmode |= OO_CAP_DTMF_H245_alphanumeric;
OOTRACEINFO3("Dtmf mode set to H.245(alphanumeric) for (%s, %s)\n",
call->callType, call->callToken);
@@ -77,7 +80,8 @@
if(!call){
gH323ep.dtmfmode ^= OO_CAP_DTMF_H245_alphanumeric;
OOTRACEINFO1("Dtmf mode H.245(alphanumeric) disabled for endpoint\n");
- }else {
+ }
+ else {
call->dtmfmode ^= OO_CAP_DTMF_H245_alphanumeric;
OOTRACEINFO3("Dtmf mode H.245(alphanumeric) disabled for (%s, %s)\n",
call->callType, call->callToken);
@@ -90,7 +94,8 @@
if(!call){
gH323ep.dtmfmode |= OO_CAP_DTMF_H245_signal;
OOTRACEINFO1("Dtmf mode set to H.245(signal) for endpoint\n");
- }else {
+ }
+ else {
call->dtmfmode |= OO_CAP_DTMF_H245_signal;
OOTRACEINFO3("Dtmf mode set to H.245(signal) for (%s, %s)\n",
call->callType, call->callToken);
@@ -103,7 +108,8 @@
if(!call){
gH323ep.dtmfmode ^= OO_CAP_DTMF_H245_signal;
OOTRACEINFO1("Dtmf mode H.245(signal) disabled for endpoint\n");
- }else {
+ }
+ else {
call->dtmfmode ^= OO_CAP_DTMF_H245_signal;
OOTRACEINFO3("Dtmf mode H.245(signal) disabled for (%s, %s)\n",
call->callType, call->callToken);
@@ -116,7 +122,8 @@
if(!call){
gH323ep.dtmfmode |= OO_CAP_DTMF_Q931;
OOTRACEINFO1("Dtmf mode set to Q.931(keypad) for the endpoint\n");
- }else {
+ }
+ else {
call->dtmfmode |= OO_CAP_DTMF_Q931;
OOTRACEINFO3("Dtmf mode set to Q.931(keypad) for the call (%s, %s)\n",
call->callType, call->callToken);
@@ -129,7 +136,8 @@
if(!call){
gH323ep.dtmfmode ^= OO_CAP_DTMF_Q931;
OOTRACEINFO1("Dtmf mode Q.931(keypad) disabled for the endpoint\n");
- }else {
+ }
+ else {
call->dtmfmode ^= OO_CAP_DTMF_Q931;
OOTRACEINFO3("Dtmf mode Q.931(keypad) disabled for the call (%s, %s)\n",
call->callType, call->callToken);
@@ -280,7 +288,8 @@
{
epCap->dir = OORX;
epCap->dir |= OOTX;
- }else
+ }
+ else
epCap->dir = dir;
epCap->cap = OO_H263VIDEO;
@@ -305,18 +314,20 @@
}
ooAppendCapToCapPrefs(NULL, cap);
gH323ep.noOfCaps++;
- }else{
+ }
+ else{
if(remote)
{
/*Add as remote capability */
if(!call->remoteCaps)
call->remoteCaps = epCap;
else{
- cur = call->remoteCaps;
+ cur = call->remoteCaps;
while(cur->next) cur = cur->next;
cur->next = epCap;
}
- }else{
+ }
+ else{
/*Add as our capability */
OOTRACEDBGC4("Adding call specific H263 video capability %s. "
"(%s, %s)\n", pictureType, call->callType,
@@ -326,7 +337,7 @@
ooResetCapPrefs(call);
}
else{
- cur = call->ourCaps;
+ cur = call->ourCaps;
while(cur->next) cur = cur->next;
cur->next = epCap;
}
@@ -357,7 +368,7 @@
params = (OOCapParams*) memAlloc(pctxt, sizeof(OOCapParams));
if(!epCap || !params)
{
- OOTRACEERR1("Error:Memory - ooCapabilityAddSimpleCapability - "
+ OOTRACEERR1("ERROR: Memory - ooCapabilityAddSimpleCapability - "
"epCap/params\n");
return OO_FAILED;
}
@@ -371,12 +382,13 @@
else
params->silenceSuppression = FALSE; /* Set to false for g711 and g729*/
- if(dir & OORXANDTX)
- {
+ if(dir & OORXANDTX) {
epCap->dir = OORX;
epCap->dir |= OOTX;
- }else
+ }
+ else {
epCap->dir = dir;
+ }
epCap->cap = cap;
epCap->capType = OO_CAP_TYPE_AUDIO;
@@ -388,11 +400,13 @@
epCap->next = NULL;
if(!call)
- {/*Add as local capability */
+ {
+ /* Add as local capability */
OOTRACEDBGC2("Adding endpoint capability %s. \n",
ooGetCapTypeText(epCap->cap));
- if(!gH323ep.myCaps)
+ if(!gH323ep.myCaps) {
gH323ep.myCaps = epCap;
+ }
else{
cur = gH323ep.myCaps;
while(cur->next) cur = cur->next;
@@ -400,33 +414,36 @@
}
ooAppendCapToCapPrefs(NULL, cap);
gH323ep.noOfCaps++;
- }else{
+ }
+ else{
if(remote)
{
- /*Add as remote capability */
- if(!call->remoteCaps)
+ /* Add as remote capability */
+ if(!call->remoteCaps) {
call->remoteCaps = epCap;
+ }
else{
- cur = call->remoteCaps;
+ cur = call->remoteCaps;
while(cur->next) cur = cur->next;
cur->next = epCap;
}
- }else{
- /*Add as our capability */
- OOTRACEDBGC4("Adding call specific capability %s. (%s, %s)\n",
- ooGetCapTypeText(epCap->cap), call->callType,
- call->callToken);
- if(!call->ourCaps){
- call->ourCaps = epCap;
- ooResetCapPrefs(call);
- }
- else{
- cur = call->ourCaps;
- while(cur->next) cur = cur->next;
- cur->next = epCap;
- }
- ooAppendCapToCapPrefs(call, cap);
- }
+ }
+ else{
+ /* Add as our capability */
+ OOTRACEDBGC4("Adding call specific capability %s. (%s, %s)\n",
+ ooGetCapTypeText(epCap->cap), call->callType,
+ call->callToken);
+ if(!call->ourCaps){
+ call->ourCaps = epCap;
+ ooResetCapPrefs(call);
+ }
+ else{
+ cur = call->ourCaps;
+ while(cur->next) cur = cur->next;
+ cur->next = epCap;
+ }
+ ooAppendCapToCapPrefs(call, cap);
+ }
}
return OO_OK;
@@ -468,7 +485,8 @@
{
epCap->dir = OORX;
epCap->dir |= OOTX;
- }else
+ }
+ else
epCap->dir = dir;
epCap->cap = cap;
@@ -492,18 +510,20 @@
}
ooAppendCapToCapPrefs(NULL, cap);
gH323ep.noOfCaps++;
- }else{
+ }
+ else{
if(remote)
{
/*Add as remote capability */
if(!call->remoteCaps)
call->remoteCaps = epCap;
else{
- cur = call->remoteCaps;
+ cur = call->remoteCaps;
while(cur->next) cur = cur->next;
cur->next = epCap;
}
- }else{
+ }
+ else{
OOTRACEDBGC4("Adding call specific capability %s. (%s, %s)\n",
ooGetCapTypeText(epCap->cap), call->callType,
call->callToken);
@@ -513,7 +533,7 @@
ooResetCapPrefs(call);
}
else{
- cur = call->ourCaps;
+ cur = call->ourCaps;
while(cur->next) cur = cur->next;
cur->next = epCap;
}
@@ -558,7 +578,7 @@
case OO_EXTELEMVIDEO:
default:
OOTRACEERR2("ERROR: Don't know how to create video capability %s\n",
- ooGetCapTypeText(epCap->cap));
+ ooGetCapTypeText(epCap->cap));
}
return NULL;
}
@@ -597,7 +617,7 @@
return ooCapabilityCreateGSMFullRateCapability(epCap, pctxt, dir);
default:
OOTRACEERR2("ERROR: Don't know how to create audio capability %d\n",
- epCap->cap);
+ epCap->cap);
}
return NULL;
}
@@ -691,24 +711,23 @@
pVideo->u.h263VideoCapability = pH263Cap;
- if(params->picFormat == OO_PICFORMAT_SQCIF)
- {
+ if(params->picFormat == OO_PICFORMAT_SQCIF) {
pH263Cap->m.sqcifMPIPresent = TRUE;
pH263Cap->sqcifMPI = params->MPI;
- }else if(params->picFormat == OO_PICFORMAT_QCIF)
- {
+ }
+ else if(params->picFormat == OO_PICFORMAT_QCIF) {
pH263Cap->m.qcifMPIPresent = TRUE;
pH263Cap->qcifMPI = params->MPI;
- }else if(params->picFormat == OO_PICFORMAT_CIF)
- {
+ }
+ else if(params->picFormat == OO_PICFORMAT_CIF) {
pH263Cap->m.cifMPIPresent = TRUE;
pH263Cap->cifMPI = params->MPI;
- }else if(params->picFormat == OO_PICFORMAT_CIF4)
- {
+ }
+ else if(params->picFormat == OO_PICFORMAT_CIF4) {
pH263Cap->m.cif4MPIPresent = TRUE;
pH263Cap->cif4MPI = params->MPI;
- }else if(params->picFormat == OO_PICFORMAT_CIF16)
- {
+ }
+ else if(params->picFormat == OO_PICFORMAT_CIF16) {
pH263Cap->m.cif16MPIPresent = TRUE;
pH263Cap->cif16MPI = params->MPI;
}
@@ -982,10 +1001,11 @@
{
if(pH263Cap->m.sqcifMPIPresent)
{
- if(params->picFormat != OO_PICFORMAT_SQCIF)
+ if(params->picFormat != OO_PICFORMAT_SQCIF)
{
return FALSE;
- }else{
+ }
+ else{
if(pH263Cap->sqcifMPI >= params->MPI)
return TRUE;
else
@@ -997,7 +1017,8 @@
if(params->picFormat != OO_PICFORMAT_QCIF)
{
return FALSE;
- }else{
+ }
+ else{
if(pH263Cap->qcifMPI >= params->MPI)
return TRUE;
else
@@ -1009,7 +1030,8 @@
if(params->picFormat != OO_PICFORMAT_CIF)
{
return FALSE;
- }else{
+ }
+ else{
if(pH263Cap->cifMPI >= params->MPI)
return TRUE;
else
@@ -1021,7 +1043,8 @@
if(params->picFormat != OO_PICFORMAT_CIF4)
{
return FALSE;
- }else{
+ }
+ else{
if(pH263Cap->cif4MPI >= params->MPI)
return TRUE;
else
@@ -1033,7 +1056,8 @@
if(params->picFormat != OO_PICFORMAT_CIF16)
{
return FALSE;
- }else{
+ }
+ else{
if(pH263Cap->cif16MPI >= params->MPI)
return TRUE;
else
@@ -1050,7 +1074,8 @@
if(params->picFormat != OO_PICFORMAT_SQCIF)
{
return FALSE;
- }else{
+ }
+ else{
if(pH263Cap->sqcifMPI <= params->MPI)
return TRUE;
else
@@ -1062,7 +1087,8 @@
if(params->picFormat != OO_PICFORMAT_QCIF)
{
return FALSE;
- }else{
+ }
+ else{
if(pH263Cap->qcifMPI <= params->MPI)
return TRUE;
else
@@ -1074,7 +1100,8 @@
if(params->picFormat != OO_PICFORMAT_CIF)
{
return FALSE;
- }else{
+ }
+ else{
if(pH263Cap->cifMPI <= params->MPI)
return TRUE;
else
@@ -1086,7 +1113,8 @@
if(params->picFormat != OO_PICFORMAT_CIF4)
{
return FALSE;
- }else{
+ }
+ else{
if(pH263Cap->cif4MPI <= params->MPI)
return TRUE;
else
@@ -1098,7 +1126,8 @@
if(params->picFormat != OO_PICFORMAT_CIF16)
{
return FALSE;
- }else{
+ }
+ else{
if(pH263Cap->cif16MPI <= params->MPI)
return TRUE;
else
@@ -1400,7 +1429,7 @@
cap = OO_G7231;
break;
default:
- return NULL;
+ return NULL;
}
OOTRACEDBGC4("Determined Simple audio data type to be of type %s. Searching"
@@ -1418,7 +1447,7 @@
while(cur)
{
OOTRACEDBGC4("Local cap being compared %s. (%s, %s)\n",
- ooGetCapTypeText(cur->cap),call->callType, call->callToken);
+ ooGetCapTypeText(cur->cap),call->callType, call->callToken);
if(cur->cap == cap && (cur->dir & dir))
break;
@@ -1593,8 +1622,8 @@
if(cur->cap == cap && (cur->dir & dir))
{
- if(((OOH263CapParams*)cur->params)->picFormat == picFormat)
- break;
+ if(((OOH263CapParams*)cur->params)->picFormat == picFormat)
+ break;
}
cur = cur->next;
}
@@ -1609,16 +1638,16 @@
if(mpi < ((OOH263CapParams*)cur->params)->MPI)
return NULL;
else{
- epCap = (ooH323EpCapability*)memAlloc(call->pctxt,
+ epCap = (ooH323EpCapability*)memAlloc(call->pctxt,
sizeof(ooH323EpCapability));
params = (OOH263CapParams*) memAlloc(call->pctxt,
sizeof(OOH263CapParams));
if(!epCap || !params)
{
- OOTRACEERR3("Error:Memory - ooIsVideoDataTypeH263Supported - "
+ OOTRACEERR3("Error:Memory - ooIsVideoDataTypeH263Supported - "
"epCap/params. (%s, %s)\n", call->callType,
call->callToken);
- return NULL;
+ return NULL;
}
epCap->params = params;
epCap->cap = cur->cap;
@@ -1734,7 +1763,7 @@
return ooIsVideoDataTypeSupported(call, data->u.videoData, dir);
case T_H245DataType_audioData:
OOTRACEDBGC3("Looking for audio dataType support. (%s, %s)\n",
- call->callType, call->callToken);
+ call->callType, call->callToken);
return ooIsAudioDataTypeSupported(call, data->u.audioData, dir);
case T_H245DataType_data:
OOTRACEDBGC3("Data type not supported.(%s, %s)\n",
@@ -1888,7 +1917,7 @@
}
int ooAddRemoteAudioCapability(OOH323CallData *call,
- H245AudioCapability *audioCap,
+ H245AudioCapability *audioCap,
int dir)
{
int rxframes=0, txframes=0;
@@ -2025,9 +2054,10 @@
{
call->jointDtmfMode |= OO_CAP_DTMF_H245_alphanumeric;
return OO_OK;
- }else if((cap->u.receiveUserInputCapability->t ==
- T_H245UserInputCapability_dtmf) &&
- (call->dtmfmode & OO_CAP_DTMF_H245_signal))
+ }
+ else if((cap->u.receiveUserInputCapability->t ==
+ T_H245UserInputCapability_dtmf) &&
+ (call->dtmfmode & OO_CAP_DTMF_H245_signal))
{
call->jointDtmfMode |= OO_CAP_DTMF_H245_signal;
return OO_OK;
@@ -2055,7 +2085,7 @@
}
OOTRACEDBGC3("Not adding to joint capabilities. (%s, %s)\n", call->callType,
- call->callToken);
+ call->callToken);
return OO_OK;
}
Modified: trunk/asterisk-ooh323c/ooh323c/src/ooCapability.h
URL: http://svn.digium.com/view/asterisk-addons/trunk/asterisk-ooh323c/ooh323c/src/ooCapability.h?rev=188&r1=187&r2=188&view=diff
==============================================================================
--- trunk/asterisk-ooh323c/ooh323c/src/ooCapability.h (original)
+++ trunk/asterisk-ooh323c/ooh323c/src/ooCapability.h Wed Jan 18 17:11:09 2006
@@ -487,8 +487,8 @@
*/
ASN1BOOL ooCapabilityCheckCompatibility(struct OOH323CallData *call,
- ooH323EpCapability *epCap,
- H245DataType *dataType, int dir);
+ ooH323EpCapability *epCap,
+ H245DataType *dataType, int dir);
/**
Modified: trunk/asterisk-ooh323c/ooh323c/src/ooCmdChannel.c
URL: http://svn.digium.com/view/asterisk-addons/trunk/asterisk-ooh323c/ooh323c/src/ooCmdChannel.c?rev=188&r1=187&r2=188&view=diff
==============================================================================
--- trunk/asterisk-ooh323c/ooh323c/src/ooCmdChannel.c (original)
+++ trunk/asterisk-ooh323c/ooh323c/src/ooCmdChannel.c Wed Jan 18 17:11:09 2006
@@ -68,9 +68,6 @@
}
else
{
-
- //TODO:Need to add support for multihomed to work with channel driver
-
/*
bind socket to a port before connecting. Thus avoiding
implicit bind done by a connect call. Avoided on windows as
@@ -156,7 +153,8 @@
{
OOTRACEINFO1("Ignoring stack command as Gk Client is not registered"
" yet\n");
- }else {
+ }
+ else {
switch(cmd.type) {
case OO_CMD_MAKECALL:
OOTRACEINFO2("Processing MakeCall command %s\n",
Modified: trunk/asterisk-ooh323c/ooh323c/src/ooDateTime.c
URL: http://svn.digium.com/view/asterisk-addons/trunk/asterisk-ooh323c/ooh323c/src/ooDateTime.c?rev=188&r1=187&r2=188&view=diff
==============================================================================
--- trunk/asterisk-ooh323c/ooh323c/src/ooDateTime.c (original)
+++ trunk/asterisk-ooh323c/ooh323c/src/ooDateTime.c Wed Jan 18 17:11:09 2006
@@ -24,7 +24,7 @@
{
struct _timeb currSysTime;
_ftime(&currSysTime);
-
+
tv->tv_sec = currSysTime.time;
tv->tv_usec = currSysTime.millitm * 1000;
Modified: trunk/asterisk-ooh323c/ooh323c/src/ooGkClient.c
URL: http://svn.digium.com/view/asterisk-addons/trunk/asterisk-ooh323c/ooh323c/src/ooGkClient.c?rev=188&r1=187&r2=188&view=diff
==============================================================================
--- trunk/asterisk-ooh323c/ooh323c/src/ooGkClient.c (original)
+++ trunk/asterisk-ooh323c/ooh323c/src/ooGkClient.c Wed Jan 18 17:11:09 2006
@@ -87,7 +87,8 @@
{
OOTRACEINFO2("Using local RAS Ip address %s\n", cur->addr);
strcpy(pGkClient->localRASIP, cur->addr);
- }else{
+ }
+ else{
OOTRACEERR1("Error:Failed to assign a local RAS IP address\n");
return OO_FAILED;
}
@@ -153,10 +154,11 @@
OOTRACEINFO1("\tGatekeeper mode - UseSpecificGatekeeper\n");
OOTRACEINFO3("\tGatekeeper To Use - %s:%d\n", pGkClient->gkRasIP,
pGkClient->gkRasPort);
- }else if(pGkClient->gkMode == RasDiscoverGatekeeper)
- {
+ }
+ else if(pGkClient->gkMode == RasDiscoverGatekeeper) {
OOTRACEINFO1("\tGatekeeper mode - UseSpecificGatekeeper\n");
- }else{
+ }
+ else {
OOTRACEERR1("Invalid GatekeeperMode\n");
}
}
@@ -213,7 +215,7 @@
OOTRACEINFO1("Gatekeeper Mode - RasUseSpecificGatekeeper\n");
if(szGkAddr)
{
- if(strlen(szGkAddr)>MAX_IP_LEN)
+ if(strlen(szGkAddr)>MAX_IP_LEN)
{
OOTRACEERR2("Error:Invalid IP address specified - %s\n", szGkAddr);
return OO_FAILED;
@@ -227,11 +229,11 @@
OOTRACEINFO3("Gatekeeper IP:port set to - %s:%d\n",
szGkAddr, pGkClient->gkRasPort);
- }else if(eGkMode == RasDiscoverGatekeeper)
- {
+ }
+ else if(eGkMode == RasDiscoverGatekeeper) {
OOTRACEINFO1("Gatekeeper Mode - RasDiscoverGatekeeper\n");
- }else if(eGkMode == RasNoGatekeeper)
- {
+ }
+ else if(eGkMode == RasNoGatekeeper) {
OOTRACEINFO1("Gatekeeper Mode - RasNoGatekeeper\n");
}
return OO_OK;
@@ -412,7 +414,7 @@
if(iRet != OO_OK)
{
OOTRACEERR1("Error: Failed to handle received RAS message\n");
- //pGkClient->state = GkClientFailed;
+ //pGkClient->state = GkClientFailed;
}
memReset(pctxt);
}
@@ -567,16 +569,17 @@
OOTRACEERR1("Error sending RAS message\n");
return OO_FAILED;
}
- }else if(pGkClient->gkMode == RasDiscoverGatekeeper &&
- !pGkClient->discoveryComplete)
- {
+ }
+ else if(pGkClient->gkMode == RasDiscoverGatekeeper &&
+ !pGkClient->discoveryComplete) {
if(ASN_OK != ooSocketSendTo(pGkClient->rasSocket, msgPtr, iLen,
MULTICAST_GKADDRESS, MULTICAST_GKPORT))
{
OOTRACEERR1("Error sending multicast RAS message\n" );
return OO_FAILED;
}
- }else {/* should never go here */
+ }
+ else {/* should never go here */
OOTRACEERR1("Error: GkClient in invalid state.\n");
return OO_FAILED;
}
@@ -1072,7 +1075,8 @@
{
ooGkClientUpdateRegisteredAliases(pGkClient,
&pRegistrationConfirm->terminalAlias, TRUE);
- }else{/* Everything registered*/
+ }
+ else{/* Everything registered*/
ooGkClientUpdateRegisteredAliases(pGkClient, NULL, TRUE);
}
@@ -1109,7 +1113,8 @@
return OO_FAILED;
}
- }else{
+ }
+ else{
pGkClient->regTimeout = 0;
OOTRACEINFO1("Gatekeeper does not support KeepAlive.\n");
}
@@ -1379,7 +1384,8 @@
OOTRACEINFO1("Gatekeeper requested a list of aliases be unregistered\n");
ooGkClientUpdateRegisteredAliases(pGkClient,
&punregistrationRequest->endpointAlias, FALSE);
- }else{
+ }
+ else{
OOTRACEINFO1("Gatekeeper requested a all aliases to be unregistered\n");
ooGkClientUpdateRegisteredAliases(pGkClient, NULL, FALSE);
@@ -1521,7 +1527,8 @@
pAdmReq->srcCallSignalAddress.t = T_H225TransportAddress_ipAddress;
pAdmReq->srcCallSignalAddress.u.ipAddress = pIpAddressRemote;
}
- }else {
+ }
+ else {
pAdmReq->m.srcCallSignalAddressPresent = TRUE;
pAdmReq->srcCallSignalAddress.t = T_H225TransportAddress_ipAddress;
pAdmReq->srcCallSignalAddress.u.ipAddress = pIpAddressLocal;
@@ -1574,7 +1581,8 @@
destAliases = gH323ep.aliases;
srcAliases = call->remoteAliases;
- }else {
+ }
+ else {
if(call->ourAliases)
srcAliases = call->ourAliases;
else
@@ -1590,7 +1598,7 @@
if(OO_OK != ooPopulateAliasList(&pGkClient->msgCtxt, destAliases,
&pAdmReq->destinationInfo))
{
- OOTRACEERR1("Error:Failed to populate destination aliases - "
+ OOTRACEERR1("Error:Failed to populate destination aliases - "
"ARQ message\n");
pGkClient->state = GkClientFailed;
memReset(pctxt);
@@ -1672,7 +1680,8 @@
pCallAdmInfo->retries = 0;
pCallAdmInfo->requestSeqNum = pAdmReq->requestSeqNum;
dListAppend(&pGkClient->ctxt, &pGkClient->callsPendingList,pCallAdmInfo);
- }else{
+ }
+ else{
for(x=0; x<pGkClient->callsPendingList.count; x++)
{
pNode = dListFindByIndex(&pGkClient->callsPendingList, x);
@@ -1855,7 +1864,8 @@
" be matched with any pending call.\n",
pAdmissionReject->requestSeqNum);
return OO_OK;
- }else{
+ }
+ else{
call = pCallAdmInfo->call;
dListRemove(&pGkClient->callsPendingList, pNode);
memFreePtr(&pGkClient->ctxt, pCallAdmInfo);
@@ -2218,8 +2228,8 @@
"GkClient\n");
ooGkClientDestroy();
return OO_FAILED;
- }else if(pGkClient->state == GkClientGkErr)
- {
+ }
+ else if(pGkClient->state == GkClientGkErr) {
OOTRACEERR1("Error: Gatekeeper error. Either Gk not responding or "
"Gk sending invalid messages\n");
if(pGkClient->gkMode == RasUseSpecificGatekeeper)
[... 3360 lines stripped ...]
More information about the asterisk-addons-commits
mailing list