asterisk/asterisk.git
10 years agoBlocked revisions 182965 via svnmerge
Jeff Peeler [Wed, 18 Mar 2009 20:03:28 +0000 (20:03 +0000)]
Blocked revisions 182965 via svnmerge

........
  r182965 | jpeeler | 2009-03-18 15:02:40 -0500 (Wed, 18 Mar 2009) | 1 line

  fix typo which broke configure
........

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

10 years agoBlocked revisions 182963 via svnmerge
Jeff Peeler [Wed, 18 Mar 2009 19:57:35 +0000 (19:57 +0000)]
Blocked revisions 182963 via svnmerge

........
  r182963 | jpeeler | 2009-03-18 14:57:05 -0500 (Wed, 18 Mar 2009) | 15 lines

  Allow H.323 Plus library to be used in addition to the OpenH323 library

  Chan_h323 can now be compiled against both the previously supported versions of
  OpenH323 as well as the current H.323 Plus (version 1.20.2). The configure
  script has been modified to look in the default install location of h323 to
  hopefully help avoid using the environment variables OPENH323DIR and PWLIBDIR.
  Also, the CLI command "h323 show version" has been added which indicates which
  version of h323 is in use.

  (closes issue 0011261)
  Reported by: vhatz
  Patches:
        asterisk-1.6.0.6-h323plus.patch uploaded by jthurman (license 614)
........

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

10 years agoFixing a lost symbol in manager.c
Tilghman Lesher [Wed, 18 Mar 2009 19:41:57 +0000 (19:41 +0000)]
Fixing a lost symbol in manager.c

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

10 years agoMerged revisions 182882 via svnmerge from
Kevin P. Fleming [Wed, 18 Mar 2009 11:40:11 +0000 (11:40 +0000)]
Merged revisions 182882 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r182882 | kpfleming | 2009-03-18 06:31:41 -0500 (Wed, 18 Mar 2009) | 3 lines

  fix another symbol namespace issue (reported by Andrew on asterisk-dev)
........

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

10 years agoa few more namespace updates... res_ael_share still needs some work before this can...
Kevin P. Fleming [Wed, 18 Mar 2009 02:39:36 +0000 (02:39 +0000)]
a few more namespace updates... res_ael_share still needs some work before this can be merged to other release branches

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

10 years agoMerged revisions 182810 via svnmerge from
Russell Bryant [Wed, 18 Mar 2009 02:28:55 +0000 (02:28 +0000)]
Merged revisions 182810 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r182810 | russell | 2009-03-17 21:09:13 -0500 (Tue, 17 Mar 2009) | 44 lines

Fix cases where the internal poll() was not being used when it needed to be.

We have seen a number of problems caused by poll() not working properly on
Mac OSX.  If you search around, you'll find a number of references to using
select() instead of poll() to work around these issues.  In Asterisk, we've
had poll.c which implements poll() using select() internally.  However, we
were still getting reports of problems.

vadim investigated a bit and realized that at least on his system, even
though we were compiling in poll.o, the system poll() was still being used.
So, the primary purpose of this patch is to ensure that we're using the
internal poll() when we want it to be used.

The changes are:

1) Remove logic for when internal poll should be used from the Makefile.
   Instead, put it in the configure script.  The logic in the configure
   script is the same as it was in the Makefile.  Ideally, we would have
   a functionality test for the problem, but that's not actually possible,
   since we would have to be able to run an application on the _target_
   system to test poll() behavior.

2) Always include poll.o in the build, but it will be empty if AST_POLL_COMPAT
   is not defined.

3) Change uses of poll() throughout the source tree to ast_poll().  I feel
   that it is good practice to give the API call a new name when we are
   changing its behavior and not using the system version directly in all cases.
   So, normally, ast_poll() is just redefined to poll().  On systems where
   AST_POLL_COMPAT is defined, ast_poll() is redefined to ast_internal_poll().

4) Change poll() in main/poll.c to be ast_internal_poll().

It's worth noting that any code that still uses poll() directly will work fine
(if they worked fine before).  So, for example, out of tree modules that are
using poll() will not stop working or anything.  However, for modules to work
properly on Mac OSX, ast_poll() needs to be used.

