[asterisk-addons-commits] may: branch 1.6.2 r1112 - /branches/1.6.2/channels/chan_ooh323.c

[SVN commits to the Asterisk addons project]

Author: may
Date: Sun Apr 25 13:32:54 2010
New Revision: 1112

URL: http://svnview.digium.com/svn/asterisk-addons?view=rev&rev=1112
Log:
Don't pass zero length callerid to ooh323 stack

Don't pass zero callerid string to ooh323 stack because it can't encode this properly and
can't generate setup message.

(issue #17186)
Reported by: vmikhelson
Patches:
      zero_callerid_num.patch uploaded by may213 (license 454)
Tested by: may213


Modified:
    branches/1.6.2/channels/chan_ooh323.c

Modified: branches/1.6.2/channels/chan_ooh323.c
URL: http://svnview.digium.com/svn/asterisk-addons/branches/1.6.2/channels/chan_ooh323.c?view=diff&rev=1112&r1=1111&r2=1112
==============================================================================
--- branches/1.6.2/channels/chan_ooh323.c (original)
+++ branches/1.6.2/channels/chan_ooh323.c Sun Apr 25 13:32:54 2010
@@ -1305,7 +1305,7 @@
 				ast_verbose("Setting dialed digits %s\n", p->caller_dialedDigits);
 			}
 			ooCallAddAliasDialedDigits(call, p->caller_dialedDigits);
-		} else if (p->callerid_num) {
+		} else if (!ast_strlen_zero(p->callerid_num)) {
 			if (ooIsDailedDigit(p->callerid_num)) {
 				if (gH323Debug) {
 					ast_verbose("setting callid number %s\n", p->callerid_num);