Asterisk
/
asterisk
mirror of https://github.com/asterisk/asterisk
1
0
Fork 0
Code Issues Releases Wiki Activity

PickupChan: Add ability to specify channel uniqueids as well as channel names.

Browse Source 
* Made PickupChan() search by channel uniqueids if the search could not
find a channel by name.

* Ensured PickupChan() never considers the picking channel for pickup.

* Made PickupChan() option p use a common search by name routine.  The
original search was erroneously case sensitive.

(issue AFS-42)

Review: https://reviewboard.asterisk.org/r/3017/


git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@403013 65c4cc65-6c06-0410-ace0-fbb531ad65f3
changes/97/197/1
Richard Mudgett 12 years ago
parent a60764d61e
commit 18c2cfa7b7
2 changed files with 94 additions and 36 deletions
Show Stats Download Patch File Download Diff File

4
CHANGES
Unescape View File

@ -38,6 +38,10 @@ Directory
USEREXIT user pressed '#' from the selection prompt to exit USEREXIT user pressed '#' from the selection prompt to exit
FAILED directory failed in a way that wasn't accounted for. Dang. FAILED directory failed in a way that wasn't accounted for. Dang.
PickupChan
--------------------------
* PickupChan now accepts channel uniqueids of channels to pickup.
Say Say
-------------------------- --------------------------
* If a channel variable SAY_DTMF_INTERRUPT is present on a channel and set * If a channel variable SAY_DTMF_INTERRUPT is present on a channel and set

126
apps/app_directed_pickup.c
Unescape View File

