Fix memory leak in 'manager show event' when command entered incorrectly
authorMatthew Jordan <mjordan@digium.com>
Thu, 6 Dec 2012 14:26:13 +0000 (14:26 +0000)
committerMatthew Jordan <mjordan@digium.com>
Thu, 6 Dec 2012 14:26:13 +0000 (14:26 +0000)
When the CLI command 'manager show event' was run incorrectly and its usage
instructions returned, a reference to the event container was leaked. This
would prevent the container from being reclaimed when Asterisk exits. We now
properly decrement the count on the ao2 object using the nifty RAII_VAR macro.

Thanks to Russell for helping me stumble on this, and Terry for writing that
ridiculously helpful macro.
........

Merged revisions 377319 from http://svn.asterisk.org/svn/asterisk/branches/11

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

main/manager.c

index 011ae10..ba5beb4 100644 (file)
@@ -7149,7 +7149,7 @@ static char *handle_manager_show_events(struct ast_cli_entry *e, int cmd, struct
 
 static char *handle_manager_show_event(struct ast_cli_entry *e, int cmd, struct ast_cli_args *a)
 {
-       struct ao2_container *events;
+       RAII_VAR(struct ao2_container *, events, NULL, ao2_cleanup);
        struct ao2_iterator it_events;
        struct ast_xml_doc_item *item, *temp;
        int length;
@@ -7184,7 +7184,6 @@ static char *handle_manager_show_event(struct ast_cli_entry *e, int cmd, struct
                        ao2_ref(item, -1);
                }
                ao2_iterator_destroy(&it_events);
-               ao2_ref(events, -1);
                return match;
        }
 
@@ -7194,7 +7193,6 @@ static char *handle_manager_show_event(struct ast_cli_entry *e, int cmd, struct
 
        if (!(item = ao2_find(events, a->argv[3], OBJ_KEY))) {
                ast_cli(a->fd, "Could not find event '%s'\n", a->argv[3]);
-               ao2_ref(events, -1);
                return CLI_SUCCESS;
        }
 
@@ -7234,7 +7232,6 @@ static char *handle_manager_show_event(struct ast_cli_entry *e, int cmd, struct
        }
 
        ao2_ref(item, -1);
-       ao2_ref(events, -1);
        return CLI_SUCCESS;
 }