11 years agoFix several XML documentation validate errors.
Leif Madsen [Tue, 2 Mar 2010 19:02:56 +0000 (19:02 +0000)]
Fix several XML documentation validate errors.

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agofix build by checking result of symlink in test_voicemail_vmsayname
Jeff Peeler [Tue, 2 Mar 2010 18:31:05 +0000 (18:31 +0000)]
fix build by checking result of symlink in test_voicemail_vmsayname

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoAdd new application VMSayName for use with voicemail.
Jeff Peeler [Tue, 2 Mar 2010 18:22:05 +0000 (18:22 +0000)]
Add new application VMSayName for use with voicemail.

VMSayName that will play the recorded name of the voicemail user if it exists,
otherwise will play the mailbox number. A unit test has been written to verify
correct functionality called test_voicemail_vmsayname.

(closes issue #14973)
Reported by: ghjm


git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agofixes ability to exit echo app
Alec L Davis [Tue, 2 Mar 2010 07:38:56 +0000 (07:38 +0000)]
fixes ability to exit echo app

when called from a ISDN channel, null frames prevent '#' exit.
Now only echo back VOICE and DTMF frames

(issue #16880)
Reported by: alecdavis
      echo_exit.diff.txt uploaded by alecdavis (license 585)
Tested by: alecdavis

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agofix asterisk setting of pritimers from chan_dahdi.conf
Alec L Davis [Tue, 2 Mar 2010 06:56:14 +0000 (06:56 +0000)]
fix asterisk setting of pritimers from chan_dahdi.conf

regression since sig_pri split.

(issue #16909)
Reported by: alecdavis
      pritimer.asterisk.diff.txt uploaded by alecdavis (license 585)
Tested by: alecdavis

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoMerged revisions 249671 via svnmerge from
Sean Bright [Mon, 1 Mar 2010 19:36:30 +0000 (19:36 +0000)]
Merged revisions 249671 via svnmerge from

  r249671 | seanbright | 2010-03-01 14:35:01 -0500 (Mon, 01 Mar 2010) | 11 lines

  Fix crash in app_voicemail related to message counting.

  We were passing a 'struct inprocess **' and treating it like a 'struct inprocess *'
  causing a segfault.

  (closes issue #16921)
  Reported by: whardier
        20100301_issue16921.patch uploaded by seanbright (license 71)
  Tested by: whardier

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoCleanup display_*message functions.
Michiel van Baak [Mon, 1 Mar 2010 19:33:26 +0000 (19:33 +0000)]
Cleanup display_*message functions.

This patch splits transmit_displaymessage into transmit_clear_display_message and transmit_display_message which better aligns with the skinny protocol. The new transmit_display_message is not used in the current code, but will be and so it is commented.

Moved handle_datetime from this function to onhook and offhook functions (display now properly cleared at the end of a call on 30VIP).

Removed skinny debug messages from inline code as there's an ast_verb in transmit_clear_display_message. Also, removed commentary that it was a clear display as it is now apparent from the function name.

Split transmit_displaypromptmessage into display and clear.

(closes issue #16878)
Reported by: wedhorn
skinny-clean02.diff uploaded by wedhorn (license 30)
skinny-clean03.diff uploaded by wedhorn (license 30)

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agofix endianes issues in chan_skinny
Michiel van Baak [Mon, 1 Mar 2010 19:27:34 +0000 (19:27 +0000)]
fix endianes issues in chan_skinny

(closes issue #16826)
Reported by: PipoCanaja
      chan_skinny.c_bigendianPatch_20100218.diff uploaded by PipoCanaja (license 994)
Tested by: wedhorn

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoConstify a bit of app_voicemail, to make ODBC and IMAP compile once again.
Tilghman Lesher [Mon, 1 Mar 2010 18:36:06 +0000 (18:36 +0000)]
Constify a bit of app_voicemail, to make ODBC and IMAP compile once again.

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoMerged revisions 249536 via svnmerge from
Jeff Peeler [Mon, 1 Mar 2010 17:11:31 +0000 (17:11 +0000)]
Merged revisions 249536 via svnmerge from

  r249536 | jpeeler | 2010-03-01 11:02:03 -0600 (Mon, 01 Mar 2010) | 11 lines

  Modify queued frames from local channels to not set the other side to up

  In this case, attended transfers were broken due to ast_feature_request_and_dial
  detecting the channel being set to up before the answer frame could be read and
  therefore failing to mark the channel as ready. This fix is a regression fix for
  244785, which should continue to work properly as well.

  (closes issue #16816)
  Reported by: jamhed
  Tested by: jamhed, corruptor

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoFix unit test that Alec Davis broke.
Tilghman Lesher [Sun, 28 Feb 2010 20:50:01 +0000 (20:50 +0000)]
Fix unit test that Alec Davis broke.

(closes issue #16927)
 Reported by: alecdavis

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agomake unit test check for NULL folder, which then defaults to INBOX
Alec L Davis [Sun, 28 Feb 2010 16:36:45 +0000 (16:36 +0000)]
make unit test check for NULL folder, which then defaults to INBOX

previous test, gave false level of assurance that code was healthy.

(issue #16927)
Reported by: alecdavis
      based on app_voicemail_test.diff.txt uploaded by alecdavis (license 585)

Tested by: alecdavis

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoProperly document voicemail API documents. Also fix a crash reported via the -dev...
Tilghman Lesher [Sun, 28 Feb 2010 07:10:22 +0000 (07:10 +0000)]
Properly document voicemail API documents.  Also fix a crash reported via the -dev list.

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agooverlap receiving: automatically send CALL PROCEEDING when dialplan starts
Alec L Davis [Sat, 27 Feb 2010 22:49:51 +0000 (22:49 +0000)]
overlap receiving: automatically send CALL PROCEEDING when dialplan starts

Following Q.931 5.2.4
When the user has determined that sufficient call information has been received the
user shall stop T302 and send CALL PROCEEDING to the network.

Previously timeouts were possible if the dialplan took a long time to issue any
response back to the network.

Verified that our local TELCO also does the same.

(issue #16789)
Reported by: alecdavis
      overlap_receiving_trunk.diff.txt uploaded by alecdavis (license 585)
Tested by: alecdavis

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoMerged revisions 249234 via svnmerge from
Kevin P. Fleming [Sat, 27 Feb 2010 14:08:35 +0000 (14:08 +0000)]
Merged revisions 249234 via svnmerge from

  r249234 | kpfleming | 2010-02-27 09:07:59 -0500 (Sat, 27 Feb 2010) | 1 line

  add a reference to the now-published IAX2 RFC

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoCleanups to fix bugs in the VM count API functions.
Tilghman Lesher [Fri, 26 Feb 2010 18:41:57 +0000 (18:41 +0000)]
Cleanups to fix bugs in the VM count API functions.

- Urgent voicemails were not attached, because the attachment code looked in the wrong folder.
- Urgent voicemails were sometimes counted twice when displaying the count of new messages.
- Backends were inconsistent as to which voicemails each API counted.
- Unit tests added to verify behavior in the future.

(closes issue #15654)
 Reported by: tomo1657
       20100225__issue15654.diff.txt uploaded by tilghman (license 14)
 Tested by: tilghman

(closes issue #16448)
 Reported by: hevad


git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoadds Time field to "test show results" cli command
David Vossel [Fri, 26 Feb 2010 18:41:36 +0000 (18:41 +0000)]
adds Time field to "test show results" cli command

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoSend a manager event when the manager BridgeAction command is used.
Mark Michelson [Fri, 26 Feb 2010 17:13:36 +0000 (17:13 +0000)]
Send a manager event when the manager BridgeAction command is used.

(closes issue #16769)
Reported by: syspert
      bridgeaction.patch uploaded by syspert (license 938)

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoMerged revisions 249100 via svnmerge from
Mark Michelson [Fri, 26 Feb 2010 17:04:58 +0000 (17:04 +0000)]
Merged revisions 249100 via svnmerge from

  r249100 | mmichelson | 2010-02-26 11:04:29 -0600 (Fri, 26 Feb 2010) | 8 lines

  For T.38 reINVITEs treat a 606 the same as a 488.

  (closes issue #16792)
  Reported by: vrban
        t38_606.patch uploaded by vrban (license 756)

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoformatting tweaks and constification
Russell Bryant [Fri, 26 Feb 2010 08:45:11 +0000 (08:45 +0000)]
formatting tweaks and constification

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoTrim trailing whitespace (to help reduce diff against cdr-q branch)
Russell Bryant [Fri, 26 Feb 2010 08:35:42 +0000 (08:35 +0000)]
Trim trailing whitespace (to help reduce diff against cdr-q branch)

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoTrim trailing whitespace, convert lists of defines to enums
Russell Bryant [Fri, 26 Feb 2010 08:26:10 +0000 (08:26 +0000)]
Trim trailing whitespace, convert lists of defines to enums

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agotrivial formatting tweak (working on reducing diff against trunk for cdr-q)
Russell Bryant [Fri, 26 Feb 2010 08:08:38 +0000 (08:08 +0000)]
trivial formatting tweak (working on reducing diff against trunk for cdr-q)

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoremove include
Russell Bryant [Fri, 26 Feb 2010 08:07:27 +0000 (08:07 +0000)]
remove include

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoconstification, remove include
Russell Bryant [Fri, 26 Feb 2010 08:07:09 +0000 (08:07 +0000)]
constification, remove include

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoRemove unnecessary includes, formatting tweak
Russell Bryant [Fri, 26 Feb 2010 08:05:36 +0000 (08:05 +0000)]
Remove unnecessary includes, formatting tweak

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoconstification and remove unnecessary include
Russell Bryant [Fri, 26 Feb 2010 08:04:07 +0000 (08:04 +0000)]
constification and remove unnecessary include

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoMerged revisions 248860 via svnmerge from
Jeff Peeler [Thu, 25 Feb 2010 23:09:54 +0000 (23:09 +0000)]
Merged revisions 248860 via svnmerge from

  r248860 | jpeeler | 2010-02-25 15:22:06 -0600 (Thu, 25 Feb 2010) | 18 lines

  Ensure that monitor recordings are written to the correct location (again)

  This is an extension to 248757. As such the dialplan test has been extended:

  exten => 5040, 1, monitor(wav,tmp/jeff/monitor_test,b)
  exten => 5040, n, dial(sip/5001)
  exten => 5041, 1, monitor(wav,/tmp/jeff/monitor_test2,b)
  exten => 5041, n, dial(sip/5001)
  exten => 5042, 1, monitor(wav,monitor_test3,b)
  exten => 5042, n, dial(sip/5001)
  exten => 5043, 1, monitor(wav,tmp/jeff/monitor_test3,m)
  exten => 5043, n, changemonitor(monitor_test4)
  exten => 5043, n, dial(sip/5001)
  exten => 5044, 1, monitor(wav,monitor_test4,m)
  exten => 5044, n, changemonitor(tmp/jeff/monitor_test5) ; this looks to fail by design and emits a warning
  exten => 5044, n, dial(sip/5001)

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoFix incorrect ACL behavior when CIDR notation of "/0" is used.
Mark Michelson [Thu, 25 Feb 2010 22:41:48 +0000 (22:41 +0000)]
Fix incorrect ACL behavior when CIDR notation of "/0" is used.


git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoMerged revisions 248859 via svnmerge from
Tilghman Lesher [Thu, 25 Feb 2010 21:22:39 +0000 (21:22 +0000)]
Merged revisions 248859 via svnmerge from

  r248859 | tilghman | 2010-02-25 15:21:05 -0600 (Thu, 25 Feb 2010) | 15 lines

  Some platforms clear /var/run at boot, which makes connecting a remote console... difficult.

  Previously, we only created the default /var/run/asterisk directory at install
  time.  While we could create it in the init script, that would not work for
  those who start asterisk manually from the command line.  So the safest thing
  to do is to create it as part of the Asterisk boot process.  This also changes
  the ownership of the directory, because the pid and ctl files are created after
  we setuid/setgid.

  (closes issue #16802)
   Reported by: Brian
         20100224__issue16802.diff.txt uploaded by tilghman (license 14)
   Tested by: tzafrir

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoMerged revisions 248757 via svnmerge from
Jeff Peeler [Thu, 25 Feb 2010 18:37:56 +0000 (18:37 +0000)]
Merged revisions 248757 via svnmerge from

  r248757 | jpeeler | 2010-02-25 12:06:54 -0600 (Thu, 25 Feb 2010) | 15 lines

  Ensure that monitor recordings are written to the correct location.

  Recordings should be placed in the monitor directory when a non-absolute path
  is used.

  Exact dialplan used for testing:
  exten => 5040, 1, monitor(wav,tmp/jeff/monitor_test,b)
  exten => 5040, n, dial(sip/5001)
  exten => 5041, 1, monitor(wav,/tmp/jeff/monitor_test2,b)
  exten => 5041, n, dial(sip/5001)
  exten => 5042, 1, monitor(wav,monitor_test3,b)
  exten => 5042, n, dial(sip/5001)


git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoBlocked revisions 248668 via svnmerge
Jeff Peeler [Wed, 24 Feb 2010 23:00:40 +0000 (23:00 +0000)]
Blocked revisions 248668 via svnmerge

  r248668 | jpeeler | 2010-02-24 16:58:54 -0600 (Wed, 24 Feb 2010) | 8 lines

  Make deletion of temporary greetings work properly with IMAP_STORAGE

  This same patch was merged in 220833, but was skipped in this branch

  (closes issue #16170)
  Reported by: francesco_r

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoAlso kill the .i files, or else the build process will not recreate them, when we
Tilghman Lesher [Wed, 24 Feb 2010 22:44:55 +0000 (22:44 +0000)]
Also kill the .i files, or else the build process will not recreate them, when we
change flags.  Fixes a weird symbol problem mmichelson was having in a group branch,
but also applies to trunk.

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoMerged revisions 248582 via svnmerge from
Tilghman Lesher [Wed, 24 Feb 2010 21:17:26 +0000 (21:17 +0000)]
Merged revisions 248582 via svnmerge from

  r248582 | tilghman | 2010-02-24 15:02:18 -0600 (Wed, 24 Feb 2010) | 7 lines

  Remove color code sequences from verbose messages that go to logfiles.
  (closes issue #16786)
   Reported by: dodo
         logger2.patch uploaded by dodo (license 989)
   Tested by: tilghman

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoRemove unnecessary warning message, make a couple of formatting tweaks
Russell Bryant [Wed, 24 Feb 2010 06:39:27 +0000 (06:39 +0000)]
Remove unnecessary warning message, make a couple of formatting tweaks

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoAdd ASTERISK_FILE_VERSION macro.
Russell Bryant [Wed, 24 Feb 2010 06:38:10 +0000 (06:38 +0000)]

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoUnit test for ast_str API.
Mark Michelson [Tue, 23 Feb 2010 22:29:06 +0000 (22:29 +0000)]
Unit test for ast_str API.


git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoMerged revisions 248396 via svnmerge from
David Vossel [Tue, 23 Feb 2010 16:34:39 +0000 (16:34 +0000)]
Merged revisions 248396 via svnmerge from

  r248396 | dvossel | 2010-02-23 10:26:05 -0600 (Tue, 23 Feb 2010) | 9 lines

  fixes invite with replaces deadlock

  (closes issue #16862)
  Reported by: pwalker
        replaces_deadlock_1.4 uploaded by dvossel (license 671)
  Tested by: pwalker, dvossel

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoMove the REF_DEBUG comment higher in the include list.
Mark Michelson [Mon, 22 Feb 2010 20:19:00 +0000 (20:19 +0000)]
Move the REF_DEBUG comment higher in the include list.

Uncommenting the REF_DEBUG definition where it was in the source
resulted in only a small part of the astobj2 references being logged
to a file. Moving this up higher in the include list causes all references
to be logged as they should be.

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoBlocked revisions 248268 via svnmerge
Olle Johansson [Mon, 22 Feb 2010 13:55:24 +0000 (13:55 +0000)]
Blocked revisions 248268 via svnmerge

  r248268 | oej | 2010-02-22 14:52:34 +0100 (Mån, 22 Feb 2010) | 2 lines

  Don't log to debug unless debug is turned on

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoMinor tweaks to comment blocks and includes.
Russell Bryant [Mon, 22 Feb 2010 06:45:52 +0000 (06:45 +0000)]
Minor tweaks to comment blocks and includes.

Fix the copyright lines, tweak doxygen formatting, and remove some unnecessary

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoTweak copyright and author lines.
Russell Bryant [Mon, 22 Feb 2010 06:43:02 +0000 (06:43 +0000)]
Tweak copyright and author lines.

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoCleanup transmit_* functions, part 1
Michiel van Baak [Sun, 21 Feb 2010 12:09:53 +0000 (12:09 +0000)]
Cleanup transmit_* functions, part 1

Break transmit_tone into transmit_start_tone and transmit_stop_tone as per the skinny protocol.

(closes issue #16874)
Reported by: wedhorn
      skinny-clean01.diff uploaded by wedhorn (license 30)

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoImprove support for RTCP reports without report blocks
Olle Johansson [Sat, 20 Feb 2010 22:37:22 +0000 (22:37 +0000)]
Improve support for RTCP reports without report blocks

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoBlocked revisions 248106 via svnmerge
Olle Johansson [Sat, 20 Feb 2010 22:29:50 +0000 (22:29 +0000)]
Blocked revisions 248106 via svnmerge

r248106 | oej | 2010-02-20 23:25:42 +0100 (Lör, 20 Feb 2010) | 2 lines

Make sure we support RTCP compound messages with zero reports


git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agomfcr2 issue 0016844 - Fix portability bit fields and make mfcr2_immediate_accept...
Moises Silva [Fri, 19 Feb 2010 18:38:34 +0000 (18:38 +0000)]
mfcr2 issue 0016844 - Fix portability bit fields and make mfcr2_immediate_accept work again, reported and patched by korihor

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agohandle_request_invite revise comment, fix coding guideline issues
David Vossel [Fri, 19 Feb 2010 17:40:26 +0000 (17:40 +0000)]
handle_request_invite revise comment, fix coding guideline issues

I'm working with this code right now trying to analyze a deadlock.
This change is just to clean up a few things before I make a more
complex patch.

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoMerged revisions 247910 via svnmerge from
Richard Mudgett [Fri, 19 Feb 2010 17:33:33 +0000 (17:33 +0000)]
Merged revisions 247910 via svnmerge from

  r247910 | rmudgett | 2010-02-19 11:18:49 -0600 (Fri, 19 Feb 2010) | 55 lines

  Merged revision 247904 from

  r247904 | rmudgett | 2010-02-19 10:49:44 -0600 (Fri, 19 Feb 2010) | 49 lines

  Make chan_misdn DTMF processing consistent with other channel technologies.

  The processing of DTMF tones on the receiving side of an ISDN channel is
  inconsistent with the way it is handled in other channels, especially
  DAHDI analog.  This causes DTMF tones sent from an ISDN phone to be
  doubled at the connected party.

  We are using the following 2 options of misdn.conf
  1) astdtmf=yes
  2) senddtmf=yes

  Option one is necessary because the asterisk DSP DTMF detection is better
  than mISDN's internal DSP.  Not as many false positives.

  Option two is necessary to transmit DTMF tones end to end when mISDN
  channels are connected to SIP channels with out of band DTMF for example.

  The symptom is that DTMF tones sent by an ISDN phone are doubled on the
  way through asterisk when two mISDN channels are connected with a Local
  channel in between or if it is bridged to an analog channel.

  The doubling of DTMF tones is because DTMF is passed inband to asterisk by
  the mISDN channel and passed out of band once again after the release of
  the DTMF tone.  Passing it inband is wrong.  Neither an analog channel nor
  SIP channel passes DTMF inband if configured to inband DTMF.  Analog and
  SIP channels filter out the DTMF tones because they use the voice frames
  returned by ast_dsp_process.  But chan_misdn passes the unfiltered input
  voice frames instead.

  To overcome one aspect of the problem, the doubling of DTMF tones when two
  mISDN channels are directly bridged, someone made an 'optimization', where
  in that case the DTMF tone passed out-of-band to the peer channel is not
  translated to an inband tone at the transmit side.  This optimization is
  bad because it does not work in general.  For example, analog channels or
  mISDN channels when bridged through an intermediary local channel will
  generate DTMF tones from out-of-band information.  Also, of course, it
  must not be done when there is no inband DTMF available.

  This patch fixes the issue.  Now chan_misdn will filter the received
  inband DTMF signal the same as other channel types.

  Another change included: No need to build an extra translation path
  because ast_process_dsp does it if required.


  JIRA ABE-2080

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoRevert an errant part of a previous cleanup, to fix a memory corruption issue.
Tilghman Lesher [Thu, 18 Feb 2010 23:13:46 +0000 (23:13 +0000)]
Revert an errant part of a previous cleanup, to fix a memory corruption issue.

(closes issue #16368)
 Reported by: thirionjwf
       res_speech.c.patch uploaded by thirionjwf (license 955)

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoIf the peer record is from realtime, it could be set to 0, due to MySQL not represent...
Tilghman Lesher [Thu, 18 Feb 2010 21:42:53 +0000 (21:42 +0000)]
If the peer record is from realtime, it could be set to 0, due to MySQL not representing NULL well in integer columns.

NULL means the value is not specified for the column, which normally means
the driver uses whatever is the default value.  However, on MySQL, placing
a NULL in either a float or integer column results in a retrieval of the 0
value.  Hence, users get an errant error on load.  This patch suppresses
that error and makes the value as if it was not there.

Note that this cannot be done in the realtime driver, because the lack of
difference between NULL and 0 can only be intepreted correctly by the
driver itself.  If we did it in the realtime driver, then it would be
effectively impossible to set any realtime field to 0, because it would act
as if the field were unspecified and possibly take on a different value.

(closes issue #16683)
 Reported by: wdoekes

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agofixes confbridge crash when no timing module is loaded.
David Vossel [Thu, 18 Feb 2010 21:23:48 +0000 (21:23 +0000)]
fixes confbridge crash when no timing module is loaded.

(closes issue #16471)
Reported by: kjotte
      M16471.diff uploaded by junky (license 177)
Tested by: kjotte, junky

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agofixes Queue with C option crash
David Vossel [Thu, 18 Feb 2010 20:58:41 +0000 (20:58 +0000)]
fixes Queue with C option crash

(closes issue #16475)
Reported by: okrief
      queue_crash.diff uploaded by dvossel (license 671)

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoMerged revisions 247651 via svnmerge from
Matthew Nicholson [Thu, 18 Feb 2010 19:39:37 +0000 (19:39 +0000)]
Merged revisions 247651 via svnmerge from

  r247651 | mnicholson | 2010-02-18 13:38:09 -0600 (Thu, 18 Feb 2010) | 6 lines

  Copy the calling party's account code to the called party if they don't already have one.

  (closes issue #16331)
  Reported by: bluefox
  Tested by: mnicholson

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoFix placing ISDN calls on hold preventing native bridging from being reexamined after...
Richard Mudgett [Thu, 18 Feb 2010 18:31:44 +0000 (18:31 +0000)]
Fix placing ISDN calls on hold preventing native bridging from being reexamined after a transfer.

Consider the following scenario:

                 /-- B
A == * == Network
                 \-- C

Party B calls party A (EuroISDN BRI phone)
Party A puts B on hold using the HOLD/RETRIEVE messages.
Party A calls party C.
Party A puts C on hold to talk with party B again.
Party A transfers B to C by hanging up.

The call does not get the opportunity to get re-transferred into the ISDN
network by the native bridge because native bridging is not being
reexamined after the initial transfer.

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoMerged revisions 247508 via svnmerge from
Leif Madsen [Thu, 18 Feb 2010 16:54:43 +0000 (16:54 +0000)]
Merged revisions 247508 via svnmerge from

  r247508 | lmadsen | 2010-02-18 11:53:44 -0500 (Thu, 18 Feb 2010) | 1 line

  Add additional link to best practices document per jsmith.

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoMerged revisions 247502 via svnmerge from
Leif Madsen [Thu, 18 Feb 2010 16:41:04 +0000 (16:41 +0000)]
Merged revisions 247502 via svnmerge from

  r247502 | lmadsen | 2010-02-18 11:38:17 -0500 (Thu, 18 Feb 2010) | 10 lines

  Add best practices documentation.

  (issue #16808)
  Reported by: lmadsen

  (issue #16810)
  Reported by: Nick_Lewis
  Tested by: lmadsen


git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoAdd a new manager event for our buddies status.
Philippe Sultan [Thu, 18 Feb 2010 16:34:08 +0000 (16:34 +0000)]
Add a new manager event for our buddies status.

The new JabberStatus event gives a concise view of the status change to the AMI
clients. Thanks fiddur!

(closes issue #16760)
Reported by: fiddur
      244498.2.diff uploaded by fiddur (license 678)
Tested by: fiddur, phsultan

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoMerged revisions 247422 via svnmerge from
Russell Bryant [Thu, 18 Feb 2010 04:20:11 +0000 (04:20 +0000)]
Merged revisions 247422 via svnmerge from

  r247422 | russell | 2010-02-17 22:19:01 -0600 (Wed, 17 Feb 2010) | 10 lines

  Tweak argument handling for wget in the sounds Makefile.

  1) Fix the check to see if we are using wget to not be full of fail.  The
  configure script populates this variable with the absolute path to wget if
  it is found, so it didn't work.

  2) Allow some extra arguments to be passed in for wget.  This is just a simple
  change to allow our Bamboo build script to tell wget to be quiet and not fill
  up our logs with download status output.

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoFix a couple of bugs in test tab completion.
Mark Michelson [Wed, 17 Feb 2010 22:44:53 +0000 (22:44 +0000)]
Fix a couple of bugs in test tab completion.

1. Add missing unlock of lists.
2. Swap order of arguments to test_cat_cmp in complete_test_name.

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoTab completion for test categories and names for "test show registered" and "test...
Mark Michelson [Wed, 17 Feb 2010 22:40:43 +0000 (22:40 +0000)]
Tab completion for test categories and names for "test show registered" and "test execute" CLI commands.

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoFix two problems in ast_str functions found while writing a unit test.
Mark Michelson [Wed, 17 Feb 2010 21:22:40 +0000 (21:22 +0000)]
Fix two problems in ast_str functions found while writing a unit test.

1. The documentation for ast_str_set and ast_str_append state that
the max_len parameter may be -1 in order to limit the size of the
ast_str to its current allocated size. The problem was that the max_len
parameter in all cases was a size_t, which is unsigned. Thus a -1 was
interpreted as UINT_MAX instead of -1. Changing the max_len parameter
to be ssize_t fixed this issue.

2. Once issue 1 was fixed, there was an off-by-one error in the case
where we attempted to write a string larger than the current allotted
size to a string when -1 was passed as the max_len parameter. When trying
to write more than the allotted size, the ast_str's __AST_STR_USED was
set to 1 higher than it should have been. Thanks to Tilghman for quickly
spotting the offending line of code.

Oh, and the unit test that I referenced in the top line of this commit
will be added to reviewboard shortly. Sit tight...

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoAdd support for GROUP_MATCH_COUNT regex matching on category
Jeff Peeler [Wed, 17 Feb 2010 19:51:53 +0000 (19:51 +0000)]
Add support for GROUP_MATCH_COUNT regex matching on category

Current support for regex matching was previously only available on the group.
Also, error reporting for regex failures has been added. In addition to this
feature enhancement a unit test has been written to check the regular expression
logic to ensure the count operation is working as expected.

(closes issue #16642)
Reported by: kobaz
      groupmatch2.patch uploaded by kobaz (license 834)


git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agomodified device2extension_test's category
David Vossel [Wed, 17 Feb 2010 19:23:35 +0000 (19:23 +0000)]
modified device2extension_test's category

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agounit test for combined device state mapping and device to exten state mapping
David Vossel [Wed, 17 Feb 2010 19:18:44 +0000 (19:18 +0000)]
unit test for combined device state mapping and device to exten state mapping


git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoaddition of dynamic parkinglots feature
David Vossel [Wed, 17 Feb 2010 18:29:48 +0000 (18:29 +0000)]
addition of dynamic parkinglots feature

This feature allows for parkinglots to be created dynamically within
the dialplan.  Thanks to all who were involved with getting this patch
written and tested!

(closes issue #15135)
Reported by: IgorG
      features.dynamic_park.v3.diff uploaded by IgorG (license 20)
      2009090400_dynamicpark.diff.txt uploaded by mvanbaak (license 7)
      dynamic_parkinglot.diff uploaded by dvossel (license 671)
Tested by: eliel, IgorG, acunningham, mvanbaak, zktech


git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoMerged revisions 247168 via svnmerge from
Mark Michelson [Wed, 17 Feb 2010 16:24:54 +0000 (16:24 +0000)]
Merged revisions 247168 via svnmerge from

  r247168 | mmichelson | 2010-02-17 10:24:17 -0600 (Wed, 17 Feb 2010) | 3 lines

  Make sure that when autofill is disabled that callers not in the front of the queue cannot place calls.

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoRTP documentation states that you can pass NULL as the module, so make sure that...
Tilghman Lesher [Wed, 17 Feb 2010 07:01:13 +0000 (07:01 +0000)]
RTP documentation states that you can pass NULL as the module, so make sure that's really the case.

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoMake all of the various rtpqos parameters in this branch available from the CHANNEL...
Tilghman Lesher [Wed, 17 Feb 2010 06:25:15 +0000 (06:25 +0000)]
Make all of the various rtpqos parameters in this branch available from the CHANNEL function.

Also includes a test for retrieving rtpqos parameters, including a NULL RTP
driver.  Additionally, some further separation of the SIP internal API into
headers was necessary.

(closes issue #16652)
 Reported by: kkm
       20100204__issue16652.diff.txt uploaded by tilghman (license 14)


git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoAdd va_end calls to __ast_str_helper.
Mark Michelson [Tue, 16 Feb 2010 23:44:33 +0000 (23:44 +0000)]
Add va_end calls to __ast_str_helper.

According to the man page for stdarg(3),
"Each invocation of va_copy() must be matched by a
corresponding invocation of va_end() in the same

There were several cases in __ast_str_helper where
va_copy was not matched with a corresponding call
to va_end.

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agogenerate connected line info update from info in h.323 packets
Alexandr Anikin [Tue, 16 Feb 2010 22:58:22 +0000 (22:58 +0000)]
generate connected line info update from info in h.323 packets
Tested by: benngard

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoAdd some clarifying documentation to the ast_str_set and ast_str_append functions.
Mark Michelson [Tue, 16 Feb 2010 21:15:38 +0000 (21:15 +0000)]
Add some clarifying documentation to the ast_str_set and ast_str_append functions.

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoswap openssl with OpenSSL in warning message.
David Vossel [Tue, 16 Feb 2010 21:03:34 +0000 (21:03 +0000)]
swap openssl with OpenSSL in warning message.

(issue #16673)

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agowarning message if openssl support is missing while attempting tls connection
David Vossel [Tue, 16 Feb 2010 20:54:48 +0000 (20:54 +0000)]
warning message if openssl support is missing while attempting tls connection

(closes issue #16673)
Reported by: michaesc
      tls_error_msg.diff uploaded by dvossel (license 671)

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoAdd unit test for dialplan pattern matching.
Mark Michelson [Tue, 16 Feb 2010 18:29:42 +0000 (18:29 +0000)]
Add unit test for dialplan pattern matching.

This test works by reading input from arrays to build a sample
dialplan. From there, patterns are attempted to be matched against
said dialplan, with the expected match given. We then search in our
example dialplan to see if we find a match and if what we find matches
what we expected it to match.

(closes issue #16809)
Reported by: lmadsen
Tested by: mmichelson


git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agofixes sample rate conversion issue with Monitor application
David Vossel [Tue, 16 Feb 2010 17:07:41 +0000 (17:07 +0000)]
fixes sample rate conversion issue with Monitor application

When using ast_seekstream with the read/write streams of a monitor,
the number of samples we are seeking must be of the same rate as the
stream or the jump calculation will be incorrect.  This patch adds logic
to correctly convert the number of samples to jump to the sample rate
the read/write stream is using.

For example, if the call is G722 (16khz) and the read/write stream is
recording a 8khz wav, seeking 320 samples of 16khz audio is not the
same as seeking 320 samples of 8khz audio when performing the ast_seekstream
on the stream.


git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoRevert changes for now, pending discussion
Tilghman Lesher [Tue, 16 Feb 2010 15:36:53 +0000 (15:36 +0000)]
Revert changes for now, pending discussion

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoAdd a few more targets for DEBUG_THREADLOCALS
Tilghman Lesher [Tue, 16 Feb 2010 05:25:11 +0000 (05:25 +0000)]
Add a few more targets for DEBUG_THREADLOCALS

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoChange the blanket rules to delete .lastclean on all CFLAGS menuselect targets to...
Tilghman Lesher [Tue, 16 Feb 2010 00:52:45 +0000 (00:52 +0000)]
Change the blanket rules to delete .lastclean on all CFLAGS menuselect targets to be more particular.

This change builds upon the recent change to menuselect to add 'touch_on_change'
as an attribute of both categories and members.  This should allow only the most
invasive defines to cause a complete rebuild, while defines which only affect a
subset of modules will only cause a rebuild of that smaller set.

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoAllow Timer B to be set on the peer, and ensure SIP rules are followed (or warn)...
Tilghman Lesher [Tue, 16 Feb 2010 00:19:38 +0000 (00:19 +0000)]
Allow Timer B to be set on the peer, and ensure SIP rules are followed (or warn) in comparison to Timer T1.

(closes issue #16643)
 Reported by: nahuelgreco
       20100204__issue16643.diff.txt uploaded by tilghman (license 14)
 Tested by: oej

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoMerged revisions 246709 via svnmerge from
Tilghman Lesher [Mon, 15 Feb 2010 23:43:28 +0000 (23:43 +0000)]
Merged revisions 246709 via svnmerge from

  r246709 | tilghman | 2010-02-15 17:42:33 -0600 (Mon, 15 Feb 2010) | 5 lines

  Make the menuselect instructions correct by allowing 'make menuselect' to actually solve dependency problems.

  (Previously, it would fail out again with the same message about running
  'make menuselect', which was NOT at all helpful.)

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoRestore triedtopribridge flag code removed in -r211197.
Richard Mudgett [Mon, 15 Feb 2010 22:08:23 +0000 (22:08 +0000)]
Restore triedtopribridge flag code removed in -r211197.

Ooops.  Failed to note that we were inside a for loop and
pri_channel_bridge() needs to be executed only once.

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoInstead of just automatically filtering out in the Makefile, give an indication of...
Tilghman Lesher [Mon, 15 Feb 2010 21:37:58 +0000 (21:37 +0000)]
Instead of just automatically filtering out in the Makefile, give an indication of dependencies in menuselect.

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agochan_sip parse code refactoring plus two new unit tests
David Vossel [Mon, 15 Feb 2010 15:45:02 +0000 (15:45 +0000)]
chan_sip parse code refactoring plus two new unit tests

Code Refactoring Changes
- read_to_parts() moved to reqresp_parser.c and has been renamed as
- get_in_brackets() moved to reqresp_parser.c
- find_closing_quotes() added to sip_utils.h
Logic Changes
- get_name_and_number() now uses parse_uri() and get_calleridname()
  for parsing. Before this change only names within quotes were
  found, when names not within quotes are possible.
New Unit Tests

(closes issue #16707)
Reported by: Nick_Lewis
      issue16706.diff uploaded by dvossel (license 671)


git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoMerged revisions 246545 via svnmerge from
David Vossel [Fri, 12 Feb 2010 23:32:33 +0000 (23:32 +0000)]
Merged revisions 246545 via svnmerge from

  r246545 | dvossel | 2010-02-12 17:30:17 -0600 (Fri, 12 Feb 2010) | 16 lines

  lock channel during datastore removal

  On channel destruction the channel's datastores are removed and
  destroyed.  Since there are public API calls to find and remove
  datastores on a channel, a lock should be held whenever datastores are
  removed and destroyed.  This resolves a crash caused by a race
  condition in app_chanspy.c.

  (closes issue #16678)
  Reported by: tim_ringenbach
        datastore_destroy_race.diff uploaded by tim ringenbach (license 540)
  Tested by: dvossel

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoBlocked revisions 246460 via svnmerge
Jason Parker [Fri, 12 Feb 2010 18:54:19 +0000 (18:54 +0000)]
Blocked revisions 246460 via svnmerge

  r246460 | qwell | 2010-02-12 12:52:28 -0600 (Fri, 12 Feb 2010) | 1 line

  Fix some silly formatting, and remove unnecessary option_debug checks

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agofixes areas where port should be removed from domain during parsing
David Vossel [Fri, 12 Feb 2010 17:59:39 +0000 (17:59 +0000)]
fixes areas where port should be removed from domain during parsing

A patch was committed recently that converted duplicate uri parsing
code to use the parse_uri function.  There were two instances where
this conversion did not mimic previous behavior exactly because the
port was not being parsed off the end of the domain. In order to do
this, a dummy pointer argument needs to be passed into parse_uri so
it will know it must parse out the port from the domain.  If a port
output paramenter is not present,   the domain is returned with the
port still attached.

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoUpdated doc for OSP lookup application.
TransNexus OSP Development [Fri, 12 Feb 2010 08:30:05 +0000 (08:30 +0000)]
Updated doc for OSP lookup application.

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agofixes some test description formatting inconsistencies so log file looks nice
David Vossel [Thu, 11 Feb 2010 21:57:37 +0000 (21:57 +0000)]
fixes some test description formatting inconsistencies so log file looks nice

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoastobj2 unit test and bug fix
David Vossel [Thu, 11 Feb 2010 18:42:25 +0000 (18:42 +0000)]
astobj2 unit test and bug fix

A bug was discovered during the creation of the astobj2 unit test.
When OBJ_MULTIPLE | OBJ_UNLINK is used, the objects being returned
had a ref count issue.  This patch resolves that.


git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoAdd a test module for the event API, test_event.c.
Russell Bryant [Wed, 10 Feb 2010 23:19:16 +0000 (23:19 +0000)]
Add a test module for the event API, test_event.c.

This module includes a single test so far that creates events using two
different methods and does some verification on the result to make sure
the correct data can be retrieved from the event that was created.

One bug was found in the event API while developing this test, which makes
me happy.  :-)


git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoadditional parse_uri test and documentation
David Vossel [Wed, 10 Feb 2010 23:13:49 +0000 (23:13 +0000)]
additional parse_uri test and documentation

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agores_pktccops needs to be able to export a symbol for chan_mgcp
Tilghman Lesher [Wed, 10 Feb 2010 21:55:42 +0000 (21:55 +0000)]
res_pktccops needs to be able to export a symbol for chan_mgcp

(closes issue #16782)
 Reported by: nahuelgreco
       res_pktccops.exports uploaded by nahuelgreco (license 162)

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoFussy compiler on another machine...
Tilghman Lesher [Wed, 10 Feb 2010 21:24:10 +0000 (21:24 +0000)]
Fussy compiler on another machine...

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoFix weird issue with unit tests on optimized build - turned out to be a signing issue.
Tilghman Lesher [Wed, 10 Feb 2010 21:19:35 +0000 (21:19 +0000)]
Fix weird issue with unit tests on optimized build - turned out to be a signing issue.

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoMerged revisions 246115 via svnmerge from
David Vossel [Wed, 10 Feb 2010 17:49:34 +0000 (17:49 +0000)]
Merged revisions 246115 via svnmerge from

  r246115 | dvossel | 2010-02-10 11:44:20 -0600 (Wed, 10 Feb 2010) | 8 lines

  fixes random deadlock in app_queue with use_weight during reload

  (closes issue #16677)
  Reported by: tim_ringenbach
        app_queue_use_weight_deadlock.diff uploaded by tim ringenbach (license 540)

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoChange channel state on local channels for busy,answer,ring.
Jeff Peeler [Wed, 10 Feb 2010 16:47:37 +0000 (16:47 +0000)]
Change channel state on local channels for busy,answer,ring.

Previously local channels channel state never changed. This became problematic
when the state of the other side of the local channel was lost, for example
during a masquerade. Changing the state of the local channel allows for the
scenario to be detected when the channel state is set to ringing, but the peer
isn't ringing. The specific problem scenario is described in 164201. Although
this was noted on one of the issues, here is the tested dialplan verified to

exten => 9700,1,Dial(Local/*9700@default&Local/0009700@default)

exten => *9700,1,Set(GLOBAL(TESTCHAN)=${CHANNEL:0:${MATH(${LEN(${CHANNEL})}-1):0:2}}1)
exten => *9700,n,wait(3) ;3 works, 1 did not
exten => *9700,n,Dial(SIP/5001)

exten => 0009700,1,Wait(1) ;1 works, 3 did not
exten => 0009700,n,ChannelRedirect(${TESTCHAN},parkedcalls,701,1)

(closes issue #14992)
Reported by: davidw

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoSolaris doesn't like outputting a NULL to a %s in format strings.
Tilghman Lesher [Wed, 10 Feb 2010 16:01:28 +0000 (16:01 +0000)]
Solaris doesn't like outputting a NULL to a %s in format strings.

Detect all platforms that don't like that, either, and ensure that when documentation is
missing, we pass a non-NULL pointer when outputting the corresponding documentation.

(closes issue #16689)
 Reported by: bklang
       20100209__issue16689__with_tests.diff.txt uploaded by tilghman (license 14)


git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoEnable warnings on atypical conditions for the FILTER function (suggested by mmichels...
Tilghman Lesher [Wed, 10 Feb 2010 15:36:57 +0000 (15:36 +0000)]
Enable warnings on atypical conditions for the FILTER function (suggested by mmichelson on the -dev list).

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoMerged revisions 245944 via svnmerge from
Tilghman Lesher [Wed, 10 Feb 2010 14:06:12 +0000 (14:06 +0000)]
Merged revisions 245944 via svnmerge from

  r245944 | tilghman | 2010-02-10 07:37:13 -0600 (Wed, 10 Feb 2010) | 2 lines

  Include examples of FILTER usage in extension patterns where a "." may be a risk.

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

11 years agoVarious updates to the unit test API.
Russell Bryant [Tue, 9 Feb 2010 23:32:14 +0000 (23:32 +0000)]
Various updates to the unit test API.

1) It occurred to me that the difference in usage between the error ast_str and
the ast_test_update_status() usage has turned out to be a bit ambiguous in
practice.  In a lot of cases, the same message was being sent to both.
In other cases, it was only sent to one or the other.  My opinion now is that
in every case, I think it makes sense to do both; we should output it to the
CLI as well as save it off for logging purposes.

This change results in most of the changes in this diff, since it required
changes to all existing unit tests.  It also allowed for some simplifications
of unit test API implementation code.

2) Update ast_test_status_update() to include the file, function, and line
number for the code providing the update.

3) There are some formatting tweaks here and there.  Hopefully they aren't too
distracting for code review purposes.  Reviewboard's diff viewer seems to do a
pretty good job of pointing out when something is a whitespace change.

4) I moved the md5_test and sha1_test into the test_utils module.  It seemed
like a better approach since these tests are so tiny.

5) I changed the number of nodes used in heap_test_2 from 1 million to
100 thousand.  The only reason for this was to reduce the time it took
for this test to run.

6) Remove an unused function prototype that was at the bottom of utils.h.

7) Simplify test_insert() using the LIST_INSERT_SORTALPHA() macro.  The one
minor difference in behavior is that it no longer checks for a test registered
with the same name.

8) Expand the code in test_alloc() to provide specific error messages for each
failure case, to clearly inform developers if they forget to set the name,
summary, description, etc.

9) Tweak the output of the "test show registered" CLI command.  I swapped the
name and category to have the category first.  It seemed more natural since
that is the sort key.

10) Don't output the status ast_str in the "test show results" CLI command.
This is going to tend to be pretty verbose, so just leave that for the
detailed test logs (test generate results).


git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3