Fix some refcounting in app_queue.c and change the
authorMark Michelson <mmichelson@digium.com>
Fri, 14 Nov 2008 16:53:38 +0000 (16:53 +0000)
committerMark Michelson <mmichelson@digium.com>
Fri, 14 Nov 2008 16:53:38 +0000 (16:53 +0000)
commitcf6c66de65b344aab18f032feb4aa851fde3f3ad
tree9c95c9f6a5849fa85869f98ea8021b673b948d7f
parentcb90752b0d2de3d1c9711bf4eb7095f3215f3721
Fix some refcounting in app_queue.c and change the
hashing used by app_queue.c to be case-insensitive.
This is accomplished by adding a new case-insensitive
hashing function.

This was necessary to prevent bad refcount errors
(and potential crashes) which would occur due to the
fact that queues were initially read from the config
file in a case-sensitive manner. Then, when a user
issued a CLI command or manager action, we allowed
for case-insensitive input and used that input to
directly try to find the queue in the hash table. The result
was either that we could not find a queue that was input or
worse, we would end up hashing to a completely bogus value
based on the input.

This commit resolves the problem presented in
issue #13703. However, that issue was reported against
1.6.0. Since this fix introduces a behavior change, I am
electing to not place this same fix in to the 1.6.0 or 1.6.1
branches, and instead will opt for a change which does not
change behavior.

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@156883 65c4cc65-6c06-0410-ace0-fbb531ad65f3
UPGRADE.txt
apps/app_queue.c
include/asterisk/strings.h