10 years agoapp_mixmonitor: Fix a reference leak in manager_mixmonitor function
Jonathan Rose [Thu, 5 Jul 2012 18:11:58 +0000 (18:11 +0000)]
app_mixmonitor: Fix a reference leak in manager_mixmonitor function

Manager_mixmonitor included an early return on failed executions of mixmonitor
that would result in a leaked channel reference.

(closes issue ASTERISK-19943)
Reported by: Mark Murawski
mixmonitor-trunk-368394.patch uploaded by Mark Murawski (license 5791)

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

10 years agoDo not send a BYE when a provisional response arrives during a re-INVITE
Matthew Jordan [Thu, 5 Jul 2012 17:03:43 +0000 (17:03 +0000)]
Do not send a BYE when a provisional response arrives during a re-INVITE

Commits r369557 and r369579 were done to improve handling of re-INVITEs
when the UA that was supposed to receive the re-INVITE fails to respond.
A limitation of those patches occurred when a UA sent a provisional
response to the re-INVITE.  This triggered a sending of a BYE in
check_pending.  This patch tweaks the handling of the re-INVITE such that
a BYE is not sent in response to those messages.

(issue ASTERISK-19992)
Reported by: Steve Davies
Tested by: Steve Davies
  (reinvite_tweak.diff license #5012 by Steve Davies)

Merged revisions 369626 from

Merged revisions 369627 from

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

10 years agoFix dev mode ooh323 warnings
Alexandr Anikin [Thu, 5 Jul 2012 11:42:23 +0000 (11:42 +0000)]
Fix dev mode ooh323 warnings

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

10 years agoAdded direct media support to ooh323 channel driver
Alexandr Anikin [Wed, 4 Jul 2012 21:42:05 +0000 (21:42 +0000)]
Added direct media support to ooh323 channel driver
options are documented in config sample
sample config rename to proper name - ooh323.conf

To change media address ooh323 send empty TCS if there was
completed TCS exchange or send facility forwardedelements
with new fast start proposal if not.
Then close transmit logical channels and renew TCS exchange.

If new fast start proposal is received then ooh323 stack call back
channel driver routine to change rtp address in the rtp instance.
If empty TCS is received then close transmit logical channels and
renew TCS exchange


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

10 years agofix small mistake in the previous
Alexandr Anikin [Wed, 4 Jul 2012 18:50:47 +0000 (18:50 +0000)]
fix small mistake in the previous

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

10 years agoFix modern gcc warning
Alexandr Anikin [Wed, 4 Jul 2012 18:46:56 +0000 (18:46 +0000)]
Fix modern gcc warning


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

10 years agoMore improvements to re-INVITEs timing out after a provisional response
Terry Wilson [Tue, 3 Jul 2012 17:07:20 +0000 (17:07 +0000)]
More improvements to re-INVITEs timing out after a provisional response

There is no need to call check_pendings() on a final response to an INVITE
when destroying the scheduler entry as it will be done later during normal

(issue ASTERISK-19992)

Merged revisions 369579 from

Merged revisions 369580 from

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

10 years agoBetter handle re-INVITEs with provisional but no final repsonses
Terry Wilson [Tue, 3 Jul 2012 14:49:19 +0000 (14:49 +0000)]
Better handle re-INVITEs with provisional but no final repsonses

A previous attempt at fixing this issue had negative side effects related
to attended transfers which this patch should resolve. Many thanks to
Steve Davies for all of the good suggestions and testing.

(closes issue ASTERISK-19992)
Reported by: Steve Davies
Tested by: Steve Davies, Terry Wilson

Merged revisions 369557 from

Merged revisions 369558 from

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

10 years agoAdd a cleaned up drop-in replacement for res_jabber called res_xmpp. This provides...
Joshua Colp [Mon, 2 Jul 2012 14:06:19 +0000 (14:06 +0000)]
Add a cleaned up drop-in replacement for res_jabber called res_xmpp. This provides the same externally facing functionality but is implemented differently internally.

This is currently not built by default but this will be changed once chan_jingle2 (insert actual name in your head when reading this after it has been merged)
is in the tree.


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

10 years agoEnsure the timer heap is protected by a lock.
Joshua Colp [Mon, 2 Jul 2012 00:35:40 +0000 (00:35 +0000)]
Ensure the timer heap is protected by a lock.

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

10 years agoEnable IPv6 support in pjproject.
Joshua Colp [Sun, 1 Jul 2012 20:03:28 +0000 (20:03 +0000)]
Enable IPv6 support in pjproject.

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

10 years agoDon't try to send connectivity checks on RTCP if RTCP is no longer present and don...
Joshua Colp [Sun, 1 Jul 2012 19:36:49 +0000 (19:36 +0000)]
Don't try to send connectivity checks on RTCP if RTCP is no longer present and don't do multiple ICE connectivity checks at once.

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

10 years agoAdd support for ICE/STUN/TURN in res_rtp_asterisk and chan_sip.
Joshua Colp [Sun, 1 Jul 2012 17:28:57 +0000 (17:28 +0000)]
Add support for ICE/STUN/TURN in res_rtp_asterisk and chan_sip.


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

10 years agoFix apparent copy and paste error where incorrect "glue" is used.
Mark Michelson [Fri, 29 Jun 2012 20:32:40 +0000 (20:32 +0000)]
Fix apparent copy and paste error where incorrect "glue" is used.

Merged revisions 369511 from

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

10 years agoHangup handlers - Dialplan subroutines that run when the channel hangs up.
Richard Mudgett [Fri, 29 Jun 2012 17:02:32 +0000 (17:02 +0000)]
Hangup handlers - Dialplan subroutines that run when the channel hangs up.

Hangup handlers are an alternative to the h extension.  They can be used
in addition to the h extension.  The idea is to attach a Gosub routine to
a channel that will execute when the call hangs up.  Whereas which h
extension gets executed depends on the location of dialplan execution when
the call hangs up, hangup handlers are attached to the call channel.  You
can attach multiple handlers that will execute in the order of most
recently added first.

(closes issue ASTERISK-19549)
Reported by: Mark Murawski
Tested by: rmudgett


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

10 years agoWith some configurations a transport is not actually specified so assume UDP in these...
Joshua Colp [Fri, 29 Jun 2012 16:56:29 +0000 (16:56 +0000)]
With some configurations a transport is not actually specified so assume UDP in these cases.

Merged revisions 369490 from

Merged revisions 369491 from

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

10 years agoRemove obsolete struct ast_channel note.
Richard Mudgett [Fri, 29 Jun 2012 16:42:32 +0000 (16:42 +0000)]
Remove obsolete struct ast_channel note.

The opaquing the ast_channel struct no longer requires .cleancount to be
changed when the struct is changed.

* Bump .cleancount value one last time because of struct ast_channel for
old times sake.

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

10 years agoMake the address family filter specific to the transport.
Joshua Colp [Fri, 29 Jun 2012 15:33:39 +0000 (15:33 +0000)]
Make the address family filter specific to the transport.

(closes issue ASTERISK-16618)
Reported by: Leif Madsen


Merged revisions 369471 from

Merged revisions 369472 from

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

10 years agoAdd the ability to set flags via the config options api
Terry Wilson [Thu, 28 Jun 2012 01:12:06 +0000 (01:12 +0000)]
Add the ability to set flags via the config options api

Allows the setting of flags via the config options api.
For example, code like this:

#define OPT1 1 << 0
#define OPT2 1 << 1
#define OPT3 1 << 2

struct thing {
   unsigned int flags;

and a config like this:



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

10 years agoAST-2012-010: Clean up after a reinvite that never gets a final response
Terry Wilson [Wed, 27 Jun 2012 21:21:27 +0000 (21:21 +0000)]
AST-2012-010: Clean up after a reinvite that never gets a final response

The basic problem is that if a re-INVITE is sent by Asterisk and it receives a
provisional response, but no final response, then the dialog is never torn
down. In addition to leaking memory, this also leaks file descriptors and will
eventually lead to Asterisk no longer being able to process calls.

This patch just keeps track of whether there is an outstanding re-INVITE, and if
there is goes ahead and cleans up everything as though there was no outstanding


(closes issue ASTERISK-19992)
Reported by: Steve Davies
Tested by: Steve Davies, Terry Wilson

Merged revisions 369436 from

Merged revisions 369437 from

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

10 years agoUnique Call ID logging Phases III and IV
Jonathan Rose [Tue, 26 Jun 2012 21:45:22 +0000 (21:45 +0000)]
Unique Call ID logging Phases III and IV

Adds call ID logging changes to specific channel drivers that weren't handled
handled in phase II of Call ID Logging. Also covers logging for threads for
threads created by systems that may be involved with many different calls.
Extra special thanks to Richard for rigorous review of chan_dahdi and its
various signalling modules.


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

10 years agoFix crash in unloading of res_adsi module
Matthew Jordan [Tue, 26 Jun 2012 13:23:12 +0000 (13:23 +0000)]
Fix crash in unloading of res_adsi module

When res_adsi is unloaded, it removes the ADSI functions that it previously installed
by passing a NULL adsi_funcs pointer to ast_adsi_install_funcs.  This function was not
checking whether or not the adsi_funcs pointer passed in was NULL before dereferencing
it to check whether or not the version of the functions matches what the core was
expecting it.

This patch makes it so that the version is only checked if a potentially valid adsi_funcs
pointer was passed in.  Passing in NULL removes the installed functions, bypassing the
version check.

Merged revisions 369390 from

Merged revisions 369391 from

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

10 years agoUpdate "manager show event" to support tab completion
Matthew Jordan [Mon, 25 Jun 2012 20:43:26 +0000 (20:43 +0000)]
Update "manager show event" to support tab completion

Thank you rmudgett for pointing out that I was missing this in the initial
check-in for AMI event documentation (r369346)

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

10 years agoFix incorrect duration reporting in CDRs created in batch mode
Matthew Jordan [Mon, 25 Jun 2012 19:39:03 +0000 (19:39 +0000)]
Fix incorrect duration reporting in CDRs created in batch mode

Certain places in core/cdr.c would, if the duration value were 0, calculate the
duration as being the delta between the current time and the time at which the
CDR record was started.  While this does not typically cause a problem in
non-batch mode, this can cause an issue in batch mode where CDR records are
gathered and written long after those calls have ended. In particular, this
affects calls that were never answered, as those are expected to have a duration
of 0.  Often, this would result in CDR logs with a significant number of calls
with lengthy durations, but dispositions of "BUSY".

Note that this does not affect cdr_csv, as that backend does not use
ast_cdr_getvar and instead directly reports the duration value.  The affected
core backends include cdr_apative_odbc and cdr_custom; other extended or
deprecated CDR backends may potentially still directly manipulate the duration

(issue ASTERISK-19860)
Reported by: Thomas Arimont

(issue AST-883)
Reported by: Thomas Arimont
Tested by: Matt Jordan


Merged revisions 369351 from

Merged revisions 369369 from

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

10 years agoRe-fix how local tag is generated when sending a 481 to an INVITE.
Mark Michelson [Mon, 25 Jun 2012 19:26:31 +0000 (19:26 +0000)]
Re-fix how local tag is generated when sending a 481 to an INVITE.

Match our local tag to whatever to-tag was sent in the initial INVITE.
Because the size of the to-tag may not fit in the buffer in the sip_pvt,
it has been changed to a string field.

(closes issue ASTERISK-19892)
reported by Walter Doekes


Merged revisions 369352 from

Merged revisions 369353 from

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

10 years agoAdd AMI event documentation
Matthew Jordan [Mon, 25 Jun 2012 17:59:34 +0000 (17:59 +0000)]
Add AMI event documentation

This patch adds the core changes necessary to support AMI event documentation
in the source files of Asterisk, and adds documentation to those AMI events
defined in the core application modules.  Event documentation is built from
the source by two new python scripts, located in build_tools: and

The script mirrors the actions of the existing AWK
get_documentation scripts, except that it will scan the entirety of a source
file for Asterisk documentation.  Upon encountering it, if the documentation
happens to be an AMI event, it will attempt to extract information about the
event directly from the manager event macro calls that raise the event.  The script combines manager event instances that
are the same event but documented in multiple source files.  It generates
the final core-[lang].xml file.

As this process can take longer to complete than a typical 'make all', it
is only performed if a new make target, 'full', is chosen.


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

10 years agoFix Bridge application occasionally returning to the wrong location.
Richard Mudgett [Mon, 25 Jun 2012 16:07:02 +0000 (16:07 +0000)]
Fix Bridge application occasionally returning to the wrong location.

* Fix do_bridge_masquerade() getting the resume location from the zombie
channel.  The code must not touch a clone channel after it has masqueraded
it.  The clone channel has become a zombie and is starting to hangup.

(closes issue ASTERISK-19985)
Reported by: jamicque
      jira_asterisk_19985_v1.8.patch (license #5621) patch uploaded by rmudgett
Tested by: jamicque

Merged revisions 369327 from

Merged revisions 369328 from

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

10 years agoMultiple revisions 369323-369324
Mark Michelson [Mon, 25 Jun 2012 15:55:25 +0000 (15:55 +0000)]
Multiple revisions 369323-369324

  r369323 | mmichelson | 2012-06-25 10:35:43 -0500 (Mon, 25 Jun 2012) | 9 lines

  Eliminate embedding of module.

  The way this is done is to stop using the optional API.
  Instead,, when loaded fills in a table of
  function pointers.

  r369324 | mmichelson | 2012-06-25 10:50:17 -0500 (Mon, 25 Jun 2012) | 2 lines

  Forgot to svn add this file in my last commit.

Merged revisions 369323-369324 from

Merged revisions 369325 from

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

10 years agoBe more consistent with the return code for requests received from invalid domain.
Mark Michelson [Mon, 25 Jun 2012 14:30:19 +0000 (14:30 +0000)]
Be more consistent with the return code for requests received from invalid domain.

When Asterisk receives an INVITE from an external domain when allowexternaldomains=no
send a 403 instead of a 404. This is consistent with Asterisk's behavior when receiving
a REGISTER in this situation.

(Closes issue ASTERISK-19601)
Reported by Matthew Jordan
ASTERISK-19601-no401.patch uploaded by Mark Michelson (License #5049)

Merged revisions 369302 from

Merged revisions 369303 from

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

10 years agoFix F and F(x) action logic in Bridge application.
Richard Mudgett [Sat, 23 Jun 2012 00:33:41 +0000 (00:33 +0000)]
Fix F and F(x) action logic in Bridge application.

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

10 years agoFix Bridge application and AMI Bridge action error handling.
Richard Mudgett [Sat, 23 Jun 2012 00:29:18 +0000 (00:29 +0000)]
Fix Bridge application and AMI Bridge action error handling.

* Fix AMI Bridge action disconnecting the AMI link on error.

* Fix AMI Bridge action and Bridge application not checking if their
masquerades were successful.

* Fix Bridge application running the h-exten when it should not.

* Made do_bridge_masquerade() return if the masquerade was successful so
the Bridge application and AMI Bridge action could deal with it correctly.

* Made bridge_call_thread_launch() hangup the passed in channels if the
bridge_call_thread fails to start.  Those channels would have been

* Made builtin_atxfer() check the success of the transfer masquerade

Merged revisions 369282 from

Merged revisions 369283 from

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

10 years agoExplicitly check caller hangup in app Queue rather than a polluted res2 value.
Richard Mudgett [Fri, 22 Jun 2012 22:12:06 +0000 (22:12 +0000)]
Explicitly check caller hangup in app Queue rather than a polluted res2 value.

Merged revisions 369262 from

Merged revisions 369263 from

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

10 years agoFix F and F(x) action logic in Queue application.
Richard Mudgett [Fri, 22 Jun 2012 21:51:05 +0000 (21:51 +0000)]
Fix F and F(x) action logic in Queue application.

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

10 years agoCheck if PBX was started and fix F and F(x) action logic in Dial application.
Richard Mudgett [Fri, 22 Jun 2012 21:43:44 +0000 (21:43 +0000)]
Check if PBX was started and fix F and F(x) action logic in Dial application.

Merged revisions 369258 from

Merged revisions 369259 from

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

10 years agoCheck if PBX was started for generic CCSS recall.
Richard Mudgett [Fri, 22 Jun 2012 21:06:36 +0000 (21:06 +0000)]
Check if PBX was started for generic CCSS recall.

Merged revisions 369238 from

Merged revisions 369239 from

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

10 years agoChange incorrect chan_sip zombie hangup debug message. They are all zombies now.
Richard Mudgett [Fri, 22 Jun 2012 20:52:54 +0000 (20:52 +0000)]
Change incorrect chan_sip zombie hangup debug message.  They are all zombies now.

Merged revisions 369235 from

Merged revisions 369236 from

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

10 years agoDon't crash on a guest directmedia call
Terry Wilson [Fri, 22 Jun 2012 20:05:22 +0000 (20:05 +0000)]
Don't crash on a guest directmedia call

A sip_pvt may not have relatedpeer set if a call doesn't match up
with a peer. If there is no relatedpeer, there is no direct media
ACL to apply, so just return that it is allowed.

(closes issue ASTERISK-20040)
Reported by: Terry Wilson

Merged revisions 369214 from

Merged revisions 369215 from

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

10 years agoFix wrong variable name in the R2 disconnect callback
Kinsey Moore [Fri, 22 Jun 2012 19:54:41 +0000 (19:54 +0000)]
Fix wrong variable name in the R2 disconnect callback

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

10 years agoDon't parse media stream state for SIP video streams
Kinsey Moore [Fri, 22 Jun 2012 17:25:06 +0000 (17:25 +0000)]
Don't parse media stream state for SIP video streams

The sendonly/recvonly/sendrecv/inactive media stream attributes were
parsed for video, but nothing was ever done with them.  With this code
removed, an UNSUPPORTED message is produced when these attributes are
used in conjunction with a video stream which is the better behavior
since they were never really supported in the first place.

Merged revisions 369195 from

Merged revisions 369206 from

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

10 years agoAdd HANGUPCAUSE hash implementation for DAHDI MFC/R2 subtech
Kinsey Moore [Fri, 22 Jun 2012 15:57:02 +0000 (15:57 +0000)]
Add HANGUPCAUSE hash implementation for DAHDI MFC/R2 subtech

This adds a minimal implementation of the "Who Hung Up?" Asterisk 11
work to chan_dahdi.c for the MFC/R2 DAHDI subtech.  Given the way that
OpenR2 interfaces with chan_dahdi, it is much harder to expose the
type of protocol information that is available in PRI, SS7, or other
channel technologies.

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

10 years agoAdd HANGUPCAUSE hash support for analog and PRI DAHDI subtechs
Kinsey Moore [Fri, 22 Jun 2012 15:10:38 +0000 (15:10 +0000)]
Add HANGUPCAUSE hash support for analog and PRI DAHDI subtechs

This is part of the DAHDI support for the Asterisk 11 "Who Hung Up?"
project and covers the implementation for the technologies implemented
in sig_analog.c and sig_pri.c. Tested on a local machine to verify
protocol and cause information is available.

(issue SWP-4222)

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

10 years agoAdd "Who Hung Up?" implementation for DAHDI SS7 subtechnology
Kinsey Moore [Fri, 22 Jun 2012 14:57:07 +0000 (14:57 +0000)]
Add "Who Hung Up?" implementation for DAHDI SS7 subtechnology

Testing was done on a local machine to verify that protocol and
cause information was being sent properly.

(issue SWP-4222)

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

10 years agoDon't waste time initializing the whole call_identifer_str[].
Richard Mudgett [Wed, 20 Jun 2012 21:33:11 +0000 (21:33 +0000)]
Don't waste time initializing the whole call_identifer_str[].

The array is either setup with a callid string or only the first element
needs to be initialized.

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

10 years agoFix chan_misdn compile error.
Richard Mudgett [Wed, 20 Jun 2012 21:32:40 +0000 (21:32 +0000)]
Fix chan_misdn compile error.

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

10 years agofix locking issue on empty callList
Alexandr Anikin [Wed, 20 Jun 2012 17:48:20 +0000 (17:48 +0000)]
fix locking issue on empty callList
(issue ASTERISK-19298)
Reported by:
        Dmitry Melekhov

Merged revisions 369146 from

Merged revisions 369147 from

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

10 years agoRemove declaration of eivr_connect_socket because it no longer exists.
Sean Bright [Wed, 20 Jun 2012 11:47:12 +0000 (11:47 +0000)]
Remove declaration of eivr_connect_socket because it no longer exists.

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

10 years agouse right definition for channel name
Alexandr Anikin [Wed, 20 Jun 2012 11:20:05 +0000 (11:20 +0000)]
use right definition for channel name

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

10 years agoAdd IPv6 Support To Manager
Michael L. Young [Wed, 20 Jun 2012 03:18:50 +0000 (03:18 +0000)]
Add IPv6 Support To Manager

This patch adds IPv6 support to AMI.

(Closes issue ASTERISK-19965)
Reported by: Michael L. Young
Tested by: Michael L. Young
    ami_ipv6_v3.diff uploaded by Michael L. Young (license 5026)


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

10 years agoFix NULL pointer segfault in ast_sockaddr_parse()
Michael L. Young [Wed, 20 Jun 2012 02:07:00 +0000 (02:07 +0000)]
Fix NULL pointer segfault in ast_sockaddr_parse()

While working with ast_parse_arg() to perform a validity check, a segfault
occurred.  The segfault occurred due to passing a NULL pointer to
ast_sockaddr_parse() from ast_parse_arg().  According to the documentation in
config.h, "result pointer to the result.  NULL is valid here, and can be used to
perform only the validity checks."

This patch fixes the segfault by checking for a NULL pointer.  This patch also
adds documentation to netsock2.h about why it is necessary to check for a NULL

(Closes issue ASTERISK-20006)
Reported by: Michael L. Young
Tested by: Michael L. Young
asterisk-20006-netsock-null-ptr.diff uploaded by Michael L. Young (license 5026)


Merged revisions 369108 from

Merged revisions 369109 from

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

10 years agocheck rtptimeouts in ooh323 channels as per config file
Alexandr Anikin [Tue, 19 Jun 2012 23:36:43 +0000 (23:36 +0000)]
check rtptimeouts in ooh323 channels as per config file
(rtp voice, video, udptl except rtcp)

(closes issue ASTERISK-19179)
Reported by: TSAREGORODTSEV Yury

Merged revisions 369091 from

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

10 years agoEnsure that pvt cause information does not break native bridging
Kinsey Moore [Tue, 19 Jun 2012 21:13:41 +0000 (21:13 +0000)]
Ensure that pvt cause information does not break native bridging

Channel drivers that allow native bridging need to handle
AST_CONTROL_PVT_CAUSE_CODE frames and previously did not handle them
properly, usually breaking out of the native bridge. This change
corrects that behavior and exposes the available cause code information
to the dialplan while native bridges are in place. This required
exposing the HANGUPCAUSE hash setter outside of channel.c, so
additional documentation has been added.

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

10 years agoFix request routing issue when outboundproxy is used.
Mark Michelson [Tue, 19 Jun 2012 15:44:42 +0000 (15:44 +0000)]
Fix request routing issue when outboundproxy is used.

Asterisk was incorrectly setting the destination of CANCELs
and ACKs for error responses to the URI of the initial INVITE.
This resulted in further requests, such as INVITEs with authentication
credentials, to be routed incorrectly. Instead, when these CANCEL
or ACKs are to be sent, we should simply keep the destination the
same as what it previously was. There is no need to alter it any.

(closes issue ASTERISK-20008)
Reported by Marcus Hunger
ASTERISK-20008.patch uploaded by Mark Michelson (license #5049)

Merged revisions 369066 from

Merged revisions 369067 from

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

10 years agoFix AST_CONTROL_PVT_CAUSE_CODE handling
Kinsey Moore [Mon, 18 Jun 2012 22:56:01 +0000 (22:56 +0000)]

When the IAX2 Who Hung Up? changes were added, they uncovered a bug in
the way AST_CONTROL_PVT_CAUSE_CODE was handled in
feature_request_and_dial().  This particular frame subtype was being
treated like more terminal control frames causing the function to be
exited prematurely.

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

10 years agoFix monitoring calls put in a parking lot.
Richard Mudgett [Mon, 18 Jun 2012 18:25:22 +0000 (18:25 +0000)]
Fix monitoring calls put in a parking lot.

* Fix a regression that was introduced by -r366167 which effectively
disabled monitoring parked calls.

(closes issue ASTERISK-20012)
Reported by: sdolloff
Tested by: rmudgett

Merged revisions 369043 from

Merged revisions 369044 from

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

10 years agoVarious small chan_skinny fixes and cleanup
Damien Wedhorn [Fri, 15 Jun 2012 21:18:56 +0000 (21:18 +0000)]
Various small chan_skinny fixes and cleanup

Added test to skinny_register to only allow device to register against
a device that is not already registered.

Addback l->device test for skinny_show_lines. Fixes segfault if a line
is configured but not configured to a device. Reverses part of r368680.

Removed redundant l->device tests in subsubstate and dumpsub. l->device
will always be valid if these routines are called. Reverses 368948 -
discussed with mjordan on irc.

Some indentation cleanup.

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

10 years agoAllow chan_sip to decline unwanted media streams
Kinsey Moore [Fri, 15 Jun 2012 17:13:20 +0000 (17:13 +0000)]
Allow chan_sip to decline unwanted media streams

This change replaces the static array of four representable media
streams with an AST_LIST so that chan_sip can keep track of offered
media streams.  This allows chan_sip to deal with offers containing
multiple same-type streams and many other situations without rejecting
the SDP offer in its entirety, yet still generating a valid response.
This also covers cases where Asterisk can not comprehend the offer if
it is in the correct format.

Previously, chan_sip would reject SDP offers or entirely ignore
individual stream offers in an effort to be more compatible which
would often result in invalid SDP responses.


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

10 years agoFix voicemail API tests by using the correct argument order for create/destroy.
Jason Parker [Fri, 15 Jun 2012 16:30:58 +0000 (16:30 +0000)]
Fix voicemail API tests by using the correct argument order for create/destroy.

Merged revisions 369024 from

Merged revisions 369026 from

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

10 years agoMultiple revisions 369001-369002
Kevin P. Fleming [Fri, 15 Jun 2012 16:20:16 +0000 (16:20 +0000)]
Multiple revisions 369001-369002

  r369001 | kpfleming | 2012-06-15 10:56:08 -0500 (Fri, 15 Jun 2012) | 11 lines

  Add support-level indications to many more source files.

  Since we now have tools that scan through the source tree looking for files
  with specific support levels, we need to ensure that every file that is
  a component of a 'core' or 'extended' module (or the main Asterisk binary)
  is explicitly marked with its support level. This patch adds support-level
  indications to many more source files in tree, but avoids adding them to
  third-party libraries that are included in the tree and to source files
  that don't end up involved in Asterisk itself.
  r369002 | kpfleming | 2012-06-15 10:57:14 -0500 (Fri, 15 Jun 2012) | 3 lines

  Add a script to enable finding source files without support-levels defined.

Merged revisions 369001-369002 from

Merged revisions 369005 from

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

10 years agoAdd HANGUPCAUSE hash support to IAX2
Kinsey Moore [Fri, 15 Jun 2012 16:17:12 +0000 (16:17 +0000)]
Add HANGUPCAUSE hash support to IAX2

Continuing with the Who Hung Up? project for Asterisk 11, this adds
support to IAX2 for the HANGUPCAUSE hash.

Additionally, this breaks out some functionality in frame.c for getting
information about frame types and subclasses.

(issue SWP-4222)

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

10 years agoRemove some symbol exports that got missed in the removal of global symbols.
Jason Parker [Fri, 15 Jun 2012 15:33:41 +0000 (15:33 +0000)]
Remove some symbol exports that got missed in the removal of global symbols.

(issue AST-807)
(issue AST-901)
(issue AST-908)

Merged revisions 368998 from

Merged revisions 368999 from

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

10 years agoRemove remaining properties mmichelson left laying around from phones branch merge.
Richard Mudgett [Fri, 15 Jun 2012 00:55:43 +0000 (00:55 +0000)]
Remove remaining properties mmichelson left laying around from phones branch merge.

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

10 years agoAllow non-normal execution routines to be able to run on hungup channels.
Richard Mudgett [Thu, 14 Jun 2012 23:22:53 +0000 (23:22 +0000)]
Allow non-normal execution routines to be able to run on hungup channels.

* Make non-normal dialplan execution routines be able to run on a hung up
channel.  This is preparation work for hangup handler routines.

* Fixed ability to support relative non-normal dialplan execution
routines.  (i.e., The context and exten are optional for the specified
dialplan location.) Predial routines are the only non-normal routines that
it makes sense to optionally omit the context and exten.  Setting a hangup
handler also needs this ability.

* Fix Return application being able to restore a dialplan location
exactly.  Channels without a PBX may not have context or exten set.

* Fixes non-normal execution routines like connected line interception and
predial leaving the dialplan execution stack unbalanced.  Errors like
missing Return statements, popping too many stack frames using StackPop,
or an application returning non-zero could leave the dialplan stack

* Fixed the AGI gosub application so it cleans up the dialplan execution
stack and handles the autoloop priority increments correctly.

* Eliminated the need for the gosub_virtual_context return location.


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

10 years agoMake the Hangup application set a softhangup flag.
Richard Mudgett [Thu, 14 Jun 2012 22:57:21 +0000 (22:57 +0000)]
Make the Hangup application set a softhangup flag.

The Hangup application used to just return -1 to cause normal dialplan
execution to hangup a channel.  For the non-normal execution routines like
predial and connected-line interception routines, the hangup request would
exit the routine early but otherwise be ignored.

* Made the Hangup application not allow setting a cause code of zero.  A
zero cause code is not defined.

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

10 years agoMove vm defines to group them better.
Richard Mudgett [Thu, 14 Jun 2012 20:49:28 +0000 (20:49 +0000)]
Move vm defines to group them better.

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

10 years agoMultiple revisions 368963,368965
Jason Parker [Thu, 14 Jun 2012 19:40:11 +0000 (19:40 +0000)]
Multiple revisions 368963,368965

  r368963 | qwell | 2012-06-14 13:47:03 -0500 (Thu, 14 Jun 2012) | 14 lines

  Remove global symbol requirement from app_voicemail.

  This uses the existing "function installation" stuff that already existed for
  other functions, like getting message counts.

  (closes issue AST-807)
  (issue AST-901)
  (issue AST-908)


  Merged revisions 368962 from
  r368965 | qwell | 2012-06-14 14:04:57 -0500 (Thu, 14 Jun 2012) | 11 lines

  These functions that were moved need to be static.

  Also wrap test functions in a #ifdef.

  (issue AST-807)
  (issue AST-901)
  (issue AST-908)

  Merged revisions 368964 from

Merged revisions 368963,368965 from

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

10 years agoAST-2012-009: Fix crash in chan_skinny due to Key Pad Button Message handling
Matthew Jordan [Thu, 14 Jun 2012 17:34:10 +0000 (17:34 +0000)]
AST-2012-009: Fix crash in chan_skinny due to Key Pad Button Message handling

AST-2012-008 (r367844) fixed a denial of service attack exploitable in the
Skinny channel driver that occurred when certain messages are sent after a
previously registered station sends an Off Hook message.  Unresolved in that
patch is an issue in the Asterisk 10 releases, wherein, if a Station Key
Pad Button Message is processed after an Off Hook message, the channel driver
will inappropriately dereference a NULL pointer.

This patch fixes those places where the message handling or the channel
callback functions would attempt to dereference the line's pointer to the

(issue ASTERISK-19905)
Reported by: Christoph Hebeisen
Tested by: mjordan, Christoph Hebeisen
  AST-2012-009-10.diff uploaded by mjordan (license 6283)

Merged revisions 368947 from

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

10 years agoRevert Makefile change to remove embedding
Mark Michelson [Thu, 14 Jun 2012 15:28:02 +0000 (15:28 +0000)]
Revert Makefile change to remove embedding

The change has resulted in a linking error for certain versions
of GCC. This is much worse than the original issue, so for now,
temporarily revert the change. A more thorough change will be
sought out.

Merged revisions 368927 from

Merged revisions 368928 from

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

10 years agoAdd a post_apply callback to the Config Options API
Terry Wilson [Thu, 14 Jun 2012 13:41:47 +0000 (13:41 +0000)]
Add a post_apply callback to the Config Options API

This adds a callback that only fires when changes have been successfully
applied via the Config Options API.


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

10 years agoAdd filename alias support to the Config Options API
Terry Wilson [Thu, 14 Jun 2012 13:35:07 +0000 (13:35 +0000)]
Add filename alias support to the Config Options API

This adds the ability to handle a single filename alias for a config
file. This is useful if a config filename has changed, but the old
filename should be supported for backwards compatibility.


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

10 years agoFix a deadlock that occurs when func_volume is used on a local channel.
Mark Michelson [Wed, 13 Jun 2012 21:17:13 +0000 (21:17 +0000)]
Fix a deadlock that occurs when func_volume is used on a local channel.

This was discovered by trying to perform a call forward to an extension
that makes use of func_volume. When the local channel is optimized away,
the datastore on the local;2 channel would have its audiohook destroyed
rather than detaching the audiohook from the channel and then destroying

With this patch, func_volume's datastore destructor takes the proper
route of detaching the audiohook and then destroying it.

(closes issue ASTERISK-19611)
reported by Volker Sauer
ASTERISK-19611.patch uploaded by Mark Michelson (license #5049)

Merged revisions 368898 from

Merged revisions 368899 from

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

10 years agoMark res_smdi/res_adsi as 'core' supported modules
Matthew Jordan [Wed, 13 Jun 2012 20:28:07 +0000 (20:28 +0000)]
Mark res_smdi/res_adsi as 'core' supported modules

Recently, various issues surrounding weak symbols have caused problems with
modules that rely on that feature to be enabled in menuselect.  This includes
app_voicemail and chan_dahdi, as they both rely upon res_smdi and res_adsi,
which, in certain circumstances, may not be enabled by default in menuselect.

Because res_smdi/res_adsi are dependencies for chan_dahdi/app_voicemail, this
patch marks both as 'core' supported modules.  This will allow both
app_voicemail and chan_dahdi to be enabled as well, regardless of whether or
not that system supports weak symbols.

(issue AST-900)
Reported by: Thomas Arimont

(issue AST-885)
Reported by: Denis Alberto Martinez

Merged revisions 368894 from

Merged revisions 368895 from

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

10 years agoRemove forced linking of res_adsi.o
Mark Michelson [Wed, 13 Jun 2012 19:51:08 +0000 (19:51 +0000)]
Remove forced linking of res_adsi.o

In GCC 4.5+ the result is that Asterisk has a phantom
module loaded at startup, claiming to be res_adsi.

(closes issue ASTERISK-19920)
reported by Leif Madsen

Merged revisions 368873 from

Merged revisions 368885 from

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

10 years agoReplace MODULES_DIR with ASTMODDIR in Makefile's INSTALLDIRS
Matthew Jordan [Wed, 13 Jun 2012 14:55:30 +0000 (14:55 +0000)]

Post Asterisk 10, the MODULES_DIR variable no longer exists, and was replaced

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

10 years agoDo not install empty directories; add ASTLIBDIR
Matthew Jordan [Wed, 13 Jun 2012 14:31:24 +0000 (14:31 +0000)]
Do not install empty directories; add ASTLIBDIR

r368830 modified the installation script to only create a directory if that
directory does not exist.  If some directory variable was empty, it would attempt
to create the empty location.  It also failed to create the ASTLIBDIR directory.
This patch fixes it such that the correct directories are made and only created if
a value specifying them actually exists.

Merged revisions 368852 from

Merged revisions 368853 from

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

10 years agoDo not perform install on existing directories
Matthew Jordan [Tue, 12 Jun 2012 18:41:50 +0000 (18:41 +0000)]
Do not perform install on existing directories

If a directory already exists, performing a 'make install' will remove the
permissions associated with the current directory and replace them with the
permissions of the user executing the install.

This patch changes this behavior to only perform an install on the directory
if the directory does not exist.  Thus, if a user later changes the permissions
on that directory, those permissions will be preserved in subsequent installs.



(closes issue ASTERISK-19492)
Reported by: Karl Fife
Tested by: Paul Belanger, Tilghman Lesher
  ASTERISK-19492 by pabelanger
  (uploaded by mjordan)

Merged revisions 368830 from

Merged revisions 368831 from

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

10 years agoSet the Caller ID "tag" on peers even if remote party information is present.
Mark Michelson [Tue, 12 Jun 2012 15:46:48 +0000 (15:46 +0000)]
Set the Caller ID "tag" on peers even if remote party information is present.

On incoming calls, we were setting the cid_tag on the dialog only if there was
no remote party information (Remote-Party-ID or P-Asserted-Identity) present.
The Caller ID tag is an invented parameter, though, and should be set no matter
the circumstance.

(closes issue ASTERISK-19859)
Reported by Thomas Arimont
(closes issue AST-884)
Reported by Trey Blancher

Merged revisions 368807 from

Merged revisions 368808 from

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

10 years agoUpdate merge property information
Matthew Jordan [Tue, 12 Jun 2012 14:09:41 +0000 (14:09 +0000)]
Update merge property information

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

10 years agoFix deadlock in SIP transfers that involve a REFER request
Matthew Jordan [Tue, 12 Jun 2012 14:07:13 +0000 (14:07 +0000)]
Fix deadlock in SIP transfers that involve a REFER request

In r367163, "send to voicemail" functionality was added to the SIP channel
driver.  This required updating the party redirecting information for the
channel based on the headers provided in the REFER request.  When the
redirecting party information is updated on the channel, a call to
ast_indicate_data occurs.  Because handle_request_refer still had the sip_pvt
locked, a deadlock could occur between the pbx_thread and the do_monitor thread
servicing the REFER request.

This patch preserves the proper locking order between the channel and the
sip_pvt by ensuring that the sip_pvt is unlocked prior to updating the party
redirecting information on the channel.

(closes issue AST-903)
Reported by: Matt Jordan
  jira_ast_903_trunk.patch by rmudgett (license 5621)

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

10 years agoParse ANI2 information from SIP From header parameters
Kinsey Moore [Tue, 12 Jun 2012 04:03:23 +0000 (04:03 +0000)]
Parse ANI2 information from SIP From header parameters

ANI2 information is now parsed out of SIP From headers when present in
the oli, isup-oli, and ss7-oli parameters and is available via the
CALLERID(ani2) dialplan function.

(closes issue ASTERISK-19912)
Patch-by: Rob Gagnon

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

10 years agoFix deadlock potential with ast_set_hangupsource() calls.
Richard Mudgett [Mon, 11 Jun 2012 17:34:08 +0000 (17:34 +0000)]
Fix deadlock potential with ast_set_hangupsource() calls.

Calling ast_set_hangupsource() with the channel lock held can result in a
deadlock because the function also locks the bridged channel.

(issue ASTERISK-19537)

(closes issue AST-891)
Reported by: Guenther Kelleter
Tested by: Guenther Kelleter

(closes issue ASTERISK-19801)
Reported by: Alec Davis

Merged revisions 368759 from

Merged revisions 368760 from

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

10 years agoFix coverity UNUSED_VALUE findings in core support level files
Kinsey Moore [Mon, 11 Jun 2012 15:23:30 +0000 (15:23 +0000)]
Fix coverity UNUSED_VALUE findings in core support level files

Most of these were just saving returned values without using them and
in some cases the variable being saved to could be removed as well.

(issue ASTERISK-19672)

Merged revisions 368738 from

Merged revisions 368739 from

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

10 years agoRecorded merge of revisions 368721 from
Kinsey Moore [Mon, 11 Jun 2012 14:12:08 +0000 (14:12 +0000)]
Recorded merge of revisions 368721 from

Fix compilation in dev-mode

Backport a compilation fix in md5.c from trunk that only showed up in
dev-mode under certain compiler versions.

Merged revisions 368719 from

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

10 years agoFix error paths in action_hangup() for AMI Hangup action.
Richard Mudgett [Fri, 8 Jun 2012 21:08:17 +0000 (21:08 +0000)]
Fix error paths in action_hangup() for AMI Hangup action.

* Check allocation function return values for failure.  Crashing is bad.

* Tweak ast_regex_string_to_regex_pattern() parameters for proper ast_str

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

10 years agoTweak ast_channel_softhangup_withcause_locked() to take a typed parameter.
Richard Mudgett [Fri, 8 Jun 2012 20:49:00 +0000 (20:49 +0000)]
Tweak ast_channel_softhangup_withcause_locked() to take a typed parameter.

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

10 years agoFix MWI update so LED display correct voicemail state after phone usage. Also fixes...
Igor Goncharovskiy [Fri, 8 Jun 2012 08:32:49 +0000 (08:32 +0000)]
Fix MWI update so LED display correct voicemail state after phone usage. Also fixes few warnings.
(closes issue #19675)
 Reported by: dbohling
       fixmwi.patch uploaded by dbohling (license 6378)

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

10 years agoSkinny cleanup (mwi_event_cb).
Damien Wedhorn [Thu, 7 Jun 2012 21:44:15 +0000 (21:44 +0000)]
Skinny cleanup (mwi_event_cb).

Original was testing for d->session, setting and testing again (all nested).

Removed duplicate testing and restructured function to test/return and then
the main code.

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

10 years agoSkinny cleanup.
Damien Wedhorn [Thu, 7 Jun 2012 21:23:42 +0000 (21:23 +0000)]
Skinny cleanup.

Removed d->registered which was mirroring d->session. Changed relevant
references to use d->session instead.

Moved setting and unsetting of l->device from session register to device
configuration. As such, l->device will always be valid unless it is has not
been configured to a device. Revised various test where checking if a device
is registered to use l->device->session.

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

10 years agoFix app_queue debug message use of args.options after the string has been parsed.
Richard Mudgett [Thu, 7 Jun 2012 20:39:25 +0000 (20:39 +0000)]
Fix app_queue debug message use of args.options after the string has been parsed.

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

10 years agoFix inverted test in app_queue for ringinuse.
Richard Mudgett [Thu, 7 Jun 2012 20:37:05 +0000 (20:37 +0000)]
Fix inverted test in app_queue for ringinuse.

Regression from -r367080 ringinuse commit.

(issue ASTERISK-19536)

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

10 years agoFix reloading an unchanged file with the Config Options API
Terry Wilson [Thu, 7 Jun 2012 20:32:07 +0000 (20:32 +0000)]
Fix reloading an unchanged file with the Config Options API

Adding multiple file support broke reloading an unchanged file. This
adds an enum for return values for the aco_process_* functions and
ensures that the config is not applied if res is not ACO_PROCESS_OK.


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

10 years agoFix a typo in format_ogg_vorbis.c: suport
Tzafrir Cohen [Thu, 7 Jun 2012 20:00:29 +0000 (20:00 +0000)]
Fix a typo in format_ogg_vorbis.c: suport


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

10 years agoAdd default handler documentation and standardize acl handler
Terry Wilson [Thu, 7 Jun 2012 15:43:37 +0000 (15:43 +0000)]
Add default handler documentation and standardize acl handler

Added documentation describing what flags and arguments to pass to
aco_option_register for default option types. Also changed the ACL
handler to use the flags parameter to differentiate between "permit"
and "deny" instead of adding an additional vararg parameter.


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

10 years agoFix POTS flash hook to orignate a second call deadlock.
Richard Mudgett [Wed, 6 Jun 2012 21:34:10 +0000 (21:34 +0000)]
Fix POTS flash hook to orignate a second call deadlock.

A deadlock can occur when a POTS phone tries to flash hook to originate a
second call for 3-way or transfer.  If another process is scanning the
channels container when the POTS line flash hooks then a deadlock will

* Release the channel and private locks when creating a new channel as a
result of a flash hook.

(closes issue ASTERISK-19842)
Reported by: rmudgett
Tested by: rmudgett

Merged revisions 368644 from

Merged revisions 368645 from

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

10 years agoFix a specific scenario where ACKs are not matched.
Mark Michelson [Wed, 6 Jun 2012 19:25:44 +0000 (19:25 +0000)]
Fix a specific scenario where ACKs are not matched.

If a dialog-starting INVITE contains a to-tag, then Asterisk
will respond with a 481. In this case, the resulting incoming
ACK would not be matched, so Asterisk would continue retransmitting
the 481 until the transaction times out.

There were two issues. Asterisk, upon creating a sip_pvt would generate
a local tag. However, when the time came to transmit the 481, since there
was a to-tag in the INVITE, Asterisk would place this original to-tag
in the 481 response. When the ACK came in, Asterisk would attempt to
match the to-tag in the ACK to the generated local tag. Unfortunately,
Asterisk never actually transmitted a response with the generated local
tag, so the to-tag in the ACK would not match.

The other problem was that when the 481 was sent, nothing was set
on the sip_pvt to indicate what CSeq is expected in the ACK.

To fix the first problem, we zero out the to-tag seen in the incoming
INVITE. This way, Asterisk, when time to send a response, will send
its generated local tag instead.

To fix the second problem, we set the sip_pvt's pendinginvite to the
CSeq of the INVITE when we send a 481.

(closes issue ASTERISK-19892)
Reported by Mark Michelson

Merged revisions 368625 from

Merged revisions 368629 from

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

10 years agoAdd feature modifier to versions produced from branches
Matthew Jordan [Wed, 6 Jun 2012 17:22:11 +0000 (17:22 +0000)]
Add feature modifier to versions produced from branches

Certain branches, such as Certified Asterisk, may have a modifier added to
them that specifies the features available in that branch.  For branches, this
modifier is expected to be reflected in the location of the branch in
subversion. For example, a subversion of URL of /certified/branches/1.8.11
would have a feature modifier of 'certified'.  This is slightly different then
how features are determined for tags, where the feature is part of the actual
tag name, e.g., "10.5.0-digiumphones".

In keeping with the nomenclature used for tags, the feature specifier for
branches is translated and placed after the revision numbers.  For the example
given previously, this would result in a branch version of
"Asterisk SVN-branch-1.8.11-cert-rXXXXXX".

Merged revisions 368604 from

Merged revisions 368605 from

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

10 years agoEnsure overlapping hold flags do not conflict
Kinsey Moore [Wed, 6 Jun 2012 16:11:01 +0000 (16:11 +0000)]
Ensure overlapping hold flags do not conflict

When changing between different modes of hold, the flags were not being
cleared out properly causing a failure to change hold states.

(closes issue ASTERISK-19919)
Patch-by: Morten Tryfoss
Reported-by: Morten Tryfoss

Merged revisions 368586 from

Merged revisions 368587 from

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

10 years agoFix parked call performing a DTMF blind transfer after being retrieved.
Richard Mudgett [Wed, 6 Jun 2012 01:11:12 +0000 (01:11 +0000)]
Fix parked call performing a DTMF blind transfer after being retrieved.

When a parked call was retrieved from the parking lot, it could not do a
blind transfer because it caused the involved calls to be hung up

* Made the ParkedCall application return the ast_bridge_call() return

(closes issue ABE-2862)
Reported by: Vlad Povorozniuc

Merged revisions 368567 from

Merged revisions 368568 from

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

10 years agoMake builtin_blindtransfer() fully use ast_async_goto() abilities.
Richard Mudgett [Wed, 6 Jun 2012 00:54:20 +0000 (00:54 +0000)]
Make builtin_blindtransfer() fully use ast_async_goto() abilities.

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

10 years agoMerge 'core' and 'core changes' sections in CHANGES file.
Jonathan Rose [Tue, 5 Jun 2012 16:25:14 +0000 (16:25 +0000)]
Merge 'core' and 'core changes' sections in CHANGES file.

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

10 years agoRecorded merge of revisions 368536 from
Kinsey Moore [Tue, 5 Jun 2012 15:28:28 +0000 (15:28 +0000)]
Recorded merge of revisions 368536 from

Resolve some build warnings

My newly upgraded compiler caught these usages of uninitialized values.
They weren't actually used.

Merged revisions 368533 from

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