[asterisk-commits] trunk r21365 - /trunk/pbx.c
asterisk-commits at lists.digium.com
asterisk-commits at lists.digium.com
Wed Apr 19 01:52:17 MST 2006
Author: rizzo
Date: Wed Apr 19 03:52:15 2006
New Revision: 21365
URL: http://svn.digium.com/view/asterisk?rev=21365&view=rev
Log:
replace repeated code to walk contexts with a function,
find_context_locked()
Modified:
trunk/pbx.c
Modified: trunk/pbx.c
URL: http://svn.digium.com/view/asterisk/trunk/pbx.c?rev=21365&r1=21364&r2=21365&view=diff
==============================================================================
--- trunk/pbx.c (original)
+++ trunk/pbx.c Wed Apr 19 03:52:15 2006
@@ -2386,6 +2386,25 @@
return oldval;
}
+/* lookup for a context with a given name,
+ * return with conlock held if found, NULL if not found
+ */
+static struct ast_context *find_context_locked(const char *context)
+{
+ struct ast_context *c = NULL;
+ if (ast_lock_contexts()) {
+ errno = EBUSY;
+ return NULL;
+ }
+ while ( (c = ast_walk_contexts(c)) ) {
+ if (!strcmp(ast_get_context_name(c), context))
+ return c;
+ }
+ ast_unlock_contexts();
+ errno = ENOENT;
+ return NULL;
+}
+
/*
* This function locks contexts list by &conlist, search for the right context
* structure, leave context list locked and call ast_context_remove_include2
@@ -2462,24 +2481,14 @@
*/
int ast_context_remove_switch(const char *context, const char *sw, const char *data, const char *registrar)
{
- struct ast_context *c = NULL;
int ret = -1; /* default error return */
-
- if (ast_lock_contexts())
- return -1;
-
- /* walk contexts and search for the right one ...*/
- while ( (c = ast_walk_contexts(c)) ) {
- /* we found one ... */
- if (!strcmp(ast_get_context_name(c), context)) {
- /* remove switch from this context ... */
- ret = ast_context_remove_switch2(c, sw, data, registrar);
- break;
- }
- }
-
- /* found or error */
- ast_unlock_contexts();
+ struct ast_context *c = find_context_locked(context);
+
+ if (c) {
+ /* remove switch from this context ... */
+ ret = ast_context_remove_switch2(c, sw, data, registrar);
+ ast_unlock_contexts();
+ }
return ret;
}
@@ -2494,29 +2503,27 @@
int ast_context_remove_switch2(struct ast_context *con, const char *sw, const char *data, const char *registrar)
{
struct ast_sw *i, *pi = NULL;
-
- if (ast_mutex_lock(&con->lock)) return -1;
+ int ret = -1;
+
+ if (ast_mutex_lock(&con->lock))
+ return -1;
/* walk switchs */
for (i = con->alts; i; pi = i, i = i->next) {
- /* find our switch */
if (!strcmp(i->name, sw) && !strcmp(i->data, data) &&
(!registrar || !strcmp(i->registrar, registrar))) {
- /* remove from list */
+ /* found, remove from list */
if (pi)
pi->next = i->next;
else
con->alts = i->next;
- /* free switch and return */
- free(i);
- ast_mutex_unlock(&con->lock);
- return 0;
- }
- }
-
- /* we can't find the right switch */
+ free(i); /* free switch and return */
+ ret = 0;
+ break;
+ }
+ }
ast_mutex_unlock(&con->lock);
- return -1;
+ return ret;
}
/*
@@ -2526,24 +2533,13 @@
*/
int ast_context_remove_extension(const char *context, const char *extension, int priority, const char *registrar)
{
- struct ast_context *c = NULL;
int ret = -1; /* default error return */
-
- if (ast_lock_contexts())
- return -1;
-
- /* walk contexts ... */
- while ( (c = ast_walk_contexts(c)) ) {
- /* ... search for the right one ... */
- if (!strcmp(ast_get_context_name(c), context)) {
- /* ... remove extension ... */
- ret = ast_context_remove_extension2(c, extension, priority,
- registrar);
- break;
- }
- }
- /* found or error */
- ast_unlock_contexts();
+ struct ast_context *c = find_context_locked(context);
+
+ if (c) { /* ... remove extension ... */
+ ret = ast_context_remove_extension2(c, extension, priority, registrar);
+ ast_unlock_contexts();
+ }
return ret;
}
@@ -2712,6 +2708,10 @@
return 0;
}
+/*
+ * Append to the list. We don't have a tail pointer because we need
+ * to scan the list anyways to check for duplicates during insertion.
+ */
int ast_register_switch(struct ast_switch *sw)
{
struct ast_switch *tmp, *prev=NULL;
@@ -3582,28 +3582,14 @@
*/
int ast_context_add_include(const char *context, const char *include, const char *registrar)
{
- struct ast_context *c = NULL;
-
- if (ast_lock_contexts()) {
- errno = EBUSY;
- return -1;
- }
-
- /* walk contexts ... */
- while ( (c = ast_walk_contexts(c)) ) {
- /* ... search for the right one ... */
- if (!strcmp(ast_get_context_name(c), context)) {
- int ret = ast_context_add_include2(c, include, registrar);
- /* ... unlock contexts list and return */
- ast_unlock_contexts();
- return ret;
- }
- }
-
- /* we can't find the right context */
- ast_unlock_contexts();
- errno = ENOENT;
- return -1;
+ int ret = -1;
+ struct ast_context *c = find_context_locked(context);
+
+ if (c) {
+ ret = ast_context_add_include2(c, include, registrar);
+ ast_unlock_contexts();
+ }
+ return ret;
}
/*! \brief Helper for get_range.
@@ -4035,24 +4021,14 @@
*/
int ast_context_remove_ignorepat(const char *context, const char *ignorepat, const char *registrar)
{
- struct ast_context *c = NULL;
-
- if (ast_lock_contexts()) {
- errno = EBUSY;
- return -1;
- }
-
- while ( (c = ast_walk_contexts(c)) ) {
- if (!strcmp(ast_get_context_name(c), context)) {
- int ret = ast_context_remove_ignorepat2(c, ignorepat, registrar);
- ast_unlock_contexts();
- return ret;
- }
- }
-
- ast_unlock_contexts();
- errno = ENOENT;
- return -1;
+ int ret = -1;
+ struct ast_context *c = find_context_locked(context);
+
+ if (c) {
+ ret = ast_context_remove_ignorepat2(c, ignorepat, registrar);
+ ast_unlock_contexts();
+ }
+ return ret;
}
int ast_context_remove_ignorepat2(struct ast_context *con, const char *ignorepat, const char *registrar)
@@ -4089,26 +4065,17 @@
* EBUSY - can't lock
* ENOENT - there is no existence of context
*/
-int ast_context_add_ignorepat(const char *con, const char *value, const char *registrar)
-{
- struct ast_context *c = NULL;
-
- if (ast_lock_contexts()) {
- errno = EBUSY;
- return -1;
- }
-
- while ( (c = ast_walk_contexts(c)) ) {
- if (!strcmp(ast_get_context_name(c), con)) {
- int ret = ast_context_add_ignorepat2(c, value, registrar);
- ast_unlock_contexts();
- return ret;
- }
- }
-
- ast_unlock_contexts();
- errno = ENOENT;
- return -1;
+int ast_context_add_ignorepat(const char *context, const char *value, const char *registrar)
+{
+ int ret = -1;
+ struct ast_context *c = find_context_locked(context);
+
+ if (c) {
+ ret = ast_context_add_ignorepat2(c, value, registrar);
+ ast_unlock_contexts();
+ }
+
+ return ret;
}
int ast_context_add_ignorepat2(struct ast_context *con, const char *value, const char *registrar)
@@ -4164,28 +4131,20 @@
* ENOENT - no existence of context
*
*/
-int ast_add_extension(const char *context, int replace, const char *extension, int priority, const char *label, const char *callerid,
+int ast_add_extension(const char *context, int replace, const char *extension,
+ int priority, const char *label, const char *callerid,
const char *application, void *data, void (*datad)(void *), const char *registrar)
{
- struct ast_context *c = NULL;
-
- if (ast_lock_contexts()) {
- errno = EBUSY;
- return -1;
- }
-
- while ( (c = ast_walk_contexts(c)) ) {
- if (!strcmp(context, ast_get_context_name(c))) {
- int ret = ast_add_extension2(c, replace, extension, priority, label, callerid,
- application, data, datad, registrar);
- ast_unlock_contexts();
- return ret;
- }
- }
-
- ast_unlock_contexts();
- errno = ENOENT;
- return -1;
+ int ret = -1;
+ struct ast_context *c = find_context_locked(context);
+
+ if (c) {
+ ret = ast_add_extension2(c, replace, extension, priority, label, callerid,
+ application, data, datad, registrar);
+ ast_unlock_contexts();
+ }
+
+ return ret;
}
int ast_explicit_goto(struct ast_channel *chan, const char *context, const char *exten, int priority)
@@ -4269,6 +4228,7 @@
return res;
}
+/*! \brief copy a string skipping whitespace */
static int ext_strncpy(char *dst, const char *src, int len)
{
int count=0;
More information about the asterisk-commits
mailing list