[asterisk-commits] file: branch file/pimp_sip_location r382147 - in /team/file/pimp_sip_location...
SVN commits to the Asterisk project
asterisk-commits at lists.digium.com
Tue Feb 26 14:43:50 CST 2013
Author: file
Date: Tue Feb 26 14:43:44 2013
New Revision: 382147
URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=382147
Log:
Multiple revisions 381631,381639-381640,381646,381659,381672,381707,381720,381731,381751,381795,381871,381884,381897,381920,381952,381979,382010,382025,382057,382072,382112,382114,382117
........
r381631 | root | 2013-02-16 17:17:55 -0400 (Sat, 16 Feb 2013) | 9 lines
Merged revisions 381628 via svnmerge from
file:///srv/subversion/repos/asterisk/trunk
........
r381628 | rmudgett | 2013-02-16 14:44:44 -0600 (Sat, 16 Feb 2013) | 2 lines
confbridge: Rename i iterator variables to iter.
........
........
r381639 | file | 2013-02-16 17:36:24 -0400 (Sat, 16 Feb 2013) | 2 lines
Tweak where answer SDP is set as the dialog can't be locked from two different threads at once.
........
r381640 | file | 2013-02-16 19:50:13 -0400 (Sat, 16 Feb 2013) | 2 lines
Nothing to see here, move along.
........
r381646 | root | 2013-02-18 15:18:03 -0400 (Mon, 18 Feb 2013) | 25 lines
Merged revisions 381644 via svnmerge from
file:///srv/subversion/repos/asterisk/trunk
........
r381644 | rmudgett | 2013-02-18 13:12:11 -0600 (Mon, 18 Feb 2013) | 18 lines
confbridge: Add flags column to CLI "confbridge list <conference>"
* Added the following flags to the CLI "confbridge list <conference>" output:
A - The user is an admin
M - The user is a marked user
W - The user must wait for a marked user to join
E - The user will be kicked after the last marked user leaves the conference
w - The user is waiting for a marked user to join
* Added the following header to the AMI ConfbridgeList events:
WaitMarked, EndMarked, and Waiting.
(closes issue AST-1101)
Reported by: John Bigelow
Patches:
confbridge-show-admin3.txt (license #5091) patch uploaded by John Bigelow
Modified
........
........
r381659 | root | 2013-02-18 16:17:55 -0400 (Mon, 18 Feb 2013) | 17 lines
Merged revisions 381656 via svnmerge from
file:///srv/subversion/repos/asterisk/trunk
................
r381656 | jrose | 2013-02-18 13:48:47 -0600 (Mon, 18 Feb 2013) | 10 lines
PRESENCE_STATE: Provide better documentation for the 'e' option.
Notes that the 'e' option actually decodes data when used as a write function
such as with the SET application while it encodes data when used to read.
Review: https://reviewboard.asterisk.org/r/2335/
........
Merged revisions 381655 from http://svn.asterisk.org/svn/asterisk/branches/11
................
........
r381672 | root | 2013-02-18 17:17:51 -0400 (Mon, 18 Feb 2013) | 19 lines
Merged revisions 381670 via svnmerge from
file:///srv/subversion/repos/asterisk/trunk
................
r381670 | wdoekes | 2013-02-18 14:31:56 -0600 (Mon, 18 Feb 2013) | 12 lines
Remove "registertrying" and add "rtp_engine" from/to sip.conf.sample
The "registertrying" option was removed in r343220. The "rtp_engine"
option was added in r186078 but erroneously named "engine" in the sample.
Note that there is no global sip setting for a different engine.
........
Merged revisions 381668 from http://svn.asterisk.org/svn/asterisk/branches/1.8
........
Merged revisions 381669 from http://svn.asterisk.org/svn/asterisk/branches/11
................
........
r381707 | root | 2013-02-18 19:17:55 -0400 (Mon, 18 Feb 2013) | 30 lines
Merged revisions 381703 via svnmerge from
file:///srv/subversion/repos/asterisk/trunk
................
r381703 | kharwell | 2013-02-18 16:23:52 -0600 (Mon, 18 Feb 2013) | 23 lines
Fixed Confbridge file recording deadlock and appending.
A deadlock occurred after starting/stopping and then restarting a confbridge
recording. Upon starting a recording a record thread is created that holds a
lock until just before exiting. Stopping the recording does not stop/exit the
thread or release the lock. The thread waits until recording begins again.
Starting a stopped recording signals the thread to continue and start recording
again. However restarting the recording also created another record thread
resulting in a deadlock. The fix was to make sure the record thread was only
created once.
Also it was noted that filenames for the recordings were being concatenated for
each start/stop. This was fixed by creating a new file for each conference
session and appending the actual recorded data within the file (e.g. passing
the 'a' option to MixMonitor).
(issue AST-1088)
Reported by: John Bigelow
Review: http://reviewboard.digium.internal/r/374/
........
Merged revisions 381702 from http://svn.asterisk.org/svn/asterisk/branches/11
................
........
r381720 | root | 2013-02-19 03:17:58 -0400 (Tue, 19 Feb 2013) | 33 lines
Merged revisions 381717-381718 via svnmerge from
file:///srv/subversion/repos/asterisk/trunk
........
r381717 | wedhorn | 2013-02-19 00:50:57 -0600 (Tue, 19 Feb 2013) | 13 lines
Fixup skinny CLI completion.
Auto complete for skinny debug allows multiple options and negation, also add
debug all option. Usage example: 'skinny debug all -packets' (each can be
autocompleted including -packet).
Change show device to use device name. Remove the duplicate ast_strdup's from
place calling device complete return immediately from complete devicename and
complete linename so that multiple options are displayed on the CLI if more
than one option available.
Review: https://reviewboard.asterisk.org/r/2333/
........
r381718 | wedhorn | 2013-02-19 00:54:23 -0600 (Tue, 19 Feb 2013) | 11 lines
Add serviceURL stuff to skinny.
Patch adds all the packet and structure stuff to skinny to enable setting
service URLs in skinny, such as corporate directories.
This stuff is only relevant during load/unload as when activated. Also
some minor changes removing duplicated counting of addons and speedials in
handle_skinny_show_devices.
Review: https://reviewboard.asterisk.org/r/2321/
........
........
r381731 | root | 2013-02-19 12:18:27 -0400 (Tue, 19 Feb 2013) | 19 lines
Merged revisions 381729 via svnmerge from
file:///srv/subversion/repos/asterisk/trunk
........
r381729 | kharwell | 2013-02-19 09:41:37 -0600 (Tue, 19 Feb 2013) | 12 lines
Added Confbridge record_file_append option.
Currently, if one starts, stops, and then starts a recording again for a
conference the recorded data is appended to the file originally created
on the first record start. An option record_file_append has been added
that defaults to "yes", but when set to "no" will force creation of a new
file between every record start/stop.
(issue AST-1088)
Reported by: John Bigelow
Review: http://reviewboard.digium.internal/r/374/
........
........
r381751 | root | 2013-02-19 13:18:01 -0400 (Tue, 19 Feb 2013) | 44 lines
Merged revisions 381741,381749 via svnmerge from
file:///srv/subversion/repos/asterisk/trunk
................
r381741 | kharwell | 2013-02-19 10:23:29 -0600 (Tue, 19 Feb 2013) | 20 lines
Confbridge channels staying active when all participants leave.
If you started/stopped recording of a conference multiple times channels
would remain active even when all participants left the conference. This
was due to the fact that a reference to the confbridge was being added
every time a start record command was issued, but when the recording was
stopped there was no matching de-reference thus keeping the conference alive.
Made sure only a single reference is added for the record thread no matter how
many times recording is started/stopped. A de-reference is issued upon thread
ending.
Note, this issue is being fixed under AST-1088 since it relates to it and
should have been corrected along with those modifications.
(issue AST-1088)
Reported by: John Bigelow
........
Merged revisions 381737 from http://svn.asterisk.org/svn/asterisk/branches/11
................
r381749 | elguero | 2013-02-19 11:17:10 -0600 (Tue, 19 Feb 2013) | 15 lines
Add The Status Of A Module To The Output Of "CLI> module show"
When a module's configuration is not loadable, we still load the module but it
is not in a running state. When trying to troubleshoot, let's say, why
chan_motif is ignoring inbound XMPP traffic, there is no way to indicate that a
loaded module is not currently running.
(closes issue ASTERISK-21108)
Reported by: Rusty Newton
Tested by: Michael L. Young
Patches:
asterisk-21108_add_status-v2.diff Michael L. Young (license 5026)
Review: https://reviewboard.asterisk.org/r/2331/
................
........
r381795 | root | 2013-02-19 16:17:58 -0400 (Tue, 19 Feb 2013) | 24 lines
Merged revisions 381792 via svnmerge from
file:///srv/subversion/repos/asterisk/trunk
................
r381792 | kharwell | 2013-02-19 13:47:42 -0600 (Tue, 19 Feb 2013) | 17 lines
Write the correct callid to the data1 field in queue_log for transfer events.
The incorrect callid was being written to the "data1" field in queue_log table
for transfer events. The callid of the queue was being written instead of the
transfer target's callid. This now gets the correct "transfer to" number and
places that in the "data1" field of the queue_log table when a transfer event
is triggered.
(closes issue ASTERISK-19960)
Reported by: vladimir shmagin
........
Merged revisions 381770 from http://svn.asterisk.org/svn/asterisk/branches/1.8
........
Merged revisions 381791 from http://svn.asterisk.org/svn/asterisk/branches/11
................
........
r381871 | root | 2013-02-21 22:18:02 -0400 (Thu, 21 Feb 2013) | 23 lines
Merged revisions 381869 via svnmerge from
file:///srv/subversion/repos/asterisk/trunk
................
r381869 | mjordan | 2013-02-21 19:52:14 -0600 (Thu, 21 Feb 2013) | 16 lines
Properly detect launchd
Asterisk was a little too pro-active in claiming that it found launchd. On
systems without launchd - such as FreeBSD - this resulted in certain items
in Asterisk that conflict with launchd to not be selectable, such as
res_timing_kqueue.
(closes issue ASTERISK-20749)
Reported by: Oleg Baranov
........
Merged revisions 381847 from http://svn.asterisk.org/svn/asterisk/branches/1.8
........
Merged revisions 381848 from http://svn.asterisk.org/svn/asterisk/branches/11
................
........
r381884 | root | 2013-02-22 12:17:59 -0400 (Fri, 22 Feb 2013) | 20 lines
Merged revisions 381881 via svnmerge from
file:///srv/subversion/repos/asterisk/trunk
................
r381881 | jrose | 2013-02-22 09:51:20 -0600 (Fri, 22 Feb 2013) | 13 lines
app_dial: Honor the 'c' flag when the calling party hangs up
Apparently this feature became broken in 11, probably as a result
of the Hangup Cause project.
(closes issue ASTERISK-21113)
Reprted by: Heiko Wundram
Patches:
app_dial.patch uploaded by Heiko Wundram (license 5822)
........
Merged revisions 381880 from http://svn.asterisk.org/svn/asterisk/branches/11
................
........
r381897 | root | 2013-02-22 16:17:57 -0400 (Fri, 22 Feb 2013) | 40 lines
Merged revisions 381894 via svnmerge from
file:///srv/subversion/repos/asterisk/trunk
................
r381894 | elguero | 2013-02-22 13:40:02 -0600 (Fri, 22 Feb 2013) | 33 lines
Fix FastAGI To Properly Check For A Connection
When IPv6 support was added to FastAGI, the intent was to have the ability to
check all addresses resolved for a host since we might receive an IPv4 address
and an IPv6 address. The problem with the current code, is that, since we are
doing O_NONBLOCK, we get EINPROGRESS when calling ast_connect() but are ignoring
this instead of handling it. We break out of the loop and continue on. When we
later call ast_poll(), it succeeds but we never check if we have a connection or
not on the socket level. We then attempt to send data to the host address that
we think is setup and it fails. We then check the errno and see that we have
"connection refused" and then return with agi failed.
This patch does the following:
* Handles EINPROGRESS by creating the function handle_connection()
- ast_poll() was moved into this function
- This function checks the results of the connection on the socket level after
calling ast_poll()
* Continues to the next address if the above fails to create a connection
* Once all addresses resolved are tried and we still are unable to establish a
connection, then we return that the FastAGI call failed
(closes issue ASTERISK-21065)
Reported by: Jeremy Kister
Tested by: Jeremy Kister, Michael L. Young
Patches:
asterisk-21065_poll_correctly_v4.diff Michael L. Young (license 5026)
Review: https://reviewboard.asterisk.org/r/2330/
........
Merged revisions 381893 from http://svn.asterisk.org/svn/asterisk/branches/11
................
........
r381920 | root | 2013-02-24 12:18:38 -0400 (Sun, 24 Feb 2013) | 34 lines
Merged revisions 381918 via svnmerge from
file:///srv/subversion/repos/asterisk/trunk
................
r381918 | mjordan | 2013-02-24 09:45:29 -0600 (Sun, 24 Feb 2013) | 27 lines
Make ParkAndAnnounce return to priority + 1 when return context is not defined
The ParkAndAnnounce application documentation for the optional return_context
parameter states the following:
return_context
The goto-style label to jump the call back into after timeout. Default
'priority+1'.
Unfortunately, the application was sending the channel back into the dialplan
at 'priority', which is the ParkAndAnnounce application call. This causes an
infinite loop of the channel constantly being parked, announced, timed out,
parked, announced, timed out... while fun, especially for those callers you
wish to drive to the end of madness, this was not the intent of the
application.
(closes issue ASTERISK-20113)
Reported by: serginuez
patches:
app_parkandannounce.diff uploaded by serginuez (License 6405)
........
Merged revisions 381916 from http://svn.asterisk.org/svn/asterisk/branches/1.8
........
Merged revisions 381917 from http://svn.asterisk.org/svn/asterisk/branches/11
................
........
r381952 | root | 2013-02-24 13:17:53 -0400 (Sun, 24 Feb 2013) | 35 lines
Merged revisions 381949 via svnmerge from
file:///srv/subversion/repos/asterisk/trunk
................
r381949 | mjordan | 2013-02-24 10:27:47 -0600 (Sun, 24 Feb 2013) | 28 lines
Don't display the AMI ALL class authorization for users if they don't have it
When converting AMI class authorizations to a string representation, the
method always appends the ALL class authorization. This is especially
important for events, as they should always communicate that class
authorization - even if the event itself does not specify ALL as a class
authorization for itself. (Events have always assumed that the ALL class
authorization is implied when they are raised)
Unfortunately, this did mean that specifying a user with restricted class
authorizations would show up in the 'manager show user' CLI command as
having the ALL class authorization.
Rather then modifying the existing string manipulation function, this patch
adds a function that will only return a string if the field being compared
explicitly matches class authorization field it is being compared against.
This prevents ALL from being returned unless it is actually specified for
the user.
(closes issue ASTERISK-20397)
Reported by: Johan Wilfer
........
Merged revisions 381939 from http://svn.asterisk.org/svn/asterisk/branches/1.8
........
Merged revisions 381943 from http://svn.asterisk.org/svn/asterisk/branches/11
................
........
r381979 | root | 2013-02-24 19:17:56 -0400 (Sun, 24 Feb 2013) | 29 lines
Merged revisions 381977 via svnmerge from
file:///srv/subversion/repos/asterisk/trunk
................
r381977 | mjordan | 2013-02-24 17:01:17 -0600 (Sun, 24 Feb 2013) | 22 lines
Set the sin_family on the bind address socket during initialization
Somehow, chan_jingle has managed to operate for years without setting the
sin_family on its bindaddr socket. This patch properly sets the field during
initial module load to AF_INET.
Note that the patch on the issue was modified slightly to change the
initialization of the socket from allocation of a chan_jingle private to the
module initialization, as the bindaddr object (which is static) only needs to
have the address set once.
(closes issue ASTERISK-19341)
Reported by: andre valentin
patches:
0105-chan_jingle.patch uploaded by avalentin (License 6064)
........
Merged revisions 381975 from http://svn.asterisk.org/svn/asterisk/branches/1.8
........
Merged revisions 381976 from http://svn.asterisk.org/svn/asterisk/branches/11
................
........
r382010 | root | 2013-02-25 03:17:53 -0400 (Mon, 25 Feb 2013) | 36 lines
Merged revisions 382007-382008 via svnmerge from
file:///srv/subversion/repos/asterisk/trunk
........
r382007 | wedhorn | 2013-02-25 00:46:00 -0600 (Mon, 25 Feb 2013) | 13 lines
Add prinotify messages to skinny.
Adds both fixed and variable prinotify messages and clearprinotify messages to skinny.
Also adds cli function for pushing messages to devices. i
Initial code by snuffy, expanded by myself to include fixed messages.
(closes issue ASTERISK-21091)
Reported by: snuffy
Tested by: snuffy, myself
Patches:
skinny-prinotify02.diff uploaded by wedhorn (license 5019)
........
r382008 | wedhorn | 2013-02-25 01:09:37 -0600 (Mon, 25 Feb 2013) | 14 lines
More called details fixup for skinny.
Basically sets the callerid and callername to the first device talked to for the
purposes of putting the the calls made log on the device. Does not affect the device
displaying who the device is currently talking to.
Also some minor changes to use sub->exten in lieu of l->lastnumberdialed.
(closes issue ASTERISK-21095)
Reported by: wedhorn
Tested by: snuffy, myself
Patches:
skinny-calllogsoutbound03.diff uploaded by wedhorn (license 5019)
........
........
r382025 | root | 2013-02-25 09:18:34 -0400 (Mon, 25 Feb 2013) | 27 lines
Merged revisions 382023 via svnmerge from
file:///srv/subversion/repos/asterisk/trunk
................
r382023 | mjordan | 2013-02-25 06:51:24 -0600 (Mon, 25 Feb 2013) | 20 lines
Clean up use of va_end/va_args in res_config_mysql
There were several problems using variadic argument macros in res_config_mysql.
* Improper use of va_end. Multiple calls to va_end were possible resulting in
an unbalanced matching of va_start/va_end.
* Calls to va_arg after a possible encounter of a SENTINEL value.
This patch corrects those errors.
(closes issue ASTERISK-19451)
Reported by: wdoekes
patches:
ASTERISK-19451-1.8--2.diff uploaded by wdoekes (License 5674)
........
Merged revisions 382021 from http://svn.asterisk.org/svn/asterisk/branches/1.8
........
Merged revisions 382022 from http://svn.asterisk.org/svn/asterisk/branches/11
................
........
r382057 | file | 2013-02-25 15:55:35 -0400 (Mon, 25 Feb 2013) | 2 lines
Remove assumption that the INVITE session will exist when the session is ended.
........
r382072 | root | 2013-02-26 12:18:37 -0400 (Tue, 26 Feb 2013) | 59 lines
Merged revisions 382067,382070 via svnmerge from
file:///srv/subversion/repos/asterisk/trunk
................
r382067 | mjordan | 2013-02-26 09:26:16 -0600 (Tue, 26 Feb 2013) | 20 lines
Ensure that the default bridge/user profiles are always available
ConfBridge and Page require that there always be a default bridge and user
profile available. While properties of the default profiles can be overriden
in the configuration file, removing them can create situations where neither
application can function properly.
This patch ensures that if an administrator removes the profiles from the
confbridge.conf configuration file, the profiles are added upon load.
Documentation clarifying this has been added to the confbridge.conf.sample file.
Review: https://reviewboard.asterisk.org/r/2356/
(closes issue AST-1115)
Reported by: John Bigelow
Tested by: John Bigelow
........
Merged revisions 382066 from http://svn.asterisk.org/svn/asterisk/branches/11
................
r382070 | mjordan | 2013-02-26 09:52:02 -0600 (Tue, 26 Feb 2013) | 30 lines
Clean up ConfBridge commands to account for wait_marked users
When ConfBridge was refactored to better handle the concept of marked,
wait_marked, and normal users co-existing in a conference (thereby implementing
a state machine for the conference), the wait_marked users were put into their
own list of conference participants, separate from the active users. This list
is used for wait_marked users when they are waiting in a conference but no
marked user has joined; normal users may have joined at this point however.
There are several AMI/CLI commands that affect conference users that were not
checking the wait_marked users list:
* CLI/AMI commands that mute/unmute a participant. In this case, wait_marked
users have to remain in their particular state and should not be affected -
however, the commands would return "Channel not found" as opposed to the
appropriate error condition.
* CLI/AMI commands that kick a participant. An admin should always be able to
kick a participant out of the conference.
This patch fixes both sets of commands, and cleans up the CLI commands slightly
by allowing them to complete a participant name (this was supposed to have been
added, but the function call was commented out and wasn't implemented).
Review: https://reviewboard.asterisk.org/r/2346/
(closes issue AST-1114)
Reported by: John Bigelow
Tested by: John Bigelow
........
Merged revisions 382068 from http://svn.asterisk.org/svn/asterisk/branches/11
................
........
r382112 | mmichelson | 2013-02-26 15:55:31 -0400 (Tue, 26 Feb 2013) | 60 lines
Merge pool_shark2 branch.
Copied from the reviewboard description:
* The ast_sip_work structure and some of its methods have been removed in
favor of the generic task serializer.
* A PJSIP module, the message distributor, has been added. This module has a
high priority and gets called into after the PJSIP transport layer but before
the PJSIP transaction layer. The distributor is used as a means of moving an
incoming message from a PJSIP thread into a SIP servant thread. Then from the
SIP servant thread, the message bubbles up through the PJSIP transaction layer,
PJSIP dialog layer, and any relevant PJSIP modules that Asterisk has registered.
This means that any PJSIP module that registers itself at the transaction layer
or higher will have all of its incoming SIP messages already in a SIP servant
thread when called back. Unless they need to serialize operations, they do not
need to do any special processing to ensure the incoming message is handled in a
specific thread. A good example of this is what the SIP OPTIONS handler does in
this set of changes.
* Two new PJSIP modules have been added above the dialog layer. The endpoint
module and the authenticator module are responsible for doing endpoint lookups
and authentication for incoming requests. The placement of these modules means
that they are only ever called into for out-of-dialog requests. This means that
on an initial INVITE, or on a REGISTER, or on an OPTIONS request, or any other
out-of-dialog request, the application-layer modules do not need to worry about
endpoint lookup or authentication since that is done already for them. If they
need the endpoint, they can call ast_pjsip_rdata_get_endpoint(). For dialog-
forming application modules, they will want to save off the endpoint that the
endpoint module finds since the endpoint will not be looked up again on in-
dialog requests, meaning they will not be able to call
ast_pjsip_rdata_get_endpoint().
* Modules that were passing incoming messages off to servants no longer need to
do this, so they have been changed not to. So the session module and options
module now handle incoming requests in-line rather than pushing them to a
servant thread.
* Since it no longer makes sense to pre-allocate a serializer before an
ast_sip_session, the functions for allocating SIP sessions has been changed not
to take a serializer any more. Instead, the allocation routine creates its own.
In addition, the behavior of endpoint references has changed from how it
previously was. Now allocating a session results in the endpoint reference count
increasing by one rather than having the session inherit the reference passed
into it. This makes endpoint reference handling much cleaner, especially in the
session module.
* To assist programmers who want to write PJSIP modules in Asterisk, there is a
helper method called ast_sip_thread_is_servant() that has been added. It can be
used to discern if the thread you currently are in is a SIP servant thread. This
means it is not as necessary to dig into the PJSIP internals to know what type
of thread you are in when called back. Instead, you can be safe and use this
function. This function is currently unused though, since there hasn't been a
need for it yet. This is because all uses of SIP servants where it may be
questionable what type of thread we are in need to serialize the task anyway.
* Documentation of the SIP threadpool has been updated.
Review: https://reviewboard.asterisk.org/r/2355
........
r382114 | mmichelson | 2013-02-26 16:09:59 -0400 (Tue, 26 Feb 2013) | 3 lines
Invert logic so that things actually start up properly.
........
r382117 | root | 2013-02-26 16:18:07 -0400 (Tue, 26 Feb 2013) | 49 lines
Merged revisions 382106,382109,382113 via svnmerge from
file:///srv/subversion/repos/asterisk/trunk
................
r382106 | tzafrir | 2013-02-26 13:29:14 -0600 (Tue, 26 Feb 2013) | 11 lines
Remove unneeded linux-gnueabi*
As of r380522 the configure scripts converts the value of linux-gnueabi*
of OSARCH to "linux-gnu". So no point in testing for those values.
........
Merged revisions 382087 from http://svn.asterisk.org/svn/asterisk/branches/1.8
........
Merged revisions 382096 from http://svn.asterisk.org/svn/asterisk/branches/11
................
r382109 | wdoekes | 2013-02-26 13:36:30 -0600 (Tue, 26 Feb 2013) | 13 lines
Correct RPID parsing for unquoted display-name.
Parsing Remote-Party-ID will now succeed if display-name is of the
*(token LWS) kind and not just the quoted-string kind.
Review: https://reviewboard.asterisk.org/r/2341/
........
Merged revisions 382107 from http://svn.asterisk.org/svn/asterisk/branches/1.8
........
Merged revisions 382108 from http://svn.asterisk.org/svn/asterisk/branches/11
................
r382113 | tzafrir | 2013-02-26 14:05:02 -0600 (Tue, 26 Feb 2013) | 14 lines
Consider linux-gnuspe as linux-gnu
* The powerpcspe Linux port uses linux-gnuspe as the OS string.
* Our build system shouldn't really care for that, so just call it linux-gnu.
* Original report: Roland Stigge , http://bugs.debian.org/701505
Review: https://reviewboard.asterisk.org/r/2357/
........
Merged revisions 382110 from http://svn.asterisk.org/svn/asterisk/branches/1.8
........
Merged revisions 382111 from http://svn.asterisk.org/svn/asterisk/branches/11
................
........
Merged revisions 381631,381639-381640,381646,381659,381672,381707,381720,381731,381751,381795,381871,381884,381897,381920,381952,381979,382010,382025,382057,382072,382112,382114,382117 from http://svn.asterisk.org/svn/asterisk/team/group/pimp_my_sip
Added:
team/file/pimp_sip_location/res/res_sip/sip_distributor.c
- copied unchanged from r382117, team/group/pimp_my_sip/res/res_sip/sip_distributor.c
Modified:
team/file/pimp_sip_location/ (props changed)
team/file/pimp_sip_location/CHANGES
team/file/pimp_sip_location/addons/res_config_mysql.c
team/file/pimp_sip_location/apps/app_confbridge.c
team/file/pimp_sip_location/apps/app_dial.c
team/file/pimp_sip_location/apps/app_parkandannounce.c
team/file/pimp_sip_location/apps/confbridge/conf_config_parser.c
team/file/pimp_sip_location/apps/confbridge/include/confbridge.h
team/file/pimp_sip_location/channels/chan_gulp.c
team/file/pimp_sip_location/channels/chan_jingle.c
team/file/pimp_sip_location/channels/chan_motif.c
team/file/pimp_sip_location/channels/chan_sip.c
team/file/pimp_sip_location/channels/chan_skinny.c
team/file/pimp_sip_location/configs/confbridge.conf.sample
team/file/pimp_sip_location/configs/sip.conf.sample
team/file/pimp_sip_location/configs/skinny.conf.sample
team/file/pimp_sip_location/configure
team/file/pimp_sip_location/configure.ac
team/file/pimp_sip_location/funcs/func_presencestate.c
team/file/pimp_sip_location/include/asterisk/module.h
team/file/pimp_sip_location/include/asterisk/res_sip.h
team/file/pimp_sip_location/include/asterisk/res_sip_session.h
team/file/pimp_sip_location/main/Makefile
team/file/pimp_sip_location/main/cli.c
team/file/pimp_sip_location/main/features.c
team/file/pimp_sip_location/main/loader.c
team/file/pimp_sip_location/main/manager.c
team/file/pimp_sip_location/res/pjproject/pjsip/include/pjsip/sip_endpoint.h
team/file/pimp_sip_location/res/pjproject/pjsip/src/pjsip/sip_endpoint.c
team/file/pimp_sip_location/res/res_agi.c
team/file/pimp_sip_location/res/res_sip.c
team/file/pimp_sip_location/res/res_sip.exports.in
team/file/pimp_sip_location/res/res_sip/config_transport.c
team/file/pimp_sip_location/res/res_sip/sip_options.c
team/file/pimp_sip_location/res/res_sip_endpoint_identifier_ip.c
team/file/pimp_sip_location/res/res_sip_session.c
team/file/pimp_sip_location/res/snmp/agent.c
Propchange: team/file/pimp_sip_location/
------------------------------------------------------------------------------
Binary property 'branch-11-merged' - no diff available.
Propchange: team/file/pimp_sip_location/
------------------------------------------------------------------------------
--- pimp-integrated (original)
+++ pimp-integrated Tue Feb 26 14:43:44 2013
@@ -1,1 +1,1 @@
-/team/group/pimp_my_sip:1-381628
+/team/group/pimp_my_sip:1-382136
Modified: team/file/pimp_sip_location/CHANGES
URL: http://svnview.digium.com/svn/asterisk/team/file/pimp_sip_location/CHANGES?view=diff&rev=382147&r1=382146&r2=382147
==============================================================================
--- team/file/pimp_sip_location/CHANGES (original)
+++ team/file/pimp_sip_location/CHANGES Tue Feb 26 14:43:44 2013
@@ -145,6 +145,9 @@
* Added announcement configuration option to user profile. If set the sound
file will be played to the user, and only the user, upon joining the
conference bridge.
+
+ * Added record_file_append option that defaults to "yes", but if set to no
+ will create a new file between each start/stop recording.
Dial
Modified: team/file/pimp_sip_location/addons/res_config_mysql.c
URL: http://svnview.digium.com/svn/asterisk/team/file/pimp_sip_location/addons/res_config_mysql.c?view=diff&rev=382147&r1=382146&r2=382147
==============================================================================
--- team/file/pimp_sip_location/addons/res_config_mysql.c (original)
+++ team/file/pimp_sip_location/addons/res_config_mysql.c Tue Feb 26 14:43:44 2013
@@ -343,9 +343,7 @@
}
/* Get the first parameter and first value in our list of passed paramater/value pairs */
- newparam = va_arg(ap, const char *);
- newval = va_arg(ap, const char *);
- if (!newparam || !newval) {
+ if (!(newparam = va_arg(ap, const char *)) || !(newval = va_arg(ap, const char *))) {
ast_log(LOG_WARNING, "MySQL RealTime: Realtime retrieval requires at least 1 parameter and 1 value to search on.\n");
release_database(dbh);
return NULL;
@@ -376,7 +374,6 @@
ESCAPE_STRING(buf, newval);
ast_str_append(&sql, 0, " AND %s%s '%s'", newparam, op, ast_str_buffer(buf));
}
- va_end(ap);
ast_debug(1, "MySQL RealTime: Retrieve SQL: %s\n", ast_str_buffer(sql));
@@ -457,9 +454,7 @@
}
/* Get the first parameter and first value in our list of passed paramater/value pairs */
- newparam = va_arg(ap, const char *);
- newval = va_arg(ap, const char *);
- if (!newparam || !newval) {
+ if (!(newparam = va_arg(ap, const char *)) || !(newval = va_arg(ap, const char *))) {
ast_log(LOG_WARNING, "MySQL RealTime: Realtime retrieval requires at least 1 parameter and 1 value to search on.\n");
ast_config_destroy(cfg);
release_database(dbh);
@@ -498,8 +493,6 @@
if (initfield) {
ast_str_append(&sql, 0, " ORDER BY %s", initfield);
}
-
- va_end(ap);
ast_debug(1, "MySQL RealTime: Retrieve SQL: %s\n", ast_str_buffer(sql));
@@ -581,9 +574,7 @@
}
/* Get the first parameter and first value in our list of passed paramater/value pairs */
- newparam = va_arg(ap, const char *);
- newval = va_arg(ap, const char *);
- if (!newparam || !newval) {
+ if (!(newparam = va_arg(ap, const char *)) || !(newval = va_arg(ap, const char *))) {
ast_log(LOG_WARNING, "MySQL RealTime: Realtime update requires at least 1 parameter and 1 value to update.\n");
release_table(table);
release_database(dbh);
@@ -633,7 +624,6 @@
internal_require(database, tablename, newparam, RQ_CHAR, ast_str_strlen(buf), SENTINEL);
}
}
- va_end(ap);
ESCAPE_STRING(buf, lookup);
ast_str_append(&sql, 0, " WHERE `%s` = '%s'", keyfield, ast_str_buffer(buf));
@@ -667,7 +657,7 @@
{
struct mysql_conn *dbh;
my_ulonglong numrows;
- int first = 1;
+ int first;
const char *newparam, *newval;
struct ast_str *sql = ast_str_thread_get(&sql_buf, 100), *buf = ast_str_thread_get(&scratch_buf, 100);
struct ast_str *where = ast_str_thread_get(&sql2_buf, 100);
@@ -706,6 +696,7 @@
return -1;
}
+ first = 1;
while ((newparam = va_arg(ap, const char *))) {
if (!(column = find_column(table, newparam))) {
ast_log(LOG_ERROR, "Updating on column '%s', but that column does not exist within the table '%s'!\n", newparam, tablename);
@@ -753,7 +744,7 @@
internal_require(database, tablename, newparam, RQ_CHAR, ast_str_strlen(buf), SENTINEL);
}
}
- va_end(ap);
+
release_table(table);
ast_str_append(&sql, 0, " %s", ast_str_buffer(where));
@@ -802,9 +793,7 @@
return -1;
}
/* Get the first parameter and first value in our list of passed paramater/value pairs */
- newparam = va_arg(ap, const char *);
- newval = va_arg(ap, const char *);
- if (!newparam || !newval) {
+ if (!(newparam = va_arg(ap, const char *)) || !(newval = va_arg(ap, const char *))) {
ast_log(LOG_WARNING, "MySQL RealTime: Realtime storage requires at least 1 parameter and 1 value to search on.\n");
release_database(dbh);
return -1;
@@ -833,7 +822,6 @@
ast_str_append(&sql2, 0, ", '%s'", ast_str_buffer(buf));
}
}
- va_end(ap);
ast_str_append(&sql, 0, "%s)", ast_str_buffer(sql2));
ast_debug(1,"MySQL RealTime: Insert SQL: %s\n", ast_str_buffer(sql));
@@ -901,7 +889,6 @@
ESCAPE_STRING(buf, newval);
ast_str_append(&sql, 0, " AND `%s` = '%s'", newparam, ast_str_buffer(buf));
}
- va_end(ap);
ast_debug(1, "MySQL RealTime: Delete SQL: %s\n", ast_str_buffer(sql));
Modified: team/file/pimp_sip_location/apps/app_confbridge.c
URL: http://svnview.digium.com/svn/asterisk/team/file/pimp_sip_location/apps/app_confbridge.c?view=diff&rev=382147&r1=382146&r2=382147
==============================================================================
--- team/file/pimp_sip_location/apps/app_confbridge.c (original)
+++ team/file/pimp_sip_location/apps/app_confbridge.c Tue Feb 26 14:43:44 2013
@@ -596,6 +596,37 @@
return tmp;
}
+static void set_rec_filename(struct conference_bridge *bridge, struct ast_str **filename)
+{
+ char *rec_file = bridge->b_profile.rec_file;
+ time_t now;
+ char *ext;
+
+ if (ast_str_strlen(*filename) && ast_test_flag(&bridge->b_profile, BRIDGE_OPT_RECORD_FILE_APPEND)) {
+ return;
+ }
+
+ time(&now);
+
+ ast_str_reset(*filename);
+ if (ast_strlen_zero(rec_file)) {
+ ast_str_set(filename, 0, "confbridge-%s-%u.wav", bridge->name, (unsigned int)now);
+ } else {
+ /* insert time before file extension */
+ ext = strrchr(rec_file, '.');
+ if (ext) {
+ ast_str_set_substr(filename, 0, rec_file, ext - rec_file);
+ ast_str_append(filename, 0, "-%u%s", (unsigned int)now, ext);
+ } else {
+ ast_str_set(filename, 0, "%s-%u", rec_file, (unsigned int)now);
+ }
+ }
+
+ if (ast_test_flag(&bridge->b_profile, BRIDGE_OPT_RECORD_FILE_APPEND)) {
+ ast_str_append(filename, 0, ",a");
+ }
+}
+
static void *record_thread(void *obj)
{
struct conference_bridge *conference_bridge = obj;
@@ -614,16 +645,7 @@
/* XXX If we get an EXIT right here, START will essentially be a no-op */
while (conference_bridge->record_state != CONF_RECORD_EXIT) {
- if (!(ast_strlen_zero(conference_bridge->b_profile.rec_file))) {
- ast_str_append(&filename, 0, "%s", conference_bridge->b_profile.rec_file);
- } else {
- time_t now;
- time(&now);
- ast_str_append(&filename, 0, "confbridge-%s-%u.wav",
- conference_bridge->name,
- (unsigned int) now);
- }
-
+ set_rec_filename(conference_bridge, &filename);
chan = ast_channel_ref(conference_bridge->record_chan);
ast_answer(chan);
pbx_exec(chan, mixmonapp, ast_str_buffer(filename));
@@ -753,9 +775,16 @@
*/
static int start_conf_record_thread(struct conference_bridge *conference_bridge)
{
+ conf_start_record(conference_bridge);
+
+ /*
+ * if the thread has already been started, don't start another
+ */
+ if (conference_bridge->record_thread != AST_PTHREADT_NULL) {
+ return 0;
+ }
+
ao2_ref(conference_bridge, +1); /* give the record thread a ref */
-
- conf_start_record(conference_bridge);
if (ast_pthread_create_background(&conference_bridge->record_thread, NULL, record_thread, conference_bridge)) {
ast_log(LOG_WARNING, "Failed to create recording channel for conference %s\n", conference_bridge->name);
@@ -2151,16 +2180,39 @@
return 0;
}
+static int kick_conference_participant(struct conference_bridge *bridge, const char *channel)
+{
+ struct conference_bridge_user *participant = NULL;
+
+ SCOPED_AO2LOCK(bridge_lock, bridge);
[... 3255 lines stripped ...]
More information about the asterisk-commits
mailing list