diff options
| author | tilghman <tilghman@f38db490-d61c-443f-a65b-d21fe96a405b> | 2009-08-07 18:16:28 +0000 |
|---|---|---|
| committer | tilghman <tilghman@f38db490-d61c-443f-a65b-d21fe96a405b> | 2009-08-07 18:16:28 +0000 |
| commit | 0a9056ed19f6149addca7325e2d5d23caed1ae53 (patch) | |
| tree | 27d8a6f088e6ba81e7f226045422de680c7f0817 /apps | |
| parent | d36a071288f50a6f40a57964e128aaef22d76bb4 (diff) | |
QUEUE_MEMBER_LIST _really_ wants the interface name, not the membername.
This is a partial revert of revision 82590, which was an attempted cleanup,
but in reality, it broke QUEUE_MEMBER_LIST, which has always been intended
as a method by which component interfaces could be queried from the queue.
Membername isn't useful here, because that field cannot be used to obtain
further information about the member. See the documentation on
QUEUE_MEMBER_LIST, RemoveQueueMember, QUEUE_MEMBER_PENALTY, and the various
AMI commands which take a member argument for further justification.
(closes issue #15664)
Reported by: rain
Patches:
app_queue-queue_member_list.diff uploaded by rain (license 327)
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@211038 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'apps')
| -rw-r--r-- | apps/app_queue.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/apps/app_queue.c b/apps/app_queue.c index 92bc97852..4d2eb6204 100644 --- a/apps/app_queue.c +++ b/apps/app_queue.c @@ -4297,8 +4297,8 @@ static int queue_function_queuememberlist(struct ast_channel *chan, char *cmd, c strncat(buf + buflen, ",", len - buflen - 1); buflen++; } - strncat(buf + buflen, m->membername, len - buflen - 1); - buflen += strlen(m->membername); + strncat(buf + buflen, m->interface, len - buflen - 1); + buflen += strlen(m->interface); /* Safeguard against overflow (negative length) */ if (buflen >= len - 2) { ao2_ref(m, -1); |