diff options
author | Kevin P. Fleming <kpfleming@digium.com> | 2009-10-06 01:24:24 +0000 |
---|---|---|
committer | Kevin P. Fleming <kpfleming@digium.com> | 2009-10-06 01:24:24 +0000 |
commit | 1c9fe00920c4d887328400bf43c3e16fc9d24c37 (patch) | |
tree | 25c2e24721eb7de30f085f43103b8aff55e28fa5 /res/res_calendar.c | |
parent | 20743ec07dbd45e4a963680cf5d72d8d02b43193 (diff) |
Recorded merge of revisions 222152 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4
........
r222152 | kpfleming | 2009-10-05 20:16:36 -0500 (Mon, 05 Oct 2009) | 20 lines
Fix ao2_iterator API to hold references to containers being iterated.
See Mantis issue for details of what prompted this change.
Additional notes:
This patch changes the ao2_iterator API in two ways: F_AO2I_DONTLOCK
has become an enum instead of a macro, with a name that fits our
naming policy; also, it is now necessary to call
ao2_iterator_destroy() on any iterator that has been
created. Currently this only releases the reference to the container
being iterated, but in the future this could also release other
resources used by the iterator, if the iterator implementation changes
to use additional resources.
(closes issue #15987)
Reported by: kpfleming
Review: https://reviewboard.asterisk.org/r/383/
........
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@222176 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'res/res_calendar.c')
-rw-r--r-- | res/res_calendar.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/res/res_calendar.c b/res/res_calendar.c index e6f8d53f3..e916ac0f0 100644 --- a/res/res_calendar.c +++ b/res/res_calendar.c @@ -1043,12 +1043,14 @@ static int calendar_query_exec(struct ast_channel *chan, const char *cmd, char * ast_debug(10, "%s (%ld - %ld) overlapped with (%ld - %ld)\n", event->summary, (long) event->start, (long) event->end, (long) start, (long) end); if (add_event_to_list(events, event, start, end) < 0) { event = ast_calendar_unref_event(event); + ao2_iterator_destroy(&i); return -1; } } event = ast_calendar_unref_event(event); } + ao2_iterator_destroy(&i); ast_channel_lock(chan); do { @@ -1292,6 +1294,7 @@ static char *handle_show_calendars(struct ast_cli_entry *e, int cmd, struct ast_ ast_cli(a->fd, FORMAT, cal->name, cal->tech->type, calendar_is_busy(cal) ? "busy" : "free"); cal = unref_calendar(cal); } + ao2_iterator_destroy(&i); return CLI_SUCCESS; #undef FORMAT @@ -1345,6 +1348,7 @@ static char *handle_show_calendar(struct ast_cli_entry *e, int cmd, struct ast_c } cal = unref_calendar(cal); } + ao2_iterator_destroy(&i); return ret; } @@ -1384,6 +1388,7 @@ static char *handle_show_calendar(struct ast_cli_entry *e, int cmd, struct ast_c event = ast_calendar_unref_event(event); } + ao2_iterator_destroy(&i); cal = unref_calendar(cal); return CLI_SUCCESS; #undef FORMAT |