diff --git a/apps/app_dial.c b/apps/app_dial.c index 490fa34145..abd6b47270 100644 --- a/apps/app_dial.c +++ b/apps/app_dial.c @@ -2098,7 +2098,7 @@ static int unload_module(void) res |= ast_unregister_application(rapp); if ((con = ast_context_find("app_dial_gosub_virtual_context"))) { - ast_context_remove_extension2(con, "s", 1, NULL); + ast_context_remove_extension2(con, "s", 1, NULL, 0); ast_context_destroy(con, "app_dial"); /* leave nothing behind */ } diff --git a/apps/app_queue.c b/apps/app_queue.c index 5e7289ae06..6d40c3976e 100644 --- a/apps/app_queue.c +++ b/apps/app_queue.c @@ -6366,7 +6366,7 @@ static int unload_module(void) ast_event_unsubscribe(device_state_sub); if ((con = ast_context_find("app_queue_gosub_virtual_context"))) { - ast_context_remove_extension2(con, "s", 1, NULL); + ast_context_remove_extension2(con, "s", 1, NULL, 0); ast_context_destroy(con, "app_queue"); /* leave no trace */ } diff --git a/apps/app_stack.c b/apps/app_stack.c index 562be8ac9c..669f7f4e76 100644 --- a/apps/app_stack.c +++ b/apps/app_stack.c @@ -496,7 +496,7 @@ static int unload_module(void) ast_agi_unregister(ast_module_info->self, &gosub_agi_command); if ((con = ast_context_find("app_stack_gosub_virtual_context"))) { - ast_context_remove_extension2(con, "s", 1, NULL); + ast_context_remove_extension2(con, "s", 1, NULL, 0); ast_context_destroy(con, "app_stack"); /* leave nothing behind */ } } diff --git a/include/asterisk/pbx.h b/include/asterisk/pbx.h index 3e3669afb2..87a594c599 100644 --- a/include/asterisk/pbx.h +++ b/include/asterisk/pbx.h @@ -600,7 +600,7 @@ int ast_context_remove_extension(const char *context, const char *extension, int const char *registrar); int ast_context_remove_extension2(struct ast_context *con, const char *extension, - int priority, const char *registrar); + int priority, const char *registrar, int already_locked); /*! * \brief Add an ignorepat diff --git a/main/features.c b/main/features.c index 06b0a3ca64..ccd182d215 100644 --- a/main/features.c +++ b/main/features.c @@ -2465,7 +2465,7 @@ int manage_parkinglot(struct ast_parkinglot *curlot, fd_set *rfds, fd_set *efds, /* And take them out of the parking lot */ con = ast_context_find(pu->parkinglot->parking_con); if (con) { - if (ast_context_remove_extension2(con, pu->parkingexten, 1, NULL)) + if (ast_context_remove_extension2(con, pu->parkingexten, 1, NULL, 0)) ast_log(LOG_WARNING, "Whoa, failed to remove the parking extension!\n"); else notify_metermaids(pu->parkingexten, curlot->parking_con, AST_DEVICE_NOT_INUSE); @@ -2500,7 +2500,7 @@ int manage_parkinglot(struct ast_parkinglot *curlot, fd_set *rfds, fd_set *efds, /* And take them out of the parking lot */ con = ast_context_find(curlot->parking_con); if (con) { - if (ast_context_remove_extension2(con, pu->parkingexten, 1, NULL)) + if (ast_context_remove_extension2(con, pu->parkingexten, 1, NULL, 0)) ast_log(LOG_WARNING, "Whoa, failed to remove the extension!\n"); else notify_metermaids(pu->parkingexten, curlot->parking_con, AST_DEVICE_NOT_INUSE); @@ -2721,7 +2721,7 @@ static int park_exec_full(struct ast_channel *chan, void *data, struct ast_parki peer = pu->chan; con = ast_context_find(parkinglot->parking_con); if (con) { - if (ast_context_remove_extension2(con, pu->parkingexten, 1, NULL)) + if (ast_context_remove_extension2(con, pu->parkingexten, 1, NULL, 0)) ast_log(LOG_WARNING, "Whoa, failed to remove the extension!\n"); else notify_metermaids(pu->parkingexten, parkinglot->parking_con, AST_DEVICE_NOT_INUSE); @@ -3284,7 +3284,7 @@ static int load_config(void) /* Remove the old parking extension */ if (!ast_strlen_zero(old_parking_con) && (con = ast_context_find(old_parking_con))) { - if(ast_context_remove_extension2(con, old_parking_ext, 1, registrar)) + if(ast_context_remove_extension2(con, old_parking_ext, 1, registrar, 0)) notify_metermaids(old_parking_ext, old_parking_con, AST_DEVICE_NOT_INUSE); ast_debug(1, "Removed old parking extension %s@%s\n", old_parking_ext, old_parking_con); } diff --git a/main/pbx.c b/main/pbx.c index 6bb8a14f64..9654bd88fb 100644 --- a/main/pbx.c +++ b/main/pbx.c @@ -4022,7 +4022,7 @@ int ast_context_remove_extension(const char *context, const char *extension, int struct ast_context *c = find_context_locked(context); if (c) { /* ... remove extension ... */ - ret = ast_context_remove_extension2(c, extension, priority, registrar); + ret = ast_context_remove_extension2(c, extension, priority, registrar, 1); ast_unlock_contexts(); } return ret; @@ -4038,14 +4038,15 @@ int ast_context_remove_extension(const char *context, const char *extension, int * it. * */ -int ast_context_remove_extension2(struct ast_context *con, const char *extension, int priority, const char *registrar) +int ast_context_remove_extension2(struct ast_context *con, const char *extension, int priority, const char *registrar, int already_locked) { struct ast_exten *exten, *prev_exten = NULL; struct ast_exten *peer; struct ast_exten ex, *exten2, *exten3; char dummy_name[1024]; - ast_wrlock_context(con); + if (!already_locked) + ast_wrlock_context(con); /* Handle this is in the new world */ @@ -4132,7 +4133,8 @@ int ast_context_remove_extension2(struct ast_context *con, const char *extension } if (!exten) { /* we can't find right extension */ - ast_unlock_context(con); + if (!already_locked) + ast_unlock_context(con); return -1; } @@ -4159,7 +4161,8 @@ int ast_context_remove_extension2(struct ast_context *con, const char *extension break; /* found our priority */ } if (!peer) { /* not found */ - ast_unlock_context(con); + if (!already_locked) + ast_unlock_context(con); return -1; } /* we are first priority extension? */ @@ -4192,7 +4195,8 @@ int ast_context_remove_extension2(struct ast_context *con, const char *extension destroy_exten(peer); /* XXX should we return -1 ? */ } - ast_unlock_context(con); + if (!already_locked) + ast_unlock_context(con); return 0; } @@ -7177,7 +7181,7 @@ void __ast_context_destroy(struct ast_context *list, struct ast_hashtab *context ast_verb(3, "Remove %s/%s/%d, registrar=%s; con=%s(%p); con->root=%p\n", tmp->name, prio_item->exten, prio_item->priority, registrar, con? con->name : "", con, con? con->root_table: NULL); - ast_context_remove_extension2(tmp, prio_item->exten, prio_item->priority, registrar); + ast_context_remove_extension2(tmp, prio_item->exten, prio_item->priority, registrar, 1); } ast_hashtab_end_traversal(prio_iter); }