asterisk/asterisk.git
11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 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

11 years agoMerged revisions 179468 via svnmerge from
Tilghman Lesher [Mon, 2 Mar 2009 23:10:18 +0000 (23:10 +0000)]
Merged revisions 179468 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r179468 | tilghman | 2009-03-02 17:09:01 -0600 (Mon, 02 Mar 2009) | 10 lines

  When ending a recording with silence detection, remember to reduce the duration.
  The end of the recording is correspondingly trimmed, but the duration was not
  trimmed by the number of seconds trimmed, so the saved duration was necessarily
  longer than the actual soundfile duration.
  (closes issue #14406)
   Reported by: sasargen
   Patches:
         20090226__bug14406.diff.txt uploaded by tilghman (license 14)
   Tested by: sasargen
........

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

11 years agoFix a reference leak in timerfd_set_rate().
Russell Bryant [Mon, 2 Mar 2009 23:06:16 +0000 (23:06 +0000)]
Fix a reference leak in timerfd_set_rate().

(found during a debugging session with dvossel and mmichelson.)

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

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

........
r179461 | russell | 2009-03-02 16:58:18 -0600 (Mon, 02 Mar 2009) | 8 lines

Ensure that only one thread is calling ast_settimeout() on a channel at a time.

For example, with an IAX2 channel, you can have both the channel thread and the
chan_iax2 processing threads calling this function, and doing so twice at the
same time is a bad thing.

(Found in a debugging session with dvossel and mmichelson)

........

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

11 years agoMerged revisions 179395 via svnmerge from
Jason Parker [Mon, 2 Mar 2009 20:16:51 +0000 (20:16 +0000)]
Merged revisions 179395 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r179395 | qwell | 2009-03-02 14:14:57 -0600 (Mon, 02 Mar 2009) | 1 line

  Remove several silly warnings in editline.  One about a broken preprocessor directive, and another about strlcpy/strlcat.

  (closes issue #14264)
  Reported by: dimas
........

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

11 years agoBackport 1.6.0 fix to trunk (failsafe if db is not loaded)
Tilghman Lesher [Mon, 2 Mar 2009 17:18:48 +0000 (17:18 +0000)]
Backport 1.6.0 fix to trunk (failsafe if db is not loaded)

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

11 years agoDo not try to remove a registration scheduled item if the scheduler context has alrea...
Joshua Colp [Mon, 2 Mar 2009 14:28:09 +0000 (14:28 +0000)]
Do not try to remove a registration scheduled item if the scheduler context has already been destroyed.

(closes issue #14580)
Reported by: alecdavis

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

11 years agoFix issue where changing the volume of both directions of audio did not work.
Joshua Colp [Mon, 2 Mar 2009 14:13:45 +0000 (14:13 +0000)]
Fix issue where changing the volume of both directions of audio did not work.

(closes issue #14574)
Reported by: KNK
Patches:
      audiohook_volume_fix.diff uploaded by KNK (license 545)

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

11 years agoSwap reversed timevals.
Mark Michelson [Sun, 1 Mar 2009 23:25:23 +0000 (23:25 +0000)]
Swap reversed timevals.

This was pointed out by ScribbleJ in #asterisk-dev. Thanks very much, ScribbleJ!

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

11 years agoProperly free memory and remove scheduler entries when a transmission failure occurs.
Mark Michelson [Sun, 1 Mar 2009 21:45:08 +0000 (21:45 +0000)]
Properly free memory and remove scheduler entries when a transmission failure occurs.

Previously, only the "data" field of the sip_pkt created during __sip_reliable_xmit
was freed when XMIT_ERROR was returned by __sip_xmit. When retrans_pkt was called,
this inevitably resulted in the reading and writing of freed memory.

XMIT_ERROR is a condition meaning that we don't want to attempt resending the packet
at all. The proper action to take is to remove the scheduler entry we just created,
free the packet's data as well as the packet itself, and unlink it from the list of
packets on the sip_pvt structure.

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

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

11 years agoMark res_ais as experimental, as the binary event format is subject to change.
Russell Bryant [Fri, 27 Feb 2009 21:47:18 +0000 (21:47 +0000)]
Mark res_ais as experimental, as the binary event format is subject to change.

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

11 years agoIf config file is blank, don't load module.
Tilghman Lesher [Fri, 27 Feb 2009 21:32:13 +0000 (21:32 +0000)]
If config file is blank, don't load module.
(Closes issue #14563)

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

11 years agoAdd a note about the ordering of entries in sip.conf in 1.6.1.
Russell Bryant [Fri, 27 Feb 2009 21:23:12 +0000 (21:23 +0000)]
Add a note about the ordering of entries in sip.conf in 1.6.1.

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

11 years agoAdd reload support to chan_skinny.
Michiel van Baak [Fri, 27 Feb 2009 20:34:00 +0000 (20:34 +0000)]
Add reload support to chan_skinny.

Special thanks goes to DEA who had to redo this patch twice
because we first put unload/load support in and later redid the way
we configure devices and lines.

(closes issue #10297)
Reported by: DEA
Patches:
      skinny-reload-trunkv2.diff uploaded by wedhorn (license 30)
      skinny-reload-trunk-v4.txt uploaded by DEA (license 3)
  With mods by me based on feedback from wedhorn and Russell and seanbright
Tested by: DEA, mvanbaak, pj

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

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

11 years agoUpdate documentation for DIALEDTIME and ANSWEREDTIME variables.
Jason Parker [Fri, 27 Feb 2009 19:04:57 +0000 (19:04 +0000)]
Update documentation for DIALEDTIME and ANSWEREDTIME variables.

(closes issue #14566)
Reported by: klaus3000
Patches:
      ANSWEREDTIME-1.4-patch.txt uploaded by klaus3000 (license 65)
      ANSWEREDTIME-trunk-patch.txt uploaded by klaus3000 (license 65)

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

11 years agoFix downloading SIREN7 and SIREN14 sound packages.
Russell Bryant [Fri, 27 Feb 2009 15:51:56 +0000 (15:51 +0000)]
Fix downloading SIREN7 and SIREN14 sound packages.

In passing, also fix downloading SLIN16 extra sound packages.

(closes issue #14565)
Reported by: jtodd

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

11 years agoMerged revisions 178956 via svnmerge from
Steve Murphy [Fri, 27 Feb 2009 03:45:58 +0000 (03:45 +0000)]
Merged revisions 178956 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

In this case, it's just a matter of reducing the default timeouts from 2000
to 1000 msec, as the max def feature digit timeout is no longer halved.

........
  r178956 | murf | 2009-02-26 14:27:32 -0700 (Thu, 26 Feb 2009) | 18 lines

  This change moves the default feature digit timeout to 1000 ms from the previous default of 500.

  As per bug 14515, a dev discussion arrived at a "mediated concensus"
  of a default feature digit timeout of 1.0 sec. Some voted for 1300;
  ctooley thought 1500 for distracted phone users in phone booths;
  kpfleming put his foot down at 1.0 sec.

  Users who found the previous default max delay of 250 msec perfect,
  are welcome to override the new default. Notice that I said that
  250 msec was the default; wait a minute, you might say, the config
  file said it was 500 msec!; well, because of the bug fix for 14515,
  we found that 500 msec was actually enforcing a max of 250. The bug
  fix would restore 500 msec, but we felt even that was a bit tight
  for most users... 2000 msec was pushed earlier by mmichelson, so
  that reduces to 1000 msec after the bug fix. Enjoy!
........

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

11 years agoSound confirmation of call pickup success.
Tilghman Lesher [Thu, 26 Feb 2009 18:41:28 +0000 (18:41 +0000)]
Sound confirmation of call pickup success.
(closes issue #13826)
 Reported by: azielke
 Patches:
       pickupsound2-trunk.patch uploaded by azielke (license 548)
       __20081124_bug_13826_updated.patch uploaded by lmadsen (license 10)
 Tested by: lmadsen

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

11 years agoIAX2 prune realtime, minor tweak to last fix
David Vossel [Thu, 26 Feb 2009 17:46:12 +0000 (17:46 +0000)]
IAX2 prune realtime, minor tweak to last fix

A return statement was missing which caused unexpected cli output.

issue #14479

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

11 years agoThese small fixes prevent compiler warnings with ubuntu 8.10's gcc-4.3.2, which tend...
Steve Murphy [Thu, 26 Feb 2009 17:45:22 +0000 (17:45 +0000)]
These small fixes prevent compiler warnings with ubuntu 8.10's gcc-4.3.2, which tend to break my dev-mode build. Not a problem in 1.6.x.

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

11 years agoBlocked revisions 178838 via svnmerge
David Vossel [Thu, 26 Feb 2009 17:26:04 +0000 (17:26 +0000)]
Blocked revisions 178838 via svnmerge

........
  r178838 | dvossel | 2009-02-26 11:24:02 -0600 (Thu, 26 Feb 2009) | 9 lines

  IAX2 prune realtime fix

  Now prune_users() and prune_peers() are called instead of reload_config() to prune all users/peers that are realtime.  These functions remove all users/peers with the rtfriend and delme flags set. iax2_prune_realtime() also lacked the code to properly delete a single friend.  For example. if iax2 prune realtime <friend> was called, only the peer instance would be removed. The user would still remain.

  (closes issue #14479)
  Reported by: mousepad99
  Review: http://reviewboard.digium.com/r/176/
........

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

11 years agoMerged revisions 178804 via svnmerge from
Steve Murphy [Thu, 26 Feb 2009 17:22:11 +0000 (17:22 +0000)]
Merged revisions 178804 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r178804 | murf | 2009-02-26 10:09:03 -0700 (Thu, 26 Feb 2009) | 28 lines

  This patch prevents the feature detection timeout from being cut in half.

  Because the ast_channel_bridge() call will return 0 and pass
  a frame pointer for both DTMF_BEGIN and DTMF_END, the feature_timer
  field in hte config struct is getting decremented twice, which
  effectively cuts the digittimeout in half. I added conditions
  to the if statement to only let DTMF_END frames to flow thru,
  which solved the problem. Also, when the frame pointer is null,
  let control flow thru-- this usually happens on timeouts. I added
  a comment to the code to explain what's going on and why.

  Many thanks to sodom for reporting this problem. Personnally, it always seemed
  like something was wrong with the featuredigittimeout, but I never
  could quite decide what... and was too busy to investigate.
  This bug forced the issue, and now we know.

  Sodom had other issues in 14515, but I couldn't reproduce them. If
  he still has problems, and wants to get them solved, he is welcome
  to reopen 14515.

  (closes issue #14515)
  Reported by: sodom
  Patches:
        14515.patch uploaded by murf (license 17)
  Tested by: murf, sodom
........

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

11 years agoFix an issue where the timer for file playback would not be stopped if DAHDI was...
Joshua Colp [Thu, 26 Feb 2009 16:42:36 +0000 (16:42 +0000)]
Fix an issue where the timer for file playback would not be stopped if DAHDI was not installed.

(closes issue #14541)
Reported by: grant

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

11 years agoIAX2 prune realtime fix
David Vossel [Thu, 26 Feb 2009 15:50:22 +0000 (15:50 +0000)]
IAX2 prune realtime fix

Iax2 prune realtime had issues.  If "iax2 prune realtime all" was called, it would appear like the command was successful, but in reality nothing happened.  This is because the reload that was supposed to take place checks the config files, sees no changes, and does nothing.  If there had been a change in the the config file, the realtime users would have been marked for deletion and everything would have been fine.  Now prune_users() and prune_peers() are called instead of reload_config() to prune all users/peers that are realtime.  These functions remove all users/peers with the rtfriend and delme flags set. iax2_prune_realtime() also lacked the code to properly delete a single friend.  For example. if iax2 prune realtime <friend> was called, only the peer instance would be removed. The user would still remain.

(closes issue #14479)
Reported by: mousepad99
Review: http://reviewboard.digium.com/r/176/

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

11 years agoEnsure there is a valid tone part before trying to play tones.
Joshua Colp [Thu, 26 Feb 2009 15:40:10 +0000 (15:40 +0000)]
Ensure there is a valid tone part before trying to play tones.

(closes issue #14558)
Reported by: alecdavis

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

11 years agoClarifications on the different models and reference to further docs.
Olle Johansson [Thu, 26 Feb 2009 15:02:53 +0000 (15:02 +0000)]
Clarifications on the different models and reference to further docs.

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

11 years agoanother minor commit to test post-commit script changes (now testing post-revprop...
Kevin P. Fleming [Thu, 26 Feb 2009 13:39:00 +0000 (13:39 +0000)]
another minor commit to test post-commit script changes (now testing post-revprop-change as well, third try)

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

11 years agominor commit to test post-commit script changes
Kevin P. Fleming [Thu, 26 Feb 2009 13:28:31 +0000 (13:28 +0000)]
minor commit to test post-commit script changes

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

11 years agoBlocked revisions 178640 via svnmerge
Steve Murphy [Wed, 25 Feb 2009 21:09:27 +0000 (21:09 +0000)]
Blocked revisions 178640 via svnmerge

........
  r178640 | murf | 2009-02-25 14:00:50 -0700 (Wed, 25 Feb 2009) | 17 lines

  This patch completes the fixes nec. to make 1.4 asterisk dialplan expressions ($[...]) 8-bit transparent

  While I was updating ast_expr2.fl, I missed one rule that would allow 8-bit chars to be caught
  in tokens; and in so doing, it absorbs the ${ sequence and messes up the
  checking of raw exprs by AEL.

  Trunk already has these changes.

  (closes issue #14543)
  Reported by: klaus3000
  Patches:
        patch.14543 uploaded by murf (license 17)
  Tested by: murf
........

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

11 years agoPicky, picky buildbots
Tilghman Lesher [Wed, 25 Feb 2009 19:49:46 +0000 (19:49 +0000)]
Picky, picky buildbots

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

11 years agoUse notification when timezone files change and re-scan then.
Tilghman Lesher [Wed, 25 Feb 2009 19:24:44 +0000 (19:24 +0000)]
Use notification when timezone files change and re-scan then.
(closes issue #14300)
 Reported by: jamessan
 Patches:
       20090127__bug14300.diff.txt uploaded by tilghman (license 14)
       20090224__bug14300.diff uploaded by jamessan (license 246)
 Tested by: jamessan
 Review: http://reviewboard.digium.com/r/136/

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

11 years agoOops, wrong direction of command
Tilghman Lesher [Wed, 25 Feb 2009 19:03:35 +0000 (19:03 +0000)]
Oops, wrong direction of command

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

11 years agoMerged revisions 178508 via svnmerge from
Russell Bryant [Wed, 25 Feb 2009 12:45:30 +0000 (12:45 +0000)]
Merged revisions 178508 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r178508 | russell | 2009-02-25 06:43:36 -0600 (Wed, 25 Feb 2009) | 2 lines

Update the copyright year for the main page of the doxygen documentation.

........

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

11 years agoMerged revisions 178445 via svnmerge from
Tilghman Lesher [Tue, 24 Feb 2009 23:27:23 +0000 (23:27 +0000)]
Merged revisions 178445 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r178445 | tilghman | 2009-02-24 17:25:24 -0600 (Tue, 24 Feb 2009) | 5 lines

  Add section about the #exec command in configuration files.
  (closes issue #14540)
   Reported by: jtodd
   Patch by: jtodd, with additional notes by tilghman (license 14)
........

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

11 years agoApparently, a void cast doesn't override warn_unused_result.
Tilghman Lesher [Tue, 24 Feb 2009 20:52:44 +0000 (20:52 +0000)]
Apparently, a void cast doesn't override warn_unused_result.

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

11 years agoThe 3 possible errors with pipe(2) are all impossible in this situation.
Tilghman Lesher [Tue, 24 Feb 2009 20:40:02 +0000 (20:40 +0000)]
The 3 possible errors with pipe(2) are all impossible in this situation.

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

11 years agoMerged revisions 178373 via svnmerge from
Russell Bryant [Tue, 24 Feb 2009 20:39:57 +0000 (20:39 +0000)]
Merged revisions 178373 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r178373 | russell | 2009-02-24 14:36:19 -0600 (Tue, 24 Feb 2009) | 6 lines

Only set dtmfcount on BEGIN, and ensure it gets reset to 0 properly.

(issue #14460)
Reported by: moliveras
Tested by: russell

........

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

11 years agoUse a SIGPIPE to kill the process, instead of depending upon the astcanary process...
Tilghman Lesher [Tue, 24 Feb 2009 20:06:48 +0000 (20:06 +0000)]
Use a SIGPIPE to kill the process, instead of depending upon the astcanary process being inherited by init.

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

11 years agoCause astcanary to exit if Asterisk exits abnormally and doesn't kill astcanary.
Tilghman Lesher [Tue, 24 Feb 2009 17:51:36 +0000 (17:51 +0000)]
Cause astcanary to exit if Asterisk exits abnormally and doesn't kill astcanary.
Also, add some documentation supporting the use of astcanary.
(closes issue #14538)
 Reported by: KNK
 Patches:
       asterisk-1.6.x-astcanary.diff uploaded by KNK (license 545)

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

11 years agoAllows manager command to see if IAX link is trunked and encrypted. Displays what...
David Vossel [Tue, 24 Feb 2009 17:42:37 +0000 (17:42 +0000)]
Allows manager command to see if IAX link is trunked and encrypted. Displays what kind of encryption is enabled as well.

Manager command "iaxpeers" now shows if a link is trunked and encrypted.  Instead of encryption saying simply "yes" or "no", it now displays what type of encryption is enabled and if keyrotation is on or not.

(closes issue #14427)
Reported by: snuffy
Patches:
iax_show_trunks.diff uploaded by snuffy (license 35)
2009022200_iax2_show_trunkencryption.diff.txt uploaded by mvanbaak (license 7)
Tested by: mvanbaak, dvossel, snuffy
Review: http://reviewboard.digium.com/r/173/

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

11 years agoMerged revisions 178205 via svnmerge from
Joshua Colp [Tue, 24 Feb 2009 15:18:38 +0000 (15:18 +0000)]
Merged revisions 178205 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r178205 | file | 2009-02-24 11:16:07 -0400 (Tue, 24 Feb 2009) | 9 lines

  Skip check for extension when subscribing for MWI.

  Since the remote side is not actually subscribing to a specific extension when
  subscribing for MWI just skip the check to see if the extension exists. They can't use it
  to specify the mailbox either since we require configuration of that in sip.conf

  (closes issue #14531)
  Reported by: festr
........

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

11 years agoMerged revisions 178141 via svnmerge from
Russell Bryant [Mon, 23 Feb 2009 23:11:37 +0000 (23:11 +0000)]
Merged revisions 178141 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r178141 | russell | 2009-02-23 17:09:01 -0600 (Mon, 23 Feb 2009) | 14 lines

Fix infinite DTMF when a BEGIN is received without an END.

This commit is related to rev 175124 of 1.4 where a previous attempt was made
to fix this problem.  The problem with the previous patch was that the inserted
code needed to go _before_ setting the lastrxts to the current timestamp.
Because those were the same, the dtmfcount variable was never decremented, and
so the END was never sent.

In passing, I removed the dtmfsamples variable which was completed unused.  I
also removed a redundant setting of the lastrxts variable.

(closes issue #14460)
Reported by: moliveras

........

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

11 years agoPermit emailsubject and emailbody to be set per mailbox.
Tilghman Lesher [Mon, 23 Feb 2009 21:02:18 +0000 (21:02 +0000)]
Permit emailsubject and emailbody to be set per mailbox.
(closes issue #14372)
 Reported by: fhackenberger
 Patches:
       voicemail_individual_subject_and_body_1.6.1 uploaded by fhackenberger (license 592)
       with additional fixes by Corydon76 (license 14)

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

11 years agoupdate the new manager commands in chan_skinny to match
Michiel van Baak [Mon, 23 Feb 2009 18:23:38 +0000 (18:23 +0000)]
update the new manager commands in chan_skinny to match
chan_sip's headers. requested by oej.

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

11 years agoChanges the way keyrotation is enabled by default
David Vossel [Mon, 23 Feb 2009 17:59:55 +0000 (17:59 +0000)]
Changes the way keyrotation is enabled by default

Key rotation was enabled by default by setting the global encryption method to IAX_ENCRYPT_KEYROTATE.  the problem with this is that if encryption is not enabled, and the encryption method is set to anything except 0, the peer appears to have encryption enabled when issuing a "iax2 show peers".  Rather than have the key rotation bit always set by default, it is now only set when an encryption method is enabled.

(closes issue #14523)
Reported by: mvanbaak

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

11 years agolist the addition of the SKINNY manager actions in the CHANGES file.
Michiel van Baak [Mon, 23 Feb 2009 17:48:32 +0000 (17:48 +0000)]
list the addition of the SKINNY manager actions in the CHANGES file.

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

11 years agoFix a regression in scheduler entry ordering, and add a regression test for it.
Russell Bryant [Mon, 23 Feb 2009 17:29:16 +0000 (17:29 +0000)]
Fix a regression in scheduler entry ordering, and add a regression test for it.

(closes issue #14522)
Reported by: pj
Tested by: russell

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

11 years agoAdd a couple of manager commands to chan_skinny
Michiel van Baak [Sun, 22 Feb 2009 23:04:37 +0000 (23:04 +0000)]
Add a couple of manager commands to chan_skinny

Added:
SKINNYdevices
SKINNYshowdevice
SKINNYlines
SKINNYshowline

(closes issue #14521)
Reported by: mvanbaak

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

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

11 years agoOn update, test against the existence of sipregs.
Tilghman Lesher [Sat, 21 Feb 2009 15:59:49 +0000 (15:59 +0000)]
On update, test against the existence of sipregs.

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