Fixed startup race condition which caused occasional stasis_mwi_state_type assertions.
authorDavid M. Lee <dlee@digium.com>
Wed, 22 May 2013 19:15:16 +0000 (19:15 +0000)
committerDavid M. Lee <dlee@digium.com>
Wed, 22 May 2013 19:15:16 +0000 (19:15 +0000)
The caching topic (which refers to the message type) was created before the
message type. If the initial subscription message gets processed before
the type can be initialized, the assertion about using an uninitialized type
fires.

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

main/app.c

index 0e8a68f..1473c06 100644 (file)
@@ -2735,6 +2735,9 @@ static void app_exit(void)
 
 int app_init(void)
 {
+       if (STASIS_MESSAGE_TYPE_INIT(stasis_mwi_state_type) != 0) {
+               return -1;
+       }
        mwi_topic_all = stasis_topic_create("stasis_mwi_topic");
        if (!mwi_topic_all) {
                return -1;
@@ -2743,9 +2746,6 @@ int app_init(void)
        if (!mwi_topic_cached) {
                return -1;
        }
-       if (STASIS_MESSAGE_TYPE_INIT(stasis_mwi_state_type) != 0) {
-               return -1;
-       }
        mwi_topic_pool = stasis_topic_pool_create(mwi_topic_all);
        if (!mwi_topic_pool) {
                return -1;