Fix a variety of memory leaks
[asterisk/asterisk.git] / res / res_calendar.c
index 5ddcaae..8c34994 100644 (file)
@@ -374,6 +374,7 @@ static int calendar_is_busy(struct ast_calendar *cal)
 
 static enum ast_device_state calendarstate(const char *data)
 {
+       enum ast_device_state state;
        struct ast_calendar *cal;
 
        if (ast_strlen_zero(data) || (!(cal = find_calendar(data)))) {
@@ -381,10 +382,13 @@ static enum ast_device_state calendarstate(const char *data)
        }
 
        if (cal->tech->is_busy) {
-               return cal->tech->is_busy(cal) ? AST_DEVICE_INUSE : AST_DEVICE_NOT_INUSE;
+               state = cal->tech->is_busy(cal) ? AST_DEVICE_INUSE : AST_DEVICE_NOT_INUSE;
+       } else {
+               state = calendar_is_busy(cal) ? AST_DEVICE_INUSE : AST_DEVICE_NOT_INUSE;
        }
 
-       return calendar_is_busy(cal) ? AST_DEVICE_INUSE : AST_DEVICE_NOT_INUSE;
+       cal = unref_calendar(cal);
+       return state;
 }
 
 static struct ast_calendar *build_calendar(struct ast_config *cfg, const char *cat, const struct ast_calendar_tech *tech)
@@ -1060,6 +1064,7 @@ static int calendar_busy_exec(struct ast_channel *chan, const char *cmd, char *d
        }
 
        strcpy(buf, calendar_is_busy(cal) ? "1" : "0");
+       cal = unref_calendar(cal);
 
        return 0;
 }
@@ -1214,6 +1219,8 @@ 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);
+                               cal = unref_calendar(cal);
+                               ao2_ref(events, -1);
                                ao2_iterator_destroy(&i);
                                return -1;
                        }
@@ -1231,6 +1238,8 @@ static int calendar_query_exec(struct ast_channel *chan, const char *cmd, char *
 
        if (!(eventlist_datastore = ast_datastore_alloc(&eventlist_datastore_info, buf))) {
                ast_log(LOG_ERROR, "Could not allocate datastore!\n");
+               cal = unref_calendar(cal);
+               ao2_ref(events, -1);
                return -1;
        }
 
@@ -1241,6 +1250,7 @@ static int calendar_query_exec(struct ast_channel *chan, const char *cmd, char *
        ast_channel_datastore_add(chan, eventlist_datastore);
        ast_channel_unlock(chan);
 
+       cal = unref_calendar(cal);
        return 0;
 }