aboutsummaryrefslogtreecommitdiffstats
path: root/apps
AgeCommit message (Collapse)AuthorFilesLines
2008-06-04Make sure that the string we set will survive the unref ofmmichelson1-2/+2
the queue member. Thanks to Russell, who pointed this out. git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@120513 f38db490-d61c-443f-a65b-d21fe96a405b
2008-06-04Tab completion when removing a member should give the member's interface,mmichelson1-1/+1
not the name, since the interface is what is expected for the command. (closes issue #12783) Reported by: davevg git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@120285 f38db490-d61c-443f-a65b-d21fe96a405b
2008-06-02Fix another typo in documentationrussell1-1/+1
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@119530 f38db490-d61c-443f-a65b-d21fe96a405b
2008-06-01small typo fix 'retires' => 'retries'mvanbaak1-1/+1
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@119478 f38db490-d61c-443f-a65b-d21fe96a405b
2008-05-30When joinempty=strict, it only failed on join if there were busy members. Iftilghman1-1/+1
all members were logged out OR paused, then it (incorrectly) let callers join the queue. (closes issue #12451) Reported by: davidw git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@119404 f38db490-d61c-443f-a65b-d21fe96a405b
2008-05-29 - Fix a typo in the argument to Playtonesrussell1-8/+3
- use ast_safe_sleep() instead of calling the wait application (thanks to tilghman for pointing these out!) git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@119012 f38db490-d61c-443f-a65b-d21fe96a405b
2008-05-29 - Mark app_milliwatt dependent on res_indications (thanks to jsmith)russell1-1/+5
- fix a typo in a log message (thanks to qwell) git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@118961 f38db490-d61c-443f-a65b-d21fe96a405b
2008-05-29Change milliwatt to use the proper tone by default (1004 Hz) instead of 1000 Hz.russell1-20/+62
An option is there to use 1000 Hz for anyone that might want it. git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@118956 f38db490-d61c-443f-a65b-d21fe96a405b
2008-05-29(closes issue #10668)murf1-9/+91
(closes issue #11721) (closes issue #12726) Reported by: arkadia Tested by: murf These changes: 1. revert the changes made via bug 10668; I should have known that such changes, even tho they made sense at the time, seemed like an omission, etc, were actually integral to the CDR system via forkCDR. It makes sense to me now that forkCDR didn't natively end any CDR's, but rather depended on natively closing them all at hangup time via traversing and closing them all, whether locked or not. I still don't completely understand the benefits of setvar and answer operating on locked cdrs, but I've seen enough to revert those changes also, and stop messing up users who depended on that behavior. bug 12726 found reverting the changes fixed his changes, and after a long review and working on forkCDR, I can see why. 2. Apply the suggested enhancements proposed in 10668, but in a completely compatible way. ForkCDR will behave exactly as before, but now has new options that will allow some actions to be taken that will slightly modify the outcome and side-effects of forkCDR. Based on conversations I've had with various people, these small tweaks will allow some users to get the behavior they need. For instance, users executing forkCDR in an AGI script will find the answer time set, and DISPOSITION set, a situation not covered when the routines were first written. 3. A small problem in the cdr serializer would output answer and end times even when they were not set. This is now fixed. git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@118858 f38db490-d61c-443f-a65b-d21fe96a405b
2008-05-27Russell noted to me that in the case that separate threads use theirmmichelson1-7/+6
own addressing system, the fix I made for issue 12376 does not guarantee uniqueness to the datastores' uids. Though I know of no system that works this way, I am going to change this right now to prevent trying to track down some future bug that may occur and cause untold hours of debugging time to track down. The change involves using a global counter which increases with each new chanspy_ds which is created. This guarantees uniqueness. git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@118509 f38db490-d61c-443f-a65b-d21fe96a405b
2008-05-27small update to the g() option of app_voicemail tomvanbaak1-0/+1
note that gain changes only work on zap channels right now. issue #12578 shows it's not clear right now. git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@118416 f38db490-d61c-443f-a65b-d21fe96a405b
2008-05-27Add a unique id to the datastore allocated in app_chanspy sincemmichelson1-2/+7
it is possible that multiple spies may be listening to the same channel. (closes issue #12376) Reported by: DougUDI Patches: 12376_chanspy_uid.diff uploaded by putnopvut (license 60) Tested by: destiny6628 (closes issue #12243) Reported by: atis git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@118365 f38db490-d61c-443f-a65b-d21fe96a405b
2008-05-14Detect another way for a connection to have gone away.tilghman1-1/+8
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@116296 f38db490-d61c-443f-a65b-d21fe96a405b
2008-05-05Don't consider a caller "handled" until the caller is bridged withmmichelson1-2/+1
a queue member. There was too much of an opportunity for the member to hang up (either during a delay, announcement, or overly long agi) between the time that he answered the phone and the time when he actually was bridged with the caller. The consequence of this was that if the member hung up in that interval, then proper abandonment details would not be noted in the queue log if the caller were to hang up at any point after the member hangup. (closes issue #12561) Reported by: ablackthorn git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@115320 f38db490-d61c-443f-a65b-d21fe96a405b
2008-04-29Use the MACRO_CONTEXT and MACRO_EXTEN channel variables instead of the ↵mmichelson1-4/+9
channel's macrocontext and macroexten fields. This is needed because if macros are daisy-chained, the incorrect context and extension are placed on the new channel. I also added locking to the channel prior to accessing these variables as noted in trunk's janitor project file. (closes issue #12549) Reported by: darren1713 Patches: app_queue.c.macroextenpatch uploaded by darren1713 (license 116) (with modifications from me) Tested by: putnopvut git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@114848 f38db490-d61c-443f-a65b-d21fe96a405b
2008-04-28When modules are embedded, they take on a different name, without the ".so"tilghman1-2/+7
extension. Specifically check for this name, when we're checking if a module is loaded. (Closes issue #12534) git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@114708 f38db490-d61c-443f-a65b-d21fe96a405b
2008-04-25Move the unlock of the spyee channel to outside the start_spying() function ↵mmichelson1-7/+6
so that the channel is not unlocked twice when using whisper mode. git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@114662 f38db490-d61c-443f-a65b-d21fe96a405b
2008-04-24Output of channel variables when eventwhencalled=vars was setmmichelson1-4/+4
was being truncated two characters. This patch corrects the problem. (closes issue #12493) Reported by: davidw git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@114628 f38db490-d61c-443f-a65b-d21fe96a405b
2008-04-23Fix an issue that caused getting the correct next channel to not always work.russell1-5/+3
Also, remove setting the amount of time to wait for a digit from 5 seconds back down to 1/10 of a second. I believe this was so the beep didn't get played over and over really fast, but a while back I put in another fix for that issue. (closes issue #12498) Reported by: jsmith Patches: app_chanspy_channel_walk.trunk.patch uploaded by jsmith (license 15) git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@114597 f38db490-d61c-443f-a65b-d21fe96a405b
2008-04-19Ensure that help text terminates with a newlinetilghman1-1/+1
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@114299 f38db490-d61c-443f-a65b-d21fe96a405b
2008-04-18For consistency sake, ensure that the values that ${CALLINGPRES} returns are ↵tilghman1-2/+5
valid as an input to SetCallingPres. (Closes issue #12472) git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@114242 f38db490-d61c-443f-a65b-d21fe96a405b
2008-04-17Declaration of the peer channel in this scope was making it so the peer ↵mmichelson1-1/+0
variable defined in the outer scope was never set properly, therefore making iterating through the channel list always restart from the beginning. This bug would have affected anyone who called chanspy without specifying a first argument. (closes issue #12461) Reported by: stever28 git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@114226 f38db490-d61c-443f-a65b-d21fe96a405b
2008-04-17Make sure we have enough room for the recording's filename.seanbright1-2/+2
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@114191 f38db490-d61c-443f-a65b-d21fe96a405b
2008-04-16Include the proper headers for using mkdir on FreeBSD.file1-0/+2
(closes issue #12430) Reported by: ys Patches: app_meetme.c.diff uploaded by ys (license 281) git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@114167 f38db490-d61c-443f-a65b-d21fe96a405b
2008-04-15Allow autofill to work in the general section of queues.conf.qwell1-2/+5
Additionally, don't try to (re)set options when they have empty values in realtime (all unset columns would have an empty value). (closes issue #12445) Reported by: atis Patches: 12445-autofill.diff uploaded by qwell (license 4) git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@114133 f38db490-d61c-443f-a65b-d21fe96a405b
2008-04-14If the datastore has been moved to another channel due to a masquerade, thenmmichelson2-4/+15
freeing the datastore here causes an eventual double free when the new channel hangs up. We should only free the datastore if we were able to successfully remove it from the channel we are referencing (i.e. the datastore was not moved). (closes issue #12359) Reported by: pguido git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@114112 f38db490-d61c-443f-a65b-d21fe96a405b
2008-04-10Forgot the 1.4 branch for russian language fix.file1-2/+2
(closes issue #12404) Reported by: IgorG Patches: voicemail_ru_hardcoded-v1.patch uploaded by IgorG (license 20) git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@114032 f38db490-d61c-443f-a65b-d21fe96a405b
2008-04-10Create the directory where name recordings will go if it does not exist.file1-1/+10
(closes issue #12311) Reported by: rkeene Patches: 12311-mkdir.diff uploaded by qwell (license 4) git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@114029 f38db490-d61c-443f-a65b-d21fe96a405b
2008-04-08Fix potential buffer overflow that could happen if more than 100 announce filesmmichelson1-1/+1
were specified when calling ParkAndAnnounce. This overflow is not exploitable remotely and so there is no need for a security advisory. (closes issue #12386) Reported by: davidw git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@113507 f38db490-d61c-443f-a65b-d21fe96a405b
2008-04-02Ensure that there is no timeout if none is specified.mmichelson1-1/+1
(closes issue #12349) Reported by: johnlange git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@112393 f38db490-d61c-443f-a65b-d21fe96a405b
2008-03-31Fix a silly infinite loop when choosing an invalid option.qwell1-0/+1
(closes issue #12315) Reported by: jmls git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@112068 f38db490-d61c-443f-a65b-d21fe96a405b
2008-03-27These small documentation updates made in response to a query inmurf1-0/+2
asterisk-users, where a user was using Playback, but needed the features of Background, and had no idea that Background existed, or that it might provide the features he needed. I thought the best way to avert these kinds of queries was to provide "See Also" references in all three of "Background", "Playback", "WaitExten". Perhaps a project to do this with all related apps is in order. git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@111391 f38db490-d61c-443f-a65b-d21fe96a405b
2008-03-26This code change is made just for clarification. It does exactlymmichelson1-1/+1
the same thing as before. It just doesn't look as wrong. git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@111121 f38db490-d61c-443f-a65b-d21fe96a405b
2008-03-26Add a lock to the vm_state structure and use the lock around mail_open callsmmichelson1-0/+7
to prevent concurrent access of the same mailstream. This, along with trunk's ability to configure TCP timeouts for IMAP storage will help to prevent crashes and hangs when using voicemail with IMAP storage. (closes issue #10487) Reported by: ewilhelmsen git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@111049 f38db490-d61c-443f-a65b-d21fe96a405b
2008-03-19Fix a bug where when calls on the trunk side hang up while on hold, the staterussell1-2/+3
is not properly reflected. (closes issue #11990, reported by anakaoka, patched by me) git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@110163 f38db490-d61c-443f-a65b-d21fe96a405b
2008-03-19Add a missing unlock in the case that memory allocation fails in app_chanspy.mmichelson1-4/+3
Thanks to Russell for confirming that this was an issue. git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@110083 f38db490-d61c-443f-a65b-d21fe96a405b
2008-03-18Fix one place where the chanspy datastore isn't removed from a channel.russell1-1/+3
(issue #12243, reported by atis, patch by me) git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@109763 f38db490-d61c-443f-a65b-d21fe96a405b
2008-03-18This patch makes it so that all queue member status changes are handled ↵mmichelson1-96/+56
through device state code. This removes several problems people were seeing where their queue members would get into an "unknown" state. Huge props go to atis on this one since he was the one who found the code section that was causing the problem and proposed the solution. I just wrote what he suggested :) (closes issue #12127) Reported by: atis Patches: 12127v3.patch uploaded by putnopvut (license 60) Tested by: atis, jvandal git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@109713 f38db490-d61c-443f-a65b-d21fe96a405b
2008-03-17Make sure that we release the lock on the spyee channel if the spyee or spy ↵mmichelson1-1/+3
has hung up (closes issue #12232) Reported by: atis git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@109012 f38db490-d61c-443f-a65b-d21fe96a405b
2008-03-13Fix another issue that was causing crashes in chanspy. This introduces a newrussell1-0/+10
datastore callback, called chan_fixup(). The concept is exactly like the fixup callback that is used in the channel technology interface. This callback gets called when the owning channel changes due to a masquerade. Before this was introduced, if a masquerade happened on a channel being spyed on, the channel pointer in the datastore became invalid. (closes issue #12187) (reported by, and lots of testing from atis) (props to file for the help with ideas) git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@108583 f38db490-d61c-443f-a65b-d21fe96a405b
2008-03-13Fix a couple uses of sprintf. The second one could actually cause an overflowrussell1-2/+2
of a stack buffer. It's not a security issue though, it only depends on your configuration. git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@108469 f38db490-d61c-443f-a65b-d21fe96a405b
2008-03-12(closes issue #12187, reported by atis, fixed by me after some brainstormingrussell1-1/+3
on the issue with mmichelson) - Update copyright info on app_chanspy. - Fix a race condition that caused app_chanspy to crash. The issue was that the chanspy datastore magic that was used to ensure that spyee channels did not disappear out from under the code did not completely solve the problem. It was actually possible for chanspy to acquire a channel reference out of its datastore to a channel that was in the middle of being destroyed. That was because datastore destruction in ast_channel_free() was done near the end. So, this left the code in app_chanspy accessing a channel that was partially, or completely invalid because it was in the process of being free'd by another thread. The following sort of shows the code path where the race occurred: ============================================================================= Thread 1 (PBX thread for spyee chan) || Thread 2 (chanspy) --------------------------------------||------------------------------------- ast_channel_free() || - remove channel from channel list || - lock/unlock the channel to ensure || that no references retrieved from || the channel list exist. || --------------------------------------||------------------------------------- || channel_spy() - destroy some channel data || - Lock chanspy datastore || - Retrieve reference to channel || - lock channel || - Unlock chanspy datastore --------------------------------------||------------------------------------- - destroy channel datastores || - call chanspy datastore d'tor || which NULL's out the ds' || - Operate on the channel ... reference to the channel || || - free the channel || || || - unlock the channel --------------------------------------||------------------------------------- ============================================================================= git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@108135 f38db490-d61c-443f-a65b-d21fe96a405b
2008-03-12Add a trigger mode that triggers on both read and write. The actual function ↵file1-1/+1
that returns the combined audio frame though will wait until both sides have fed in audio, or until one side stops (such as the case when you call Wait). (closes issue #11945) Reported by: xheliox git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@108083 f38db490-d61c-443f-a65b-d21fe96a405b
2008-03-11Add a newline on a logqwell1-1/+1
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@107712 f38db490-d61c-443f-a65b-d21fe96a405b
2008-03-11Add an additional check for setting conference parameter when using the ↵file1-1/+1
marked user options. It was possible for it to return to a no listen/no talk state if a masquerade happened. (closes issue #12136) Reported by: aragon git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@107637 f38db490-d61c-443f-a65b-d21fe96a405b
2008-03-11Fix a minor spelling error.file1-1/+1
(closes issue #12183) Reported by: darrylc git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@107582 f38db490-d61c-443f-a65b-d21fe96a405b
2008-03-11backport a fix from trunkkpfleming1-6/+2
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@107472 f38db490-d61c-443f-a65b-d21fe96a405b
2008-03-11stop checking for mktime() in the configure script... we don't use it, and ↵kpfleming1-2/+2
the test is buggy under gcc 4.3 git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@107461 f38db490-d61c-443f-a65b-d21fe96a405b
2008-03-10Move where unanswered CDRs are dropped to the CDR core, not everything uses ↵file1-19/+0
app_dial. (closes issue #11516) Reported by: ys Patches: branch_1.4_cdr.diff uploaded by ys (license 281) Tested by: anest, jcapp, dartvader git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@107016 f38db490-d61c-443f-a65b-d21fe96a405b
2008-03-07Only start the SLA thread if SLA has actually been configured.russell1-1/+2
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@106895 f38db490-d61c-443f-a65b-d21fe96a405b
generated by cgit v1.2.3 (git 2.25.1) at 2024-09-22 06:59:27 +0000