diff options
| author | Richard Mudgett <rmudgett@digium.com> | 2013-06-20 17:21:40 +0000 |
|---|---|---|
| committer | Richard Mudgett <rmudgett@digium.com> | 2013-06-20 17:21:40 +0000 |
| commit | cd40e179a9bb01f10175ac3cecc633d631f69286 (patch) | |
| tree | f8ff825633e0ee901f3f042544f98743cbdd682b /apps/confbridge/conf_config_parser.c | |
| parent | 33eb15a242694e47707b70065c86fad19035d5c8 (diff) | |
Fix potential bridge hook resource leak if the hook install fails.
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@392335 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'apps/confbridge/conf_config_parser.c')
| -rw-r--r-- | apps/confbridge/conf_config_parser.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/apps/confbridge/conf_config_parser.c b/apps/confbridge/conf_config_parser.c index 042968c59..35d96ae31 100644 --- a/apps/confbridge/conf_config_parser.c +++ b/apps/confbridge/conf_config_parser.c @@ -2161,8 +2161,10 @@ int conf_set_menu_to_user(const char *menu_name, struct confbridge_user *user) ao2_ref(menu, +1); pvt->menu = menu; - ast_bridge_dtmf_hook(&user->features, pvt->menu_entry.dtmf, menu_hook_callback, - pvt, menu_hook_destroy, 0); + if (ast_bridge_dtmf_hook(&user->features, pvt->menu_entry.dtmf, + menu_hook_callback, pvt, menu_hook_destroy, 0)) { + menu_hook_destroy(pvt); + } } ao2_unlock(menu); |