Merged revisions 104596 via svnmerge from
authorJoshua Colp <jcolp@digium.com>
Wed, 27 Feb 2008 17:12:08 +0000 (17:12 +0000)
committerJoshua Colp <jcolp@digium.com>
Wed, 27 Feb 2008 17:12:08 +0000 (17:12 +0000)
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r104596 | file | 2008-02-27 13:07:33 -0400 (Wed, 27 Feb 2008) | 4 lines

Use the lock (which already existed, it just wasn't used) on the updaters list to protect the contents instead of the overall module list lock.
(closes issue #12080)
Reported by: ChaseVenters

........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@104597 65c4cc65-6c06-0410-ace0-fbb531ad65f3

main/loader.c

index 79a716e..94a806a 100644 (file)
@@ -930,10 +930,10 @@ void ast_update_use_count(void)
           resource has changed */
        struct loadupdate *m;
 
-       AST_LIST_LOCK(&module_list);
+       AST_LIST_LOCK(&updaters);
        AST_LIST_TRAVERSE(&updaters, m, entry)
                m->updater();
-       AST_LIST_UNLOCK(&module_list);
+       AST_LIST_UNLOCK(&updaters);
 }
 
 int ast_update_module_list(int (*modentry)(const char *module, const char *description, int usecnt, const char *like),
@@ -978,9 +978,9 @@ int ast_loader_register(int (*v)(void))
                return -1;
 
        tmp->updater = v;
-       AST_LIST_LOCK(&module_list);
+       AST_LIST_LOCK(&updaters);
        AST_LIST_INSERT_HEAD(&updaters, tmp, entry);
-       AST_LIST_UNLOCK(&module_list);
+       AST_LIST_UNLOCK(&updaters);
 
        return 0;
 }
@@ -989,7 +989,7 @@ int ast_loader_unregister(int (*v)(void))
 {
        struct loadupdate *cur;
 
-       AST_LIST_LOCK(&module_list);
+       AST_LIST_LOCK(&updaters);
        AST_LIST_TRAVERSE_SAFE_BEGIN(&updaters, cur, entry) {
                if (cur->updater == v)  {
                        AST_LIST_REMOVE_CURRENT(entry);
@@ -997,7 +997,7 @@ int ast_loader_unregister(int (*v)(void))
                }
        }
        AST_LIST_TRAVERSE_SAFE_END;
-       AST_LIST_UNLOCK(&module_list);
+       AST_LIST_UNLOCK(&updaters);
 
        return cur ? 0 : -1;
 }