diff options
author | Joshua Colp <jcolp@digium.com> | 2016-03-29 09:03:55 -0500 |
---|---|---|
committer | Gerrit Code Review <gerrit2@gerrit.digium.api> | 2016-03-29 09:03:55 -0500 |
commit | 6ce25bd62a0f31aa616272585e4f1ea1f2b86fd9 (patch) | |
tree | ec7f54a4f648bd507e98295f4ebfb4fa036abaf5 /res/res_parking.c | |
parent | 77a9272431296772f5930301175bc0832076ac3e (diff) | |
parent | 13e75ee04fae19528be83093a7dc6bb4ff600aa2 (diff) |
Merge "res_parking: Misc fixes."
Diffstat (limited to 'res/res_parking.c')
-rw-r--r-- | res/res_parking.c | 22 |
1 files changed, 5 insertions, 17 deletions
diff --git a/res/res_parking.c b/res/res_parking.c index 14b0d7ab8..78ae481ba 100644 --- a/res/res_parking.c +++ b/res/res_parking.c @@ -782,31 +782,21 @@ int parking_lot_cfg_create_extensions(struct parking_lot_cfg *lot_cfg) } /* We need the contexts list locked to safely be able to both read and lock the specific context within */ - if (ast_wrlock_contexts()) { - ast_log(LOG_ERROR, "Failed to lock the contexts list.\n"); - return -1; - } + ast_wrlock_contexts(); if (!(lot_context = ast_context_find_or_create(NULL, NULL, lot_cfg->parking_con, parkext_registrar_pointer))) { ast_log(LOG_ERROR, "Parking lot '%s' -- Needs a context '%s' which does not exist and Asterisk was unable to create\n", lot_cfg->name, lot_cfg->parking_con); - if (ast_unlock_contexts()) { - ast_assert(0); - } + ast_unlock_contexts(); return -1; } /* Once we know what context we will be modifying, we need to write lock it because we will be reading extensions * and we don't want something else to destroy them while we are looking at them. */ - if (ast_wrlock_context(lot_context)) { - ast_log(LOG_ERROR, "failed to obtain write lock on context\n"); - return -1; - } + ast_wrlock_context(lot_context); - if (ast_unlock_contexts()) { - ast_assert(0); - } + ast_unlock_contexts(); /* Handle generation/confirmation for the Park extension */ if ((existing_exten = pbx_find_extension(NULL, NULL, &find_info, lot_cfg->parking_con, lot_cfg->parkext, 1, NULL, NULL, E_MATCH))) { @@ -874,9 +864,7 @@ int parking_lot_cfg_create_extensions(struct parking_lot_cfg *lot_cfg) } } - if (ast_unlock_context(lot_context)) { - ast_assert(0); - } + ast_unlock_context(lot_context); return 0; } |