app_queue: Fix extension state subscriptions removed on dialplan reload
authorIvan Poddubny <ivan.poddubny@gmail.com>
Tue, 12 Dec 2017 21:38:01 +0000 (22:38 +0100)
committerIvan Poddubny <ivan.poddubny@gmail.com>
Tue, 12 Dec 2017 22:00:51 +0000 (23:00 +0100)
The approach with having a single global subscription to all extension
state changes has one issue: dynamically created hints don't have any
watchers and are therefore garbage collected on the first dialplan
reload.

This change creates a state subscription for every queue member with a
hint as state_interface, thus increasing the count of watches for
hints, so they are not destroyed prematurely anymore.

There are 2 side effects:
1. The state change callback in app_queue is not executed when
   there are no members referring to the extension.
2. The callback is called multiple times for the same hint if it's
   associated with more than one queue member.

Reported by: Steven T. Wheeler

ASTERISK-18411 #close

Change-Id: I4956af2136ea2a7f110ac9272eae5f6e676d8f89

apps/app_queue.c

index 801695f..a140847 100644 (file)
@@ -1575,6 +1575,7 @@ struct member {
        char state_exten[AST_MAX_EXTENSION]; /*!< Extension to get state from (if using hint) */
        char state_context[AST_MAX_CONTEXT]; /*!< Context to use when getting state (if using hint) */
        char state_interface[AST_CHANNEL_NAME]; /*!< Technology/Location from which to read devicestate changes */
+       int state_id;                        /*!< Extension state callback id (if using hint) */
        char membername[80];                 /*!< Member name to use in queue logs */
        int penalty;                         /*!< Are we a last resort? */
        int calls;                           /*!< Number of calls serviced by this member */
@@ -2632,12 +2633,21 @@ static int get_queue_member_status(struct member *cur)
        return ast_strlen_zero(cur->state_exten) ? ast_device_state(cur->state_interface) : extensionstate2devicestate(ast_extension_state(NULL, cur->state_context, cur->state_exten));
 }
 
+static void destroy_queue_member_cb(void *obj)
+{
+       struct member *mem = obj;
+
+       if (mem->state_id != -1) {
+               ast_extension_state_del(mem->state_id, extension_state_cb);
+       }
+}
+
 /*! \brief allocate space for new queue member and set fields based on parameters passed */
 static struct member *create_queue_member(const char *interface, const char *membername, int penalty, int paused, const char *state_interface, int ringinuse)
 {
        struct member *cur;
 
-       if ((cur = ao2_alloc(sizeof(*cur), NULL))) {
+       if ((cur = ao2_alloc(sizeof(*cur), destroy_queue_member_cb))) {
                cur->ringinuse = ringinuse;
                cur->penalty = penalty;
                cur->paused = paused;
@@ -2664,6 +2674,10 @@ static struct member *create_queue_member(const char *interface, const char *mem
 
                        ast_copy_string(cur->state_exten, exten, sizeof(cur->state_exten));
                        ast_copy_string(cur->state_context, S_OR(context, "default"), sizeof(cur->state_context));
+
+                       cur->state_id = ast_extension_state_add(cur->state_context, cur->state_exten, extension_state_cb, NULL);
+               } else {
+                       cur->state_id = -1;
                }
                cur->status = get_queue_member_status(cur);
        }
@@ -11084,8 +11098,6 @@ static int unload_module(void)
 
        device_state_sub = stasis_unsubscribe_and_join(device_state_sub);
 
-       ast_extension_state_del(0, extension_state_cb);
-
        ast_unload_realtime("queue_members");
        ao2_cleanup(queues);
        ao2_cleanup(pending_members);
@@ -11243,8 +11255,6 @@ static int load_module(void)
        err |= STASIS_MESSAGE_TYPE_INIT(queue_agent_dump_type);
        err |= STASIS_MESSAGE_TYPE_INIT(queue_agent_ringnoanswer_type);
 
-       ast_extension_state_add(NULL, NULL, extension_state_cb, NULL);
-
        if (err) {
                unload_module();
                return AST_MODULE_LOAD_DECLINE;