diff options
author | Matthew Jordan <mjordan@digium.com> | 2012-01-04 22:23:28 +0000 |
---|---|---|
committer | Matthew Jordan <mjordan@digium.com> | 2012-01-04 22:23:28 +0000 |
commit | baa7f14aabf2beb5cc3da52ac10c2127da271812 (patch) | |
tree | 712a99c0d2f0eea0191355033d078357edd343ba /apps/confbridge/conf_config_parser.c | |
parent | 12e3f412b5de2c6ddbeaf24ed745875f5ad4cac0 (diff) |
Fix for ConfBridge config parser unlocking channel mutex too many times
When looking up a ConfBridge profile, the config parser would, if it
found a channel datastore on the channel requesting the bridge profile,
unlock the channel mutex twice. Since that's a little aggressive,
it now only unlocks it once.
(closes issue ASTERISK-19042)
Reported by: Matt Jordan
Tested by: Matt Jordan
Patches:
19042 uploaded by David Vossel (license 5628)
........
Merged revisions 349619 from http://svn.asterisk.org/svn/asterisk/branches/10
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@349634 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'apps/confbridge/conf_config_parser.c')
-rw-r--r-- | apps/confbridge/conf_config_parser.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/apps/confbridge/conf_config_parser.c b/apps/confbridge/conf_config_parser.c index 58bb7acc1..9bb8b2af9 100644 --- a/apps/confbridge/conf_config_parser.c +++ b/apps/confbridge/conf_config_parser.c @@ -1332,8 +1332,9 @@ const struct user_profile *conf_find_user_profile(struct ast_channel *chan, cons conf_user_profile_copy(result, &b_data->u_profile); return result; } + } else { + ast_channel_unlock(chan); } - ast_channel_unlock(chan); } if (ast_strlen_zero(user_profile_name)) { |