(closes issue #13404)
Reported by: agalbraith
Tested by: russell, vadim

http://reviewboard.digium.com/r/198/

........

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

10 years agoMerged revisions 182808 via svnmerge from
Kevin P. Fleming [Wed, 18 Mar 2009 02:21:23 +0000 (02:21 +0000)]
Merged revisions 182808 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r182808 | kpfleming | 2009-03-17 20:55:22 -0500 (Tue, 17 Mar 2009) | 5 lines

  Improve the build system to *properly* remove unnecessary symbols from the runtime global namespace. Along the way, change the prefixes on some internal-only API calls to use a common prefix.

  With these changes, for a module to export symbols into the global namespace, it must have *both* the AST_MODFLAG_GLOBAL_SYMBOLS flag and a linker script that allows the linker to leave the symbols exposed in the module's .so file (see res_odbc.exports for an example).
........

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

10 years agoAdd support for the "name" option in the CHANNEL() function.
Russell Bryant [Tue, 17 Mar 2009 21:28:04 +0000 (21:28 +0000)]
Add support for the "name" option in the CHANNEL() function.

Review: http://reviewboard.digium.com/r/199/

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

10 years agoAllow H.323 Plus library to be used in addition to the OpenH323 library
Jeff Peeler [Tue, 17 Mar 2009 20:47:31 +0000 (20:47 +0000)]
Allow H.323 Plus library to be used in addition to the OpenH323 library

Chan_h323 can now be compiled against both the previously supported versions of
OpenH323 as well as the current H.323 Plus (version 1.20.2). The configure
script has been modified to look in the default install location of h323 to
hopefully help avoid using the environment variables OPENH323DIR and PWLIBDIR.
Also, the CLI command "h323 show version" has been added which indicates which
version of h323 is in use.

(closes issue #11261)
Reported by: vhatz
Patches:
      asterisk-1.6.0.6-h323plus.patch uploaded by jthurman (license 614)

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

10 years agoBlocked revisions 182652 via svnmerge
Jason Parker [Tue, 17 Mar 2009 20:14:17 +0000 (20:14 +0000)]
Blocked revisions 182652 via svnmerge

........
  r182652 | qwell | 2009-03-17 15:13:40 -0500 (Tue, 17 Mar 2009) | 7 lines

  Allow dahdichanname to work as advertised.

  (closes issue #14056)
  Reported by: dsedivec
  Patches:
        load_from_zapata_conf.patch uploaded by dsedivec (license 638)
........

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

10 years agoFixing CHANGES in rev 182596.
David Vossel [Tue, 17 Mar 2009 18:06:55 +0000 (18:06 +0000)]
Fixing CHANGES in rev 182596.

Progress DTMF was added into app_dial's D() option.  In CHANGES it should have been updated under 1.6.3 rather than 1.6.2.

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

10 years agoOption to send DTMF when receiving PROGRESS status
David Vossel [Tue, 17 Mar 2009 17:17:51 +0000 (17:17 +0000)]
Option to send DTMF when receiving PROGRESS status

The D() option in app_dial is only able to send DTMF after the call has been answered.  A progress option has been added to D() to allow DTMF to be sent upon receiving PROGRESS.  This allows DTMF to be sent before the call is answered.

(closes issue #12123)
Reported by: VoipForces
Patches:
app_dial.c_patch_trunk_valid uploaded by VoipForces (license 419)
dtmf_progress.patch uploaded by dvossel (license 671)
Tested by: VoipForces, dvossel

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

10 years agoTweak the handling of the frame list inside of ast_answer().
Russell Bryant [Tue, 17 Mar 2009 15:22:12 +0000 (15:22 +0000)]
Tweak the handling of the frame list inside of ast_answer().

This does not change any behavior, but moves the frames from the local frame
list back to the channel read queue using an O(n) algorithm instead of O(n^2).

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

10 years agocorrect logic flaw in ast_answer() changes in r182525
Kevin P. Fleming [Tue, 17 Mar 2009 14:59:33 +0000 (14:59 +0000)]
correct logic flaw in ast_answer() changes in r182525

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

10 years agoImprove behavior of ast_answer() to not lose incoming frames
Kevin P. Fleming [Tue, 17 Mar 2009 14:38:11 +0000 (14:38 +0000)]
Improve behavior of ast_answer() to not lose incoming frames

ast_answer(), when supplied a delay before returning to the caller, use ast_safe_sleep() to implement the delay. Unfortunately during this time any incoming frames are discarded, which is problematic for T.38 re-INVITES and other sorts of channel operations.

When a delay is not passed to ast_answer(), it still delays for up to 500 milliseconds, waiting for media to arrive. Again, though, it discards any control frames, or non-voice media frames.

This patch rectifies this situation, by storing all incoming frames during the delay period on a list, and then requeuing them onto the channel before returning to the caller.

http://reviewboard.digium.com/r/196/

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

10 years agoDon't include a space before the optional extra text that may follow a help
Sean Bright [Tue, 17 Mar 2009 14:24:53 +0000 (14:24 +0000)]
Don't include a space before the optional extra text that may follow a help
string.

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

10 years agoMerged revisions 182449 via svnmerge from
Tilghman Lesher [Tue, 17 Mar 2009 05:51:54 +0000 (05:51 +0000)]
Merged revisions 182449 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r182449 | tilghman | 2009-03-17 00:50:52 -0500 (Tue, 17 Mar 2009) | 7 lines

  Fix race in astdb
  The underlying db1 implementation does not fully isolate the pages retrieved
  from astdb, so the lock protecting accesses needs to be extended until the
  copy from the shared memory structure is done.
  (closes issue #14682)
   Reported by: makoto
........

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

10 years agoOPENR2 uses an incorrect string value if the extension delimiter is not present.
Richard Mudgett [Tue, 17 Mar 2009 01:54:53 +0000 (01:54 +0000)]
OPENR2 uses an incorrect string value if the extension delimiter is not present.

*  Fixed OPENR2 using an incorrect string value if the extension
delimiter is not present in the Dial() function.  This was fixed for
SS7 and PRI in trunk -r172400.
*  Made OPENR2 stripmsd behavior the same as the SS7, PRI, and others.
*  Removed trailing whitespace that appeared with OPENR2.

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

10 years agoUpdate UPGRADE.txt and CHANGES for 1.6.3
Russell Bryant [Mon, 16 Mar 2009 20:53:21 +0000 (20:53 +0000)]
Update UPGRADE.txt and CHANGES for 1.6.3

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

10 years agoAdd MFC/R2 support for chan_dahdi.
Russell Bryant [Mon, 16 Mar 2009 20:35:58 +0000 (20:35 +0000)]
Add MFC/R2 support for chan_dahdi.

This commit introduces official support for R2 signaling in chan_dahdi.  The
modifications to chan_dahdi, and the supporting library, LibOpenR2, were both
written by Moises Silva.

Many users are using this code, or a variant of it, in Asterisk 1.2, 1.4 and 1.6
in Brazil, México and Argentina. An unknown number of users (but at least 1)
are using it in each of the following countries: Colombia, Nepal, Thailand,
Venezuela, Perú, and probably others.

To use this code, LibOpenR2 must be installed from http://www.libopenr2.org/.
Information about configuration can be found in configs/chan_dahdi.conf.sample.

The code committed is the most up to date version, which was being maintained
in svn/asterisk/team/moy/mfcr2/.

I would also like to include a Thank You to the many others that tested this
code beyond those listed in this commit message.  These are the names that I
could find in the mantis issue.

(closes issue #12509)
Reported by: moy
Patches:
      chan_zap-mfr2.patch uploaded by moy (license 222)
Tested by: moy, korihor, viniciusfontes, Skarmeth, loloski, asbestoshead, titogarrido, heliocoelhojr, konsultex, ncorrare, ecarruda, rtorresduque, PTorres, ychen

Review: http://reviewboard.digium.com/r/40/

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

10 years agoMerged revisions 182281 via svnmerge from
David Vossel [Mon, 16 Mar 2009 17:49:58 +0000 (17:49 +0000)]
Merged revisions 182281 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r182281 | dvossel | 2009-03-16 12:47:42 -0500 (Mon, 16 Mar 2009) | 7 lines

  Randomize IAX2 encryption padding

  The 16-32 byte random padding at the beginning of an encrypted IAX2 frame turns out to not be all that random at all.  This patch calls ast_random to fill the padding buffer with random data.  The padding is randomized at the beginning of every encrypted call and for every encrypted retransmit frame.

  Review: http://reviewboard.digium.com/r/193/
........

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

10 years agoFix an off-by-one error in the FILE() function, and extend FILE()'s length parameter...
Tilghman Lesher [Mon, 16 Mar 2009 17:33:38 +0000 (17:33 +0000)]
Fix an off-by-one error in the FILE() function, and extend FILE()'s length parameter to work like variable substitution.
Previously, FILE() returned one less character than specified, due to the
terminating NULL.  Both the offset and length parameters now behave
identically to the way variable substitution offsets and lengths also work.
(closes issue #14670)
 Reported by: BMC

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

10 years agoMerged revisions 182208 via svnmerge from
Tilghman Lesher [Mon, 16 Mar 2009 15:50:55 +0000 (15:50 +0000)]
Merged revisions 182208 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r182208 | tilghman | 2009-03-16 10:39:15 -0500 (Mon, 16 Mar 2009) | 7 lines

  Fixup glare detection, to fix a memory leak of a local pvt structure.
  (closes issue #14656)
   Reported by: caspy
   Patches:
         20090313__bug14656__2.diff.txt uploaded by tilghman (license 14)
   Tested by: caspy
........

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

10 years agoFix a memory leak in the ast_answer / __ast_answer API call.
Joshua Colp [Mon, 16 Mar 2009 13:58:24 +0000 (13:58 +0000)]
Fix a memory leak in the ast_answer / __ast_answer API call.

For a channel that is not yet answered this API call will wait
until a voice frame is received on the channel before returning.
It does this by waiting for frames on the channel and reading them
in. The frames read in were not freed when they should have been.

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

10 years agoChange faulty comparison used when announcing average hold minutes and seconds
Mark Michelson [Fri, 13 Mar 2009 21:26:20 +0000 (21:26 +0000)]
Change faulty comparison used when announcing average hold minutes and seconds

(closes issue #14227)
Reported by: caspy

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

10 years agoRemove ast_ prefix from functions which are not public.
Mark Michelson [Fri, 13 Mar 2009 17:49:01 +0000 (17:49 +0000)]
Remove ast_ prefix from functions which are not public.

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

10 years agoMerged revisions 181990 via svnmerge from
Mark Michelson [Fri, 13 Mar 2009 17:26:43 +0000 (17:26 +0000)]
Merged revisions 181990 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r181990 | mmichelson | 2009-03-13 12:12:32 -0500 (Fri, 13 Mar 2009) | 35 lines

  Check the DYNAMIC_FEATURES of both the chan and peer when interpreting DTMF.

  Dynamic features defined in the applicationmap section of features.conf allow
  one to specify whether the caller, callee, or both have the ability to use the
  feature. The documentation in the features.conf.sample file could be interpreted
  to mean that one only needs to set the DYNAMIC_FEATURES channel variable on the
  calling channel in order to allow for the callee to be able to use the features
  which he should have permission to use. However, the DYNAMIC_FEATURES variable
  would only be read from the channel of the participant that pressed the DTMF
  sequence to activate the feature. The result of this was that the callee was
  unable to use dynamic features unless the dialplan writer had taken measures
  to be sure that the DYNAMIC_FEATURES variable was set on the callee's channel.

  This commit changes the behavior of ast_feature_interpret to concatenate the
  values of DYNAMIC_FEATURES from both parties involved in the bridge. The features
  themselves determine who has permission to use them, so there is no reason to believe
  that one side of the bridge could gain the ability to perform an action that they
  should not have the ability to perform.

  Kevin Fleming pointed out on the asterisk-users list that the typical way that this
  was worked around in the past was by setting _DYNAMIC_FEATURES on the calling channel
  so that the value would be inherited by the called channel. While this works, the
  documentation alone is not enough to figure out why this is necessary for the callee
  to be able to use dynamic features. In this particular case, changing the code to match
  the documentation is safe, easy, and will generally make things easier for people for
  future installations.

  This bug was originally reported on the asterisk-users list by David Ruggles.

  (closes issue #14657)
  Reported by: mmichelson
  Patches:
        14657.patch uploaded by mmichelson (license 60)
........

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

10 years agoFix an issue with requesting a T38 reinvite before the call is answered.
Joshua Colp [Fri, 13 Mar 2009 17:25:09 +0000 (17:25 +0000)]
Fix an issue with requesting a T38 reinvite before the call is answered.

The code responsible for sending the T38 reinvite did not check if an INVITE was
already being handled. This caused things to get confused and the call to fail.
The code now defers sending the T38 reinvite until the current INVITE is done being
handled.

(issue AST-191)

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

10 years agoimprove a bit of suboptimal code
Kevin P. Fleming [Fri, 13 Mar 2009 16:55:38 +0000 (16:55 +0000)]
improve a bit of suboptimal code

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

10 years agoMerged revisions 181898 via svnmerge from
Richard Mudgett [Fri, 13 Mar 2009 01:26:22 +0000 (01:26 +0000)]
Merged revisions 181898 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

Just recording the v1.4 change in trunk since it originally came from here.

........
  r181898 | rmudgett | 2009-03-12 20:19:29 -0500 (Thu, 12 Mar 2009) | 4 lines

  Use the correct branch integrated property when generating the version string.

  Copied the make_version file from Asterisk trunk.
........

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

10 years agoRun the macro on the queue member's channel when he answers, not the caller's channel.
Mark Michelson [Thu, 12 Mar 2009 21:43:51 +0000 (21:43 +0000)]
Run the macro on the queue member's channel when he answers, not the caller's channel.

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

10 years agoMerged revisions 181768 via svnmerge from
Mark Michelson [Thu, 12 Mar 2009 18:30:58 +0000 (18:30 +0000)]
Merged revisions 181768 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r181768 | mmichelson | 2009-03-12 13:29:48 -0500 (Thu, 12 Mar 2009) | 22 lines

  Properly send a 487 on an INVITE we have not responded to if we receive a BYE.

  If we receive an INVITE from an endpoint and then later receive a BYE from that
  same endpoint before we have sent a final response for the INVITE, then we need
  to respond to the INVITE with a 487.

  There was logic in the code prior to this commit which seemed to exist solely to
  handle this situation, but there was one condition in an if statement which
  was incorrect. The only way we would send a 487 was if the sip_pvt had no owner
  channel. This made no sense since we created the owner channel when we received
  the INVITE, meaning that the majority of the time we would never send the 487.
  The 487 being sent should not rely on whether we have created a channel. Its
  delivery should be dependent on the current state of the initial INVITE transaction.
  With this commit, that logic is now correctly in place.

  (closes issue #14149)
  Reported by: legranjl
  Patches:
        14149.patch uploaded by mmichelson (license 60)
  Tested by: legranjl
........

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

10 years agoAdjust translation table column widths based upon the translation times.
Tilghman Lesher [Thu, 12 Mar 2009 17:32:13 +0000 (17:32 +0000)]
Adjust translation table column widths based upon the translation times.
Previously, only 5 columns were displayed, and if a translation time exceeded
99,999 useconds, it would be displayed as 0, instead of its actual time.
(closes issue #14532)
 Reported by: pj
 Patches:
       20090311__bug14532.diff.txt uploaded by tilghman (license 14)
 Tested by: pj

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

10 years agoMerged revisions 181664 via svnmerge from
Joshua Colp [Thu, 12 Mar 2009 16:56:58 +0000 (16:56 +0000)]
Merged revisions 181664 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r181664 | file | 2009-03-12 13:56:20 -0300 (Thu, 12 Mar 2009) | 2 lines

  Fix incorrect usage of strncasecmp... I really meant to use strcasecmp.
........

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

10 years agoMerged revisions 181659-181660 via svnmerge from
Joshua Colp [Thu, 12 Mar 2009 16:53:52 +0000 (16:53 +0000)]
Merged revisions 181659-181660 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r181659 | file | 2009-03-12 13:50:37 -0300 (Thu, 12 Mar 2009) | 8 lines

  Fix another scenario where depending on configuration the stream would not get read.

  For custom commands we don't know whether the audio is coming from a stream or not
  so we are going to have to read the data despite no channels.

  (closes issue #14416)
  Reported by: caspy
........
  r181660 | file | 2009-03-12 13:52:45 -0300 (Thu, 12 Mar 2009) | 2 lines

  Fix logic flaw in previous commit.
........

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

10 years agoMerged revisions 181655 via svnmerge from
Joshua Colp [Thu, 12 Mar 2009 16:32:20 +0000 (16:32 +0000)]
Merged revisions 181655 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r181655 | file | 2009-03-12 13:29:19 -0300 (Thu, 12 Mar 2009) | 10 lines

  Fix issue with streaming MOH failing if nobody is listening.

  When a music class is setup to actually provide music on hold
  from a stream we need to constantly read audio from it since it
  will constantly be providing audio. This is now done despite there
  being no channels listening to it.

  (closes issue #14416)
  Reported by: caspy
........

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

10 years agoFix crash when sleep and retries argument was not given to RetryDial application.
Joshua Colp [Thu, 12 Mar 2009 13:24:12 +0000 (13:24 +0000)]
Fix crash when sleep and retries argument was not given to RetryDial application.

(closes issue #14647)
Reported by: sherpya

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

10 years agoWhitespace chages.
Richard Mudgett [Thu, 12 Mar 2009 01:33:04 +0000 (01:33 +0000)]
Whitespace chages.

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

10 years agoUse the correct branch integrated property when generating the version string
Richard Mudgett [Thu, 12 Mar 2009 01:00:29 +0000 (01:00 +0000)]
Use the correct branch integrated property when generating the version string

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

10 years agoProvide correct hint to debug SIP trouble in the default config
Michiel van Baak [Wed, 11 Mar 2009 23:14:22 +0000 (23:14 +0000)]
Provide correct hint to debug SIP trouble in the default config

(closes issue #14646)
Reported by: strk

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

10 years agoMake handling of the BRIDGE_PLAY_SOUND variable thread-safe.
Russell Bryant [Wed, 11 Mar 2009 22:25:57 +0000 (22:25 +0000)]
Make handling of the BRIDGE_PLAY_SOUND variable thread-safe.

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

10 years agoMerged revisions 181436 via svnmerge from
Jason Parker [Wed, 11 Mar 2009 22:20:13 +0000 (22:20 +0000)]
Merged revisions 181436 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r181436 | qwell | 2009-03-11 17:18:42 -0500 (Wed, 11 Mar 2009) | 4 lines

  Allow prefix to set localstatedir (when used and different from the default).

  This is similar to the /etc change that was made for the non-FreeBSD case.
........

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

10 years agoMake handling of the BRIDGEPVTCALLID variable thread-safe.
Russell Bryant [Wed, 11 Mar 2009 22:14:55 +0000 (22:14 +0000)]
Make handling of the BRIDGEPVTCALLID variable thread-safe.

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

10 years agoMerged revisions 181423 via svnmerge from
Russell Bryant [Wed, 11 Mar 2009 21:49:29 +0000 (21:49 +0000)]
Merged revisions 181423 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r181423 | russell | 2009-03-11 16:42:58 -0500 (Wed, 11 Mar 2009) | 9 lines

Make code that updates BRIDGEPEER variable thread-safe.

It is not safe to read the name field of an ast_channel without the channel
locked.  This patch fixes some places in channel.c where this was being done,
and lead to crashes related to masquerades.

(closes issue #14623)
Reported by: guillecabeza

........

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

10 years agoMerged revisions 181340 via svnmerge from
David Vossel [Wed, 11 Mar 2009 17:34:57 +0000 (17:34 +0000)]
Merged revisions 181340 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r181340 | dvossel | 2009-03-11 12:25:31 -0500 (Wed, 11 Mar 2009) | 11 lines

  encrypted IAX2 during packet loss causes decryption to fail on retransmitted frames

  If an iax channel is encrypted, and a retransmit frame is sent, that packet's iseqno is updated while it is encrypted.  This causes the entire frame to be corrupted.  When the corrupted frame is sent, the other side decrypts it and sends a VNAK back because the decrypted frame doesn't make any sense.  When we get the VNAK, we look through the sent queue and send the same corrupted frame causing a loop.  To fix this, encrypted frames requiring retransmission are decrypted, updated, then re-encrypted.  Since key-rotation may change the key held by the pvt struct, the keys used for encryption/decryption are held within the iax_frame to guarantee they remain correct.

  (closes issue #14607)
  Reported by: stevenla
  Tested by: dvossel

  Review: http://reviewboard.digium.com/r/192/
........

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

10 years agoMerged revisions 181328 via svnmerge from
Joshua Colp [Wed, 11 Mar 2009 17:26:40 +0000 (17:26 +0000)]
Merged revisions 181328 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r181328 | file | 2009-03-11 14:22:52 -0300 (Wed, 11 Mar 2009) | 14 lines

  Fix issue where an attended transfer could not be completed under a rare scenario.

  When completing an attended transfer chan_sip does a check to make sure the extension
  in the URI portion of the Refer-To header is a local valid extension. We don't actually
  need to check this since we know for sure the other channel is already up and talking to
  the extension. Some devices do not put the extension in the Refer-To header either, which
  can cause the extension check to fail. We now no longer do this check if it is an attended
  transfer.

  (closes issue #14628)
  Reported by: sverre
  Patches:
        14628.diff uploaded by file (license 11)
........

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

10 years agoTurn off malloc debugging of astobj2, since it apparently doesn't work too well durin...
Tilghman Lesher [Wed, 11 Mar 2009 17:04:46 +0000 (17:04 +0000)]
Turn off malloc debugging of astobj2, since it apparently doesn't work too well during startup.

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

10 years agoMerged revisions 181295 via svnmerge from
Joshua Colp [Wed, 11 Mar 2009 16:40:48 +0000 (16:40 +0000)]
Merged revisions 181295 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r181295 | file | 2009-03-11 13:36:50 -0300 (Wed, 11 Mar 2009) | 9 lines

  Fix a problem with inband DTMF detection on outgoing SIP calls when dtmfmode=auto.

  When dtmfmode was set to auto the inband DTMF detector was not setup
  on outgoing SIP calls. This caused inband DTMF detection to fail.
  The inband DTMF detector is now setup for both dtmfmode inband and auto.

  (closes issue #13713)
  Reported by: makoto
........

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

10 years agoReplace contents of this doc with a pointer to its new home
Russell Bryant [Wed, 11 Mar 2009 16:19:38 +0000 (16:19 +0000)]
Replace contents of this doc with a pointer to its new home

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

10 years agoFix segfault when dialing a typo'd queue
Mark Michelson [Wed, 11 Mar 2009 14:28:40 +0000 (14:28 +0000)]
Fix segfault when dialing a typo'd queue

If trying to dial a non-existent queue, there would
be a segfault when attempting to access q->weight, even
though q was NULL. This problem was introduced during
the queue-reset merge and thus only affects trunk.

(closes issue #14643)
Reported by: alecdavis

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

10 years agoDon't play the "you are about to be placed into the conference" and "the leader has...
Joshua Colp [Wed, 11 Mar 2009 13:44:42 +0000 (13:44 +0000)]
Don't play the "you are about to be placed into the conference" and "the leader has left the conference" sounds if the quiet
option is enabled. (reported by Vadim Lebedev on the asterisk-dev list)

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

10 years agoFix malloc debug macros to work properly with h323.
Jeff Peeler [Wed, 11 Mar 2009 04:06:44 +0000 (04:06 +0000)]
Fix malloc debug macros to work properly with h323.

The main problem here was that cstdlib was undefining free thereby causing the
proper debug macros to not be used. ast_h323.cxx has been changed to call
ast_free instead to avoid the issue.

A few other issues were addressed:
- There were a few instances of functions improperly passing ast_free instead
of ast_free_ptr.
- Some clean up was done to avoid the debug macros intentionally being redefined.
(copied below from Kevin's commit, appreciate the help)
- disable astmm.h from doing anything when STANDALONE is defined, which is used
by the tools in the utils/ directory that use parts of Asterisk header files in
hackish ways; also ensure that utils/extconf.c and utils/conf2ael.c are
compiled with STANDALONE defined.

(closes issue #13593)
Reported by: pj

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

10 years agoBlocked revisions 181133 via svnmerge
Jeff Peeler [Wed, 11 Mar 2009 03:30:19 +0000 (03:30 +0000)]
Blocked revisions 181133 via svnmerge

........
  r181133 | jpeeler | 2009-03-10 22:25:04 -0500 (Tue, 10 Mar 2009) | 13 lines

  Fix malloc debug macros to work properly with h323.

  The main problem here was that cstdlib was undefining free thereby causing the
  proper debug macros to not be used. ast_h323.cxx has been changed to call
  ast_free instead to avoid the issue. Because using the ast prefix calls are
  a better choice, ast_free_ptr is the new wrapper for free to pass to functions.
  Also, a little bit of clean up was done to avoid the debug macros intentionally
  being redefined.

  (closes issue #13593)
  Reported by: pj
........

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

10 years agotabs to spaces
Russell Bryant [Wed, 11 Mar 2009 02:25:24 +0000 (02:25 +0000)]
tabs to spaces

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

10 years agoAdd missing comment that quotes RFC 3891
Mark Michelson [Wed, 11 Mar 2009 00:49:00 +0000 (00:49 +0000)]
Add missing comment that quotes RFC 3891

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

10 years agoMerged revisions 181029,181031 via svnmerge from
Mark Michelson [Wed, 11 Mar 2009 00:46:47 +0000 (00:46 +0000)]
Merged revisions 181029,181031 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r181029 | mmichelson | 2009-03-10 19:30:26 -0500 (Tue, 10 Mar 2009) | 9 lines

  Fix incorrect tag checking on transfers when pedantic=yes is enabled.

  (closes issue #14611)
  Reported by: klaus3000
  Patches:
        patch_chan_sip_attended_transfer_1.4.23.txt uploaded by klaus3000 (license 65)
  Tested by: klaus3000
........
  r181031 | mmichelson | 2009-03-10 19:32:40 -0500 (Tue, 10 Mar 2009) | 3 lines

  Remove unused variables.
........

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

10 years agoAdd MALLOC_DEBUG to various utility APIs, so that memory leaks can be tracked back...
Tilghman Lesher [Wed, 11 Mar 2009 00:29:59 +0000 (00:29 +0000)]
Add MALLOC_DEBUG to various utility APIs, so that memory leaks can be tracked back to their source.
(related to issue #14636)

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

10 years agoSpacing changes only
Tilghman Lesher [Wed, 11 Mar 2009 00:28:28 +0000 (00:28 +0000)]
Spacing changes only

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

10 years agoMerged revisions 180941 via svnmerge from
Jason Parker [Tue, 10 Mar 2009 22:03:41 +0000 (22:03 +0000)]
Merged revisions 180941 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r180941 | qwell | 2009-03-10 17:02:18 -0500 (Tue, 10 Mar 2009) | 1 line

  Make things happier when using autoconf 2.62+
........

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

10 years agoAdd some notes on getting in contact with the dev community
Russell Bryant [Tue, 10 Mar 2009 22:03:16 +0000 (22:03 +0000)]
Add some notes on getting in contact with the dev community

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

10 years agoRemove difficulty and language specifiers
Russell Bryant [Tue, 10 Mar 2009 21:55:49 +0000 (21:55 +0000)]
Remove difficulty and language specifiers

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

10 years agoExpand upon documentation of manager event project
Russell Bryant [Tue, 10 Mar 2009 21:45:54 +0000 (21:45 +0000)]
Expand upon documentation of manager event project

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

10 years agolist the move of the astvarrundir from /var/run to /var/run/asterisk
Michiel van Baak [Tue, 10 Mar 2009 21:15:29 +0000 (21:15 +0000)]
list the move of the astvarrundir from /var/run to /var/run/asterisk
(actually its $(localstatedir)/run/asterisk

Makes setups with asterisk as non-root easier to manage because you can
setup permissions on this dir instead of touching a file and setting
permissions on that.
Files that come to mind are asterisk.pid and asterisk.ctl socket.

Prodded by and ok @russell

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

10 years agoadd more projects
Russell Bryant [Tue, 10 Mar 2009 19:36:21 +0000 (19:36 +0000)]
add more projects

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

10 years agoadd more project ideas
Russell Bryant [Tue, 10 Mar 2009 19:23:41 +0000 (19:23 +0000)]
add more project ideas

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

10 years agoReset the thread local string buffer when handling the UserEvent action.
Joshua Colp [Tue, 10 Mar 2009 14:40:38 +0000 (14:40 +0000)]
Reset the thread local string buffer when handling the UserEvent action.

(closes issue #14593)
Reported by: JimDickenson

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

10 years agoAdd current mentors list, and first pass on a project list broken out of "PineMango"
Russell Bryant [Mon, 9 Mar 2009 22:00:42 +0000 (22:00 +0000)]
Add current mentors list, and first pass on a project list broken out of "PineMango"

I will work on adding projects that have been sent to be via email tomorrow.

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

10 years agoAdd Doxygen documentation for API changes from 1.6.0 to 1.6.1
Jeff Peeler [Mon, 9 Mar 2009 20:58:17 +0000 (20:58 +0000)]
Add Doxygen documentation for API changes from 1.6.0 to 1.6.1

Copied from my review board description:
This is a continuation of the API changes documentation started for describing
changes between releases. Most of the API changes were pretty simple needing
only to be brought to attention via the new "Asterisk API Changes" list.
However, if you see anything that needs further explanation feel free to
supplement what is there. The current method of documenting is to add (in the
header file): \version <ver number> <description of changes> and then to add
the function to the change list in doxyref.h on the AstAPIChanges page. I also
made sure all the functions that were newly added were tagged with \since
1.6.1. I think this is a good habit to start both for the historical aspect as
well as for the future ability to easily add a "New Asterisk API" page.

Review: http://reviewboard.digium.com/r/190/

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

10 years agoAdd skeleton for GSoC ideas list
Russell Bryant [Mon, 9 Mar 2009 14:14:34 +0000 (14:14 +0000)]
Add skeleton for GSoC ideas list

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

10 years agoMake some minor updates to the doxygen configuration
Russell Bryant [Sat, 7 Mar 2009 15:36:00 +0000 (15:36 +0000)]
Make some minor updates to the doxygen configuration

- add bridges directory to be processed
- add some res/ subdirs
- alphabetize subdirs
- use consistent indentation

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

10 years agoMerged revisions 180567 via svnmerge from
Mark Michelson [Fri, 6 Mar 2009 18:25:44 +0000 (18:25 +0000)]
Merged revisions 180567 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r180567 | mmichelson | 2009-03-06 12:23:09 -0600 (Fri, 06 Mar 2009) | 2 lines

  Make compilation succeed in dev-mode when IMAP storage is enabled.
........

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

10 years agoMerged revisions 180532 via svnmerge from
David Vossel [Fri, 6 Mar 2009 17:26:38 +0000 (17:26 +0000)]
Merged revisions 180532 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r180532 | dvossel | 2009-03-06 11:19:55 -0600 (Fri, 06 Mar 2009) | 9 lines

  Fix handling of backreferences for ENUM lookups

  enum.c did not handle regex backtraces correctly.  The '\1' in the regex is a backreference that requires a pattern match to be inserted.  The way the code used to work is that it would find the backreference and insert the entire input string minus the '+'.  This is incorrect.  The regexec() function takes in a variable called pmatch which is an array of structs containing the start and end indexes for each backreference substring.  The original code actually passed the pmatch array pointer into regexec but never did anything with it.  Now when a backtrace is found, the backtrace number is looked up in the pmatch array and the correct substring is inserted.

  (closes issue #14576)
  Reported by: chris-mac
  Review: http://reviewboard.digium.com/r/187/
........

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

10 years agoMerged revisions 180464 via svnmerge from
Mark Michelson [Thu, 5 Mar 2009 23:26:58 +0000 (23:26 +0000)]
Merged revisions 180464 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r180464 | mmichelson | 2009-03-05 17:26:11 -0600 (Thu, 05 Mar 2009) | 16 lines

  [IMAP] Fix message retrieval issues when identical mailbox names were defined in separate contexts.

  There was a fix put in a while back so that an X-Asterisk-VM-Context message header was
  added to stored IMAP voicemails. This would allow for us to differentiate if the same
  mailbox name was used in multiple contexts. The problem still left was that not all places
  where messages were retrieved actually attempted to use this header for information when
  retrieving messages. This commit fixes that so that MWI and message retrieval from VoiceMailMain
  work as expected.

  (closes issue #13853)
  Reported by: vicks1
  Patches:
        13853_v2.patch uploaded by mmichelson (license 60)
  Tested by: lmadsen
........

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

10 years agoMerged revisions 180380 via svnmerge from
Mark Michelson [Thu, 5 Mar 2009 19:14:14 +0000 (19:14 +0000)]
Merged revisions 180380 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r180380 | mmichelson | 2009-03-05 12:58:48 -0600 (Thu, 05 Mar 2009) | 25 lines

  Fix broken mailbox parsing when searchcontexts option is enabled.

  When using the searchcontexts option in voicemail.conf, the code
  made the assumption that all mailbox names defined were unique across
  all contexts. However, the code did nothing to actually enforce this
  assumption, nor did it do anything to alert a user that he may have
  created an ambiguity in his voicemail.conf file by defining the same
  mailbox name in multiple contexts.

  With this change, we now will issue a nice long warning if searchcontexts
  is on and we encounter the same mailbox name in multiple contexts and ignore
  any duplicates after the first box. Whether searchcontexts is enabled or not,
  if we come across a duplicate mailbox in the same context, then we will issue
  a warning and ignore the duplicated mailbox. I have also added a small note
  to voicemail.conf.sample in the explanation for searchcontexts explaining
  that you cannot define the same mailbox in multiple contexts if you have
  enabled the option.

  (closes issue #14599)
  Reported by: lmadsen
  Patches:
        14599.patch uploaded by mmichelson (license 60) (with slight modification)
  Tested by: lmadsen
........

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

10 years agoMake sure we terminate the first s| command so we can actually produce correct files.
Michiel van Baak [Thu, 5 Mar 2009 19:05:20 +0000 (19:05 +0000)]
Make sure we terminate the first s| command so we can actually produce correct files.

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

10 years agoMerged revisions 180372 via svnmerge from
Kevin P. Fleming [Thu, 5 Mar 2009 18:29:38 +0000 (18:29 +0000)]
Merged revisions 180372 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r180372 | kpfleming | 2009-03-05 12:22:16 -0600 (Thu, 05 Mar 2009) | 9 lines

  Fix problems when RTP packet frame size is changed

  During some code analysis, I found that calling ast_rtp_codec_setpref() on an ast_rtp session does not work as expected; it does not adjust the smoother that may on the RTP session, in fact it summarily drops it, even if it has data in it, even if the current format's framing size has not changed. This is not good.

  This patch changes this behavior, so that if the packetization size for the current format changes, any existing smoother is safely updated to use the new size, and if no smoother was present, one is created. A new API call for smoothers, ast_smoother_reconfigure(), was required to implement these changes.

  Review: http://reviewboard.digium.com/r/184/
........

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

10 years agoMerge phase 1 support for the new bridging architecture.
Joshua Colp [Thu, 5 Mar 2009 18:18:27 +0000 (18:18 +0000)]
Merge phase 1 support for the new bridging architecture.

This commit brings in the bridging core, bridging technologies,
and the ConfBridge application.

For usage information on the ConfBridge application please see
the output of "core show application ConfBridge" from the CLI.

For API documentation please see the doxygen page describing the
architecture and the documentation for each API call.

Review: http://reviewboard.digium.com/r/93/

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

10 years agoAlso highlight the preamble and postamble
Tilghman Lesher [Thu, 5 Mar 2009 06:21:10 +0000 (06:21 +0000)]
Also highlight the preamble and postamble

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

10 years agoAdd syntax coloring files for Vim, including a new one for AEL
Tilghman Lesher [Thu, 5 Mar 2009 01:41:37 +0000 (01:41 +0000)]
Add syntax coloring files for Vim, including a new one for AEL

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

10 years agoResolve object matching issues related to the removal of the sip_user object.
Russell Bryant [Wed, 4 Mar 2009 21:01:05 +0000 (21:01 +0000)]
Resolve object matching issues related to the removal of the sip_user object.

Previously, chan_sip had both sip_peer and sip_user objects in memory.  A
patch went in to remove sip_user to simplify the code, since everything
could be done with just sip_peer.  This patch resolves some regressions
found that were introduced by those changes.

This code comes from svn/asterisk/team/group/sip-object-matching/.

Here is a list of the changes that have been made:

1) When doing a match by name with the find_peer() function, make it much
   easier to specify which objects should be matched by having a parameter
   that specifies exactly which object types should be considered.  Also,
   update find_by_name() to handle this parameter.  Finally, update all
   code to use the new option values.

2) When looking up an object for an outbound request by name, consider
   peers only.  (create_addr())

3) Only match peers on an incoming registration request.

4) When doing authentication (except for SUBSCRIBE), look up users
   by name, instead of all objects by name.

5) When doing authentication (except for SUBSCRIBE), after looking for
   a user by name, look for a peer by IP address, instead of all objects
   by IP address.

6) When handling the SIP qualify CLI command or manager action, look for
   a peer by name, instead of any object by name.

7) When handling the SIP unregister CLI command, look for a peer by name,
   instead of any object by name.

9) In sip_do_debug_peer(), search for a peer by name, instead of any object
   by name.

9) When handling the SIPPEER() dialplan function, search for a peer by name,
   instead of any object by name.

10) In the following session timer related functions, st_get_se(),
    st_get_refresher(), and st_get_mode(), when looking for an object for a
    given sip_pvt using pvt->peername, look for a peer by name, instead of any
    object by name.

11) Fix build_peer() to properly handle the case where separate type=peer and
    type=user entries were specified in sip.conf.

(closes issue #14505)
Reported by: lmadsen

Review: http://reviewboard.digium.com/r/172/

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

10 years agoSpacing changes only
Tilghman Lesher [Wed, 4 Mar 2009 20:48:42 +0000 (20:48 +0000)]
Spacing changes only

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

10 years agoMerged revisions 180194 via svnmerge from
Joshua Colp [Wed, 4 Mar 2009 19:24:59 +0000 (19:24 +0000)]
Merged revisions 180194 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r180194 | file | 2009-03-04 15:22:50 -0400 (Wed, 04 Mar 2009) | 4 lines

  Look for the number in a callerid string starting from the end. This way a value using <> can exist in the name portion.

  (issue #AST-194)
........

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

10 years agoAllow for "magic" pickups to work when we wish to ignore the context
Mark Michelson [Wed, 4 Mar 2009 17:03:32 +0000 (17:03 +0000)]
Allow for "magic" pickups to work when we wish to ignore the context

When the subscription context for a call pickup subscription differs
from the context of the call pickup target, there's not an easy way
to divine what context should be used for the pickup. The way to work
around this is to use PICKUPMARK as the context for the pickup.

This has been documented in the sip.conf.sample file

(ABE-1708)

closes issue #14567
submitted by: alecdavis

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

10 years agoRemove duplicate 'k' and 'K' Dial options.
Joshua Colp [Wed, 4 Mar 2009 14:39:28 +0000 (14:39 +0000)]
Remove duplicate 'k' and 'K' Dial options.

(closes issue #14601)
Reported by: alecdavis
Patches:
      app_dial.optionk.diff.txt uploaded by alecdavis (license 585)

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

10 years agoMy bad! left check_expr2 in the ALL_UTILS list by mistake. Already done to 1.6.x
Steve Murphy [Tue, 3 Mar 2009 23:35:26 +0000 (23:35 +0000)]
My bad! left check_expr2 in the ALL_UTILS list by mistake. Already done to 1.6.x

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

10 years agoapp_read does not break from prompt loop with user terminated empty string
David Vossel [Tue, 3 Mar 2009 23:21:18 +0000 (23:21 +0000)]
app_read does not break from prompt loop with user terminated empty string

In app.c, ast_app_getdata is called to stream the prompts and receive DTMF input.  If ast_app_getdata() receives an empty string caused by the user inputing the end of string character, in this case '#', it should break from the prompt loop and return to app_read, but instead it cycles through all the prompts.  I've added a return value for this special case in ast_readstring() which uses an enum I've delcared in apps.h.  This enum is now used as a return value for ast_app_getdata().

(closes issue #14279)
Reported by: Marquis
Patches:
fix_app_read.patch uploaded by Marquis (license 32)
read-ampersanmd.patch2 uploaded by dvossel (license 671)
Tested by: Marquis, dvossel
Review: http://reviewboard.digium.com/r/177/

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

10 years agoBlocked revisions 180010 via svnmerge
Jason Parker [Tue, 3 Mar 2009 23:02:04 +0000 (23:02 +0000)]
Blocked revisions 180010 via svnmerge

........
  r180010 | qwell | 2009-03-03 17:01:06 -0600 (Tue, 03 Mar 2009) | 1 line

  Make sure we still support zapchan in users.conf, in addition to dahdichan.
........

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

10 years agoMerged revisions 180006 via svnmerge from
Mark Michelson [Tue, 3 Mar 2009 22:49:07 +0000 (22:49 +0000)]
Merged revisions 180006 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r180006 | mmichelson | 2009-03-03 16:48:18 -0600 (Tue, 03 Mar 2009) | 17 lines

  Clarify some documentation of queues.conf.sample

  It had always been possible to explicitly specify a "blank"
  value for a sound file in queues.conf and have no sound played
  back. The problem with this is that it would result in some ugly
  CLI warnings from file.c.

  This commit introduces a check when playing a file in app_queue
  to see if the name of the file is zero-length and return early if
  that is the case. Also, the ability to specify the blank sound
  files in queues.conf is now mentioned more clearly in queues.conf.sample

  (closes issue #14227)
  Reported by: caspy
........

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

10 years agoMerged revisions 179807 via svnmerge from
Steve Murphy [Tue, 3 Mar 2009 22:12:02 +0000 (22:12 +0000)]
Merged revisions 179807 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

I had some work to do to port these changes to trunk; the
check_expr stuff hasn't been updated here for quite some
time, it appears. I added some more tests to the check_expr2
suite. I had to play around with the makefile a bit, etc.

I added STANDALONE2 #ifdefs to ast_expr2.y so as not to
conflict structure with aelparse.

........
  r179807 | murf | 2009-03-03 11:11:34 -0700 (Tue, 03 Mar 2009) | 19 lines

  These changes allow AEL to better check ${} constructs within $[...], that are concatenated with text.

  I modified and added rules in ast_expr2.fl to better handle
  the concatenations.

  I added some default routines to ast_expr2.y so the standalone would
  compile. It also looks like I haven't run this thru bison since 2.1, so
  it's good to get this updated.

  The Makefile has comments added now for check_expr2 and check_expr to
  explain what they are for, and how to run them.

  The testexpr2s stuff has been removed, in favor of check_expr2.

  expr2.testinput has been updated to include the two expressions
  that inspired these changes (from mcnobody on #asterisk this morning)
  The regression has been run and all looks well.
........

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

10 years agoapp_meetme not setting filename and fileformat correctly for realtime
David Vossel [Tue, 3 Mar 2009 22:01:24 +0000 (22:01 +0000)]
app_meetme not setting filename and fileformat correctly for realtime

When app_meetme finds a realtime conference, it doesn't get the filename and fileformat correctly when 'r' is set.  Now app_meetme first checks to see if fileformat and filename are declared in the db, if they're not it checks the .conf file, if its not declared there either it then uses defaults.

(closes issue #14545)
Reported by: dalbaech
Patches:
app_meetme-realtime5.patch uploaded by dvossel (license 671)
Realtime_Conference_Record_workaround.txt uploaded by dalbaech (license 705)
Tested by: dvossel, dalbaech
Review: http://reviewboard.digium.com/r/180/

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

10 years agoAdd documentation for timing modules used in Asterisk
Mark Michelson [Tue, 3 Mar 2009 20:59:16 +0000 (20:59 +0000)]
Add documentation for timing modules used in Asterisk

This document specifies the timing modules available in Asterisk beginning
with Asterisk 1.6.1. The document goes into detail about the differences
between each and gives a general overview of what timing is used for in
Asterisk. There is also a section which can be used to help customize
your setup or to troubleshoot timing issues you may have.

I also added messages to the DAHDI timing test used in res_timing_dahdi.c
that points to this new documentation if people experience problems.

Big thanks to all who contributed comments on this.

(closes issue #14490)
Reported by: mmichelson
Patches:
      timing.txt uploaded by mmichelson (license 60)

Review: http://reviewboard.digium.com/r/164/

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

10 years agofix a leaked channel lock (and future deadlock) when we try to pick up our own channel
Brian Degenhardt [Tue, 3 Mar 2009 20:02:20 +0000 (20:02 +0000)]
fix a leaked channel lock (and future deadlock) when we try to pick up our own channel

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

10 years agoMerged revisions 179840 via svnmerge from
Joshua Colp [Tue, 3 Mar 2009 18:28:46 +0000 (18:28 +0000)]
Merged revisions 179840 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r179840 | file | 2009-03-03 14:27:09 -0400 (Tue, 03 Mar 2009) | 9 lines

  Do not assume that the bridge_cdr is still attached to the channel when the 'h' exten is finished executing.

  It is possible for a masquerade operation to occur when the 'h' exten is operating. This operation moves
  the CDR records around causing the bridge_cdr to no longer exist on the channel where it is expected to.
  We can not safely modify it afterwards because of this, so don't even try.

  (closes issue #14564)
  Reported by: meric
........

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

10 years agoConvert pbx_spool to use string fields instead of statically-sized buffers.
Mark Michelson [Tue, 3 Mar 2009 17:03:47 +0000 (17:03 +0000)]
Convert pbx_spool to use string fields instead of statically-sized buffers.

In tests run after making this conversion, I noticed an approximate 85%
reduction in memory usage for call file processing.

Review: http://reviewboard.digium.com/r/168/

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

10 years agoMerged revisions 179741 via svnmerge from
Russell Bryant [Tue, 3 Mar 2009 16:47:28 +0000 (16:47 +0000)]
Merged revisions 179741 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r179741 | russell | 2009-03-03 10:45:46 -0600 (Tue, 03 Mar 2009) | 6 lines

Ensure chan->fdno always gets reset to -1 after handling a channel fd event.

Since setting fdno to -1 had to be moved, a couple of other code paths that
do process an fd event return early and do not pass through the code path
where it was moved to.  So, set it to -1 in a few other places, too.

........

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

10 years agoPlease prefix default values with DEFAULT
Olle Johansson [Tue, 3 Mar 2009 15:13:42 +0000 (15:13 +0000)]
Please prefix default values with DEFAULT

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

10 years agoMerged revisions 179671 via svnmerge from
Joshua Colp [Tue, 3 Mar 2009 14:40:04 +0000 (14:40 +0000)]
Merged revisions 179671 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r179671 | file | 2009-03-03 10:38:09 -0400 (Tue, 03 Mar 2009) | 3 lines

  Move where fdno is set to the default value to *after* the read callback of the channel driver is called.
  We have to do this as the underlying channel driver may need the fdno value to determine what to read.
........

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

10 years agoMerged revisions 179608 via svnmerge from
Russell Bryant [Tue, 3 Mar 2009 13:54:41 +0000 (13:54 +0000)]
Merged revisions 179608 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r179608 | russell | 2009-03-03 07:53:52 -0600 (Tue, 03 Mar 2009) | 9 lines

Make it easier to detect an improper call to ast_read().

When you call ast_waitfor() on a channel, the index into the channel fds array
that holds the file descriptor that poll() determines has input available is
stored in fdno.  This patch clears out this value after a call to ast_read()
and also reports errors if ast_read() is called without an fdno set.

From a discussion on the asterisk-dev list.

........

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

10 years agoMerged revisions 179536 via svnmerge from
Jeff Peeler [Tue, 3 Mar 2009 00:01:51 +0000 (00:01 +0000)]
Merged revisions 179536 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r179536 | jpeeler | 2009-03-02 17:54:39 -0600 (Mon, 02 Mar 2009) | 15 lines

  Fix bridging regression from commit 176701

  This fixes a bad regression where the bridge would exit after an attended
  transfer was made. The problem was due to nexteventts getting set after the
  masquerade which caused the bridge to return AST_BRIDGE_COMPLETE.

  (closes issue #14315)
  Reported by: tim_ringenbach
........

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

10 years agoMerged revisions 179532 via svnmerge from
Russell Bryant [Mon, 2 Mar 2009 23:36:38 +0000 (23:36 +0000)]
Merged revisions 179532 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r179532 | russell | 2009-03-02 17:34:13 -0600 (Mon, 02 Mar 2009) | 40 lines

Move ast_waitfor() down to avoid the results of the API call becoming stale.

This call to ast_waitfor() was being done way too soon in this section of code.
Specifically, there was code in between the call to waitfor and the code that
uses the result that puts the channel in autoservice.  By putting the channel
in autoservice, the previous results of ast_waitfor() become meaningless,
as the autoservice thread will do it's own ast_waitfor() and ast_read()
on the channel.

So, when we came back out of autoservice and eventually hit the block of code
that calls ast_read() on the channel, there may not actually be any input on
the channel available.  Even though the previous call to ast_waitfor() in
app_meetme said there was input, the autoservice thread has since serviced
the channel for some period of time.

This bug manifested itself while dvossel was doing some testing of MeetMe in
Asterisk trunk.  He was using the timerfd timing module.  When the code hit
ast_read() erroneously, it determined that it must have been called because of
input on the timer fd, as chan->fdno was set to AST_TIMING_FD, since that was
the cause of the last legitimate call to ast_read() done by autoservice.

In this test, an IAX2 channel was calling into the MeetMe conference.  It was
_much_ more likely to be seen with an IAX2 channel because of the way audio
is handled.  Every audio frame that comes in results in a call to
ast_queue_frame(), which then uses ast_timer_enable_continuous() to notify
the channel thread that a frame is waiting to be handled.  So, the chances
of ast_waitfor() indicating that a channel needs servicing due to a timer
event on an IAX2 event is very high.

Finally, it is interesting to note that if a different timing interface was
being used, this bug would probably not be noticed.  When ast_read() is called
and erroneously thinks that there is a timer event to handle, it calls the
ast_timer_ack() function.  The pthread and dahdi timing modules handle the
ack() function being called when there is no event by simply ignoring it.
In the case of the timerfd module, it results in a read() on the timer fd
that will block forever, as there is no data to read.  This caused Asterisk
to lock up very quickly.

Thanks to dvossel and mmichelson for the fun debugging session.  :-)

........

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