@ -97,20 +97,29 @@ ASTERISK_FILE_VERSION(__FILE__, "$Revision$")
Pickup a ringing channel. Pickup a ringing channel.
</synopsis> </synopsis>
<syntax > <syntax >
<parameter name="Technology/Resource" argsep="&amp;" required="true"> <parameter name="channel" argsep="&amp;" required="true">
<argument name="Technology/Resource" required="true" /> <argument name="channel" required="true" />
<argument name="Technology2/Resource2" required="false" multiple="true" /> <argument name="channel2" required="false" multiple="true" />
<para>List of channel names or channel uniqueids to pickup if ringing.
For example, a channel name could be <literal>SIP/bob</literal> or
<literal>SIP/bob-00000000</literal> to find
<literal>SIP/bob-00000000</literal>.
</para>
</parameter> </parameter>
<parameter name="options" required="false"> <parameter name="options" required="false">
<optionlist> <optionlist>
<option name="p"> <option name="p">
<para>All channel names listed specify partial names. Used when find channel by callid.</para> <para>Supplied channel names are prefixes. For example,
<literal>SIP/bob</literal> will match
<literal>SIP/bob-00000000</literal> and
<literal>SIP/bobby-00000000</literal>.
</para>
</option> </option>
</optionlist> </optionlist>
</parameter> </parameter>
</syntax> </syntax>
<description> <description>
<para>This will pickup a specified <replaceable>channel</replaceable> if ringing.</para> <para>Pickup a specified <replaceable>channel</replaceable> if ringing.</para>
</description> </description>
</application> </application>
***/ ***/
@ -119,17 +128,48 @@ static const char app[] = "Pickup";
static const char app2[] = "PickupChan"; static const char app2[] = "PickupChan";
struct pickup_by_name_args { struct pickup_by_name_args {
/*! Channel attempting to pickup a call. */
struct ast_channel *chan;
/*! Channel uniqueid or partial channel name to match. */
const char *name; const char *name;
/*! Length of partial channel name to match. */
size_t len; size_t len;
}; };
static int pickup_by_name_cb(void *obj, void *arg, void *data, int flags) static int find_by_name(void *obj, void *arg, void *data, int flags)
{ {
struct ast_channel *target = obj;/*!< Potential pickup target */ struct ast_channel *target = obj;/*!< Potential pickup target */
struct pickup_by_name_args *args = data; struct pickup_by_name_args *args = data;
if (args->chan == target) {
/* The channel attempting to pickup a call cannot pickup itself. */
return 0;
}
ast_channel_lock(target);
if (!strncasecmp(ast_channel_name(target), args->name, args->len)
&& ast_can_pickup(target)) {
/* Return with the channel still locked on purpose */
return CMP_MATCH | CMP_STOP;
}
ast_channel_unlock(target);
return 0;
}
static int find_by_uniqueid(void *obj, void *arg, void *data, int flags)
{
struct ast_channel *target = obj;/*!< Potential pickup target */
struct pickup_by_name_args *args = data;
if (args->chan == target) {
/* The channel attempting to pickup a call cannot pickup itself. */
return 0;
}
ast_channel_lock(target); ast_channel_lock(target);
if (!strncasecmp(ast_channel_name(target), args->name, args->len) && ast_can_pickup(target)) { if (!strcasecmp(ast_channel_uniqueid(target), args->name)
&& ast_can_pickup(target)) {
/* Return with the channel still locked on purpose */ /* Return with the channel still locked on purpose */
return CMP_MATCH | CMP_STOP; return CMP_MATCH | CMP_STOP;
} }
@ -139,45 +179,55 @@ static int pickup_by_name_cb(void *obj, void *arg, void *data, int flags)
} }
/*! \brief Helper Function to walk through ALL channels checking NAME and STATE */ /*! \brief Helper Function to walk through ALL channels checking NAME and STATE */
static struct ast_channel *my_ast_get_channel_by_name_locked(const char *channame) static struct ast_channel *find_by_channel(struct ast_channel *chan, const char *channame)
{ {
struct ast_channel *target;
char *chkchan; char *chkchan;
struct pickup_by_name_args pickup_args; struct pickup_by_name_args pickup_args;
/* Check if channel name contains a '-'. pickup_args.chan = chan;
* In this case the channel name will be interpreted as full channel name.
*/
if (strchr(channame, '-')) { if (strchr(channame, '-')) {
/* check full channel name */ /*
* Use the given channel name string as-is. This allows a full channel
* name with a typical sequence number to be used as well as still
* allowing the odd partial channel name that has a '-' in it to still
* work, i.e. Local/bob@en-phone.
*/
pickup_args.len = strlen(channame); pickup_args.len = strlen(channame);
pickup_args.name = channame; pickup_args.name = channame;
} else { } else {
/* need to append a '-' for the comparison so we check full channel name, /*
* i.e SIP/hgc- , use a temporary variable so original stays the same for * Append a '-' for the comparison so we check the channel name less
* debugging. * a sequence number, i.e Find SIP/bob- and not SIP/bobby.
*/ */
pickup_args.len = strlen(channame) + 1; pickup_args.len = strlen(channame) + 1;
chkchan = ast_alloca(pickup_args.len + 1); chkchan = ast_alloca(pickup_args.len + 1);
strcpy(chkchan, channame); strcpy(chkchan, channame);/* Safe */
strcat(chkchan, "-"); strcat(chkchan, "-");
pickup_args.name = chkchan; pickup_args.name = chkchan;
} }
target = ast_channel_callback(find_by_name, NULL, &pickup_args, 0);
if (target) {
return target;
}
return ast_channel_callback(pickup_by_name_cb, NULL, &pickup_args, 0); /* Now try a search for uniqueid. */
pickup_args.name = channame;
pickup_args.len = 0;
return ast_channel_callback(find_by_uniqueid, NULL, &pickup_args, 0);
} }
/*! \brief Attempt to pick up named channel, does not use context */ /*! \brief Attempt to pick up named channel. */
static int pickup_by_channel(struct ast_channel *chan, char *pickup) static int pickup_by_channel(struct ast_channel *chan, const char *name)
{ {
int res = -1; int res = -1;
struct ast_channel *target;/*!< Potential pickup target */ struct ast_channel *target;/*!< Potential pickup target */
target = my_ast_get_channel_by_name_locked(pickup); /* The found channel is already locked. */
target = find_by_channel(chan, name);
if (target) { if (target) {
/* Just check that we are not picking up the SAME as target. (i.e. ourself) */ res = ast_do_pickup(chan, target);
if (chan != target) {
res = ast_do_pickup(chan, target);
}
ast_channel_unlock(target); ast_channel_unlock(target);
target = ast_channel_unref(target); target = ast_channel_unref(target);
} }
@ -322,21 +372,25 @@ static int pickup_exec(struct ast_channel *chan, const char *data)
} }
/* Find channel for pick up specified by partial channel name */ /* Find channel for pick up specified by partial channel name */
static int find_by_part(void *obj, void *arg, void *data, int flags) static struct ast_channel *find_by_part(struct ast_channel *chan, const char *part)
{ {
struct ast_channel *target = obj;/*!< Potential pickup target */ struct ast_channel *target;
const char *part = data; struct pickup_by_name_args pickup_args;
int len = strlen(part);
ast_channel_lock(target); pickup_args.chan = chan;
if (len <= strlen(ast_channel_name(target)) && !strncmp(ast_channel_name(target), part, len)
&& ast_can_pickup(target)) { /* Try a partial channel name search. */
/* Return with the channel still locked on purpose */ pickup_args.name = part;
return CMP_MATCH | CMP_STOP; pickup_args.len = strlen(part);
target = ast_channel_callback(find_by_name, NULL, &pickup_args, 0);
if (target) {
return target;
} }
ast_channel_unlock(target);
return 0; /* Now try a search for uniqueid. */
pickup_args.name = part;
pickup_args.len = 0;
return ast_channel_callback(find_by_uniqueid, NULL, &pickup_args, 0);
} }
/* Attempt to pick up specified by partial channel name */ /* Attempt to pick up specified by partial channel name */
@ -346,7 +400,7 @@ static int pickup_by_part(struct ast_channel *chan, const char *part)
int res = -1; int res = -1;
/* The found channel is already locked. */ /* The found channel is already locked. */
target = ast_channel_callback(find_by_part, NULL, (char *) part, 0); target = find_by_part(chan, part);
if (target) { if (target) {
res = ast_do_pickup(chan, target); res = ast_do_pickup(chan, target);
ast_channel_unlock(target); ast_channel_unlock(target);

Write Preview
Loading…
Cancel
Save