Merge "sounds: Add Swedish sounds to Makefile and XML"
authorMatt Jordan <mjordan@digium.com>
Fri, 8 May 2015 18:17:59 +0000 (13:17 -0500)
committerGerrit Code Review <gerrit2@gerrit.digium.api>
Fri, 8 May 2015 18:17:59 +0000 (13:17 -0500)
12 files changed:
.gitignore
Makefile
contrib/editors/asterisk.vim
doc/.gitignore
doc/asterisk-ng-doxygen.in [moved from contrib/asterisk-ng-doxygen with 100% similarity]
include/asterisk/res_pjsip_pubsub.h
include/asterisk/test.h
include/asterisk/vector.h
res/res_pjsip_exten_state.c
res/res_pjsip_pubsub.c
res/res_pjsip_pubsub.exports.in
tests/test_vector.c

index 4b19213..0281e10 100644 (file)
@@ -31,3 +31,6 @@ menuselect-tree
 *.pyc
 *.gcno
 *.gcda
+latex
+doxygen.log
+
index 3252f76..17a2551 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -429,6 +429,8 @@ _clean:
        rm -f doc/core-en_US.xml
        rm -f doc/full-en_US.xml
        rm -f doc/rest-api/*.wiki
+       rm -f doxygen.log
+       rm -rf latex
        rm -f rest-api-templates/*.pyc
        @$(MAKE) -C menuselect clean
        cp -f .cleancount .lastclean
@@ -445,6 +447,7 @@ distclean: $(SUBDIRS_DIST_CLEAN) _clean
        rm -f include/asterisk/autoconfig.h
        rm -f include/asterisk/buildopts.h
        rm -rf doc/api
+       rm -f doc/asterisk-ng-doxygen
        rm -f build_tools/menuselect-deps
 
 datafiles: _all doc/core-en_US.xml
@@ -763,7 +766,8 @@ webvmail:
        @echo " +-------------------------------------------+"
 
 progdocs:
-       # Note, Makefile conditionals must not be tabbed out. Wasted hours with that.
+# Note, Makefile conditionals must not be tabbed out. Wasted hours with that.
+       @cp doc/asterisk-ng-doxygen.in doc/asterisk-ng-doxygen
 ifeq ($(DOXYGEN),:)
        @echo "Doxygen is not installed.  Please install and re-run the configuration script."
 else
@@ -771,20 +775,20 @@ ifeq ($(DOT),:)
        @echo "DOT is not installed. Doxygen will not produce any diagrams. Please install and re-run the configuration script."
 else
        # Enable DOT
-       @echo "HAVE_DOT = YES" >> contrib/asterisk-ng-doxygen
+       @echo "HAVE_DOT = YES" >> doc/asterisk-ng-doxygen
 endif
        # Set Doxygen PROJECT_NUMBER variable
 ifneq ($(ASTERISKVERSION),UNKNOWN__and_probably_unsupported)
-       @echo "PROJECT_NUMBER = $(ASTERISKVERSION)" >> contrib/asterisk-ng-doxygen
+       @echo "PROJECT_NUMBER = $(ASTERISKVERSION)" >> doc/asterisk-ng-doxygen
 else
        echo "Asterisk Version is unknown, not configuring Doxygen PROJECT_NUMBER."
 endif
-       # Validate Doxygen Configuration
-       @doxygen -u contrib/asterisk-ng-doxygen
+       # Validate and auto-update local copy
+       @doxygen -u doc/asterisk-ng-doxygen
        # Run Doxygen
-       @doxygen contrib/asterisk-ng-doxygen
+       @doxygen doc/asterisk-ng-doxygen
        # Remove configuration backup file
-       @rm -f contrib/asterisk-ng-doxygen.bak
+       @rm -f doc/asterisk-ng-doxygen.bak
 endif
 
 install-logrotate:
index a96a836..dd53a70 100644 (file)
@@ -14,7 +14,8 @@ syn sync clear
 syn sync fromstart
 
 syn keyword     asteriskTodo            TODO contained
-syn match       asteriskComment         ";.*" contains=asteriskTodo
+syn match       asteriskComment         "\\\@<!;.*" contains=asteriskTodo
+syn region      asteriskBlockComment    start=/\\\@<!;---\@!/ end=/--;/ contains=asteriskBlockComment,asteriskTodo
 syn match       asteriskContext         "\[.\{-}\]"
 syn match       asteriskExten           "^\s*exten\s*=>\?\s*[^,]\+" contains=asteriskPattern
 syn match       asteriskExten           "^\s*\(register\|channel\|ignorepat\|include\|\(no\)\?load\)\s*=>\?"
@@ -67,6 +68,7 @@ if version >= 508 || !exists("did_conf_syntax_inits")
   endif
 
   HiLink        asteriskComment         Comment
+  HiLink        asteriskBlockComment    Comment
   HiLink        asteriskExten           String
   HiLink        asteriskContext         Preproc
   HiLink        asteriskPattern         Type
index e098ec0..3461c58 100644 (file)
@@ -1,2 +1,4 @@
 core-en_US.xml
-rest-api
\ No newline at end of file
+rest-api
+api
+asterisk-ng-doxygen
index d32b246..afa0d69 100644 (file)
@@ -406,6 +406,16 @@ void ast_sip_subscription_get_remote_uri(struct ast_sip_subscription *sub, char
 const char *ast_sip_subscription_get_resource_name(struct ast_sip_subscription *sub);
 
 /*!
+ * \brief Get whether the subscription has been terminated or not.
+ *
+ * \param sub The subscription.
+ * \retval 0 not terminated.
+ * \retval 1 terminated.
+ * \since 13.4.0
+ */
+int ast_sip_subscription_is_terminated(const struct ast_sip_subscription *sub);
+
+/*!
  * \brief Get a header value for a subscription.
  *
  * For notifiers, the headers of the inbound SUBSCRIBE that started the dialog
index 2ea8332..69f8837 100644 (file)
@@ -389,6 +389,28 @@ int __ast_test_status_update(const char *file, const char *func, int line, struc
                }                                                       \
        } while(0)
 
+/*!
+ * \brief Check a test condition, report error and goto cleanup label if failed.
+ *
+ * \since 13.4.0
+ *
+ * This macro evaluates \a condition. If the condition evaluates to true (non-zero),
+ * nothing happens. If it evaluates to false (zero), then the failure is printed
+ * using \ref ast_test_status_update, the variable \a rc_variable is set to AST_TEST_FAIL,
+ * and a goto to \a cleanup_label is executed.
+ *
+ * \param test Currently executing test
+ * \param condition Boolean condition to check.
+ * \param rc_variable Variable to receive AST_TEST_FAIL.
+ * \param cleanup_label The label to go to on failure.
+ */
+#define ast_test_validate_cleanup(test, condition, rc_variable, cleanup_label) ({ \
+       if (!(condition)) {     \
+               ast_test_status_update((test), "%s: %s\n", "Condition failed", #condition); \
+               rc_variable = AST_TEST_FAIL; \
+               goto cleanup_label; \
+       } \
+})
 
 #endif /* TEST_FRAMEWORK */
 #endif /* _AST_TEST_H */
index 23db8a5..255c30b 100644 (file)
 } while (0)
 
 /*!
+ * \brief Deallocates this vector pointer.
+ *
+ * If any code to free the elements of this vector need to be run, that should
+ * be done prior to this call.
+ *
+ * \param vec Pointer to a malloc'd vector structure.
+ */
+#define AST_VECTOR_PTR_FREE(vec) do { \
+       AST_VECTOR_FREE(vec); \
+       ast_free(vec); \
+} while (0)
+
+/*!
  * \brief Deallocates this locked vector
  *
  * If any code to free the elements of this vector need to be run, that should
 } while(0)
 
 /*!
+ * \brief Deallocates this locked vector pointer.
+ *
+ * If any code to free the elements of this vector need to be run, that should
+ * be done prior to this call.
+ *
+ * \param vec Pointer to a malloc'd vector structure.
+ */
+#define AST_VECTOR_RW_PTR_FREE(vec) do { \
+       AST_VECTOR_RW_FREE(vec); \
+       ast_free(vec); \
+} while(0)
+
+/*!
+ * \internal
+ */
+#define __make_room(idx, vec) ({ \
+       int res = 0;                                                            \
+       do {                                                                                                            \
+               if ((idx) >= (vec)->max) {                                                              \
+                       size_t new_max = ((idx) + 1) * 2;                               \
+                       typeof((vec)->elems) new_elems = ast_calloc(1,          \
+                               new_max * sizeof(*new_elems));                                  \
+                       if (new_elems) {                                                                        \
+                               memcpy(new_elems, (vec)->elems,                                 \
+                                       (vec)->current * sizeof(*new_elems));           \
+                               ast_free((vec)->elems);                                                 \
+                               (vec)->elems = new_elems;                                               \
+                               (vec)->max = new_max;                                                   \
+                       } else {                                                                                        \
+                               res = -1;                                                                               \
+                               break;                                                                                  \
+                       }                                                                                                       \
+               }                                                                                                               \
+       } while(0);                                                                                                     \
+       res;                                                                                                            \
+})
+
+/*!
  * \brief Append an element to a vector, growing the vector if needed.
  *
  * \param vec Vector to append to.
  * \return Non-zero on failure.
  */
 #define AST_VECTOR_APPEND(vec, elem) ({                                                \
-       int res = 0;                                                            \
-       do {                                                                    \
-               if ((vec)->current + 1 > (vec)->max) {                          \
-                       size_t new_max = (vec)->max ? 2 * (vec)->max : 1;       \
-                       typeof((vec)->elems) new_elems = ast_realloc(           \
-                               (vec)->elems, new_max * sizeof(*new_elems));    \
-                       if (new_elems) {                                        \
-                               (vec)->elems = new_elems;                       \
-                               (vec)->max = new_max;                           \
-                       } else {                                                \
-                               res = -1;                                       \
-                               break;                                          \
-                       }                                                       \
-               }                                                               \
+       int res = 0;                                                                                    \
+       do {                                                                                                    \
+               if (__make_room((vec)->current, vec) != 0) {            \
+                       res = -1;                                                                               \
+                       break;                                                                                  \
+               }                                                                                                       \
                (vec)->elems[(vec)->current++] = (elem);                        \
-       } while (0);                                                            \
-       res;                                                                    \
+       } while (0);                                                                                    \
+       res;                                                                                                    \
 })
 
 /*!
  * index means you can not use the UNORDERED assortment of macros. These macros alter the ordering
  * of the vector itself.
  */
-#define AST_VECTOR_REPLACE(vec, idx, elem) ({                                  \
-       int res = 0;                                                                                            \
-       do {                                                                                                            \
-               if (((idx) + 1) > (vec)->max) {                                                 \
-                       size_t new_max = ((idx) + 1) * 2;                                       \
-                       typeof((vec)->elems) new_elems = ast_calloc(1,          \
-                               new_max * sizeof(*new_elems));                                  \
-                       if (new_elems) {                                                                        \
-                               memcpy(new_elems, (vec)->elems,                                 \
-                                       (vec)->current * sizeof(*new_elems));           \
-                               ast_free((vec)->elems);                                                 \
-                               (vec)->elems = new_elems;                                               \
-                               (vec)->max = new_max;                                                   \
-                       } else {                                                                                        \
-                               res = -1;                                                                               \
-                               break;                                                                                  \
-                       }                                                                                                       \
-               }                                                                                                               \
-               (vec)->elems[(idx)] = (elem);                                                   \
-               if (((idx) + 1) > (vec)->current) {                                             \
-                       (vec)->current = (idx) + 1;                                                     \
-               }                                                                                                               \
-       } while(0);                                                                                                     \
-       res;                                                                                                            \
+#define AST_VECTOR_REPLACE(vec, idx, elem) ({          \
+       int res = 0;                                                                    \
+       do {                                                                                    \
+               if (__make_room((idx), vec) != 0) {                     \
+                       res = -1;                                                               \
+                       break;                                                                  \
+               }                                                                                       \
+               (vec)->elems[(idx)] = (elem);                           \
+               if (((idx) + 1) > (vec)->current) {                     \
+                       (vec)->current = (idx) + 1;                             \
+               }                                                                                       \
+       } while(0);                                                                             \
+       res;                                                                                    \
 })
 
 /*!
        int res = 0; \
        size_t __move; \
        do { \
-               if ((vec)->current + 1 > (vec)->max) { \
-                       size_t new_max = (vec)->max ? 2 * (vec)->max : 1; \
-                       typeof((vec)->elems) new_elems = ast_realloc( \
-                               (vec)->elems, new_max * sizeof(*new_elems)); \
-                       if (new_elems) { \
-                               (vec)->elems = new_elems; \
-                               (vec)->max = new_max; \
-                       } else { \
-                               res = -1; \
-                               break; \
-                       } \
+               if (__make_room(((idx) > (vec)->current ? (idx) : (vec)->current), vec) != 0) {                                                 \
+                       res = -1;                                                                               \
+                       break;                                                                                  \
+               }                                                                                                               \
+               if ((vec)->current > 0 && (idx) < (vec)->current) { \
+                       __move = ((vec)->current - (idx)) * sizeof(typeof((vec)->elems[0])); \
+                       memmove(&(vec)->elems[(idx) + 1], &(vec)->elems[(idx)], __move); \
                } \
-               __move = ((vec)->current - 1) * sizeof(typeof((vec)->elems[0])); \
-               memmove(&(vec)->elems[(idx) + 1], &(vec)->elems[(idx)], __move); \
                (vec)->elems[(idx)] = (elem); \
-               (vec)->current++; \
+               (vec)->current = ((idx) > (vec)->current ? (idx) : (vec)->current) + 1; \
        } while (0); \
        res; \
 })
 })
 
 /*!
- * \brief Execute a callback on every element in a vector
+ * \brief Default callback for AST_VECTOR_CALLBACK()
+ *
+ * \param elem Element to compare against
+ * \param value Value to compare with the vector element.
+ *
+ * \return CMP_MATCH always.
+ */
+#define AST_VECTOR_MATCH_ALL(element) (CMP_MATCH)
+
+
+/*!
+ * \brief Execute a callback on every element in a vector returning the first matched
  *
  * \param vec Vector to operate on.
  * \param callback A callback that takes at least 1 argument (the element)
  * plus number of optional arguments
+ * \param default_value A default value to return if no elements matched
  *
- * \return the number of elements visited before the end of the vector
- * was reached or CMP_STOP was returned.
+ * \return the first element matched before CMP_STOP was returned
+ * or the end of the vector was reached. Otherwise, default_value
  */
-#define AST_VECTOR_CALLBACK(vec, callback, ...) ({ \
+#define AST_VECTOR_CALLBACK(vec, callback, default_value, ...) ({ \
        size_t idx; \
+       typeof((vec)->elems[0]) res = default_value;                            \
        for (idx = 0; idx < (vec)->current; idx++) { \
                int rc = callback((vec)->elems[idx], ##__VA_ARGS__);    \
-               if (rc == CMP_STOP) { \
-                       idx++; \
+               if (rc & CMP_MATCH) { \
+                       res = (vec)->elems[idx]; \
                        break; \
                }\
+               if (rc & CMP_STOP) { \
+                       break; \
+               }\
+       } \
+       res; \
+})
+
+/*!
+ * \brief Execute a callback on every element in a vector returning the matching
+ * elements in a new vector
+ *
+ * \param vec Vector to operate on.
+ * \param callback A callback that takes at least 1 argument (the element)
+ * plus number of optional arguments
+ *
+ * \return a vector containing the elements matched before CMP_STOP was returned
+ * or the end of the vector was reached. The vector may be empty and could be NULL
+ * if there was not enough memory to allocate it's control structure.
+ *
+ * \warning The returned vector must have AST_VECTOR_PTR_FREE()
+ * called on it after you've finished with it.
+ *
+ * \note The type of the returned vector must be traceable to the original vector.
+ *
+ * The following will resut in "error: assignment from incompatible pointer type"
+ * because these declare 2 different structures.
+ *
+ * \code
+ * AST_VECTOR(, char *) vector_1;
+ * AST_VECTOR(, char *) *vector_2;
+ *
+ * vector_2 = AST_VECTOR_CALLBACK_MULTIPLE(&vector_1, callback);
+ * \endcode
+ *
+ * This will work because you're using the type of the first
+ * to declare the second:
+ *
+ * \code
+ * AST_VECTOR(mytype, char *) vector_1;
+ * struct mytype *vector_2 = NULL;
+ *
+ * vector_2 = AST_VECTOR_CALLBACK_MULTIPLE(&vector_1, callback);
+ * \endcode
+ *
+ * This will also work because you're declaring both vector_1 and
+ * vector_2 from the same definition.
+ *
+ * \code
+ * AST_VECTOR(, char *) vector_1, *vector_2 = NULL;
+ *
+ * vector_2 = AST_VECTOR_CALLBACK_MULTIPLE(&vector_1, callback);
+ * \endcode
+ */
+#define AST_VECTOR_CALLBACK_MULTIPLE(vec, callback, ...) ({ \
+       size_t idx; \
+       typeof((vec)) new_vec; \
+       do { \
+               new_vec = ast_malloc(sizeof(*new_vec)); \
+               if (!new_vec) { \
+                       break; \
+               } \
+               if (AST_VECTOR_INIT(new_vec, AST_VECTOR_SIZE((vec))) != 0) { \
+                       ast_free(new_vec); \
+                       new_vec = NULL; \
+                       break; \
+               } \
+               for (idx = 0; idx < (vec)->current; idx++) { \
+                       int rc = callback((vec)->elems[idx], ##__VA_ARGS__);    \
+                       if (rc & CMP_MATCH) { \
+                               AST_VECTOR_APPEND(new_vec, (vec)->elems[idx]); \
+                       } \
+                       if (rc & CMP_STOP) { \
+                               break; \
+                       }\
+               } \
+       } while(0); \
+       new_vec; \
+})
+
+/*!
+ * \brief Execute a callback on every element in a vector disregarding callback return
+ *
+ * \param vec Vector to operate on.
+ * \param callback A callback that takes at least 1 argument (the element)
+ * plus number of optional arguments
+ */
+#define AST_VECTOR_CALLBACK_VOID(vec, callback, ...) ({ \
+       size_t idx; \
+       for (idx = 0; idx < (vec)->current; idx++) { \
+               callback((vec)->elems[idx], ##__VA_ARGS__);     \
        } \
-       idx; \
 })
 
 /*!
index da9b133..a05e191 100644 (file)
@@ -37,6 +37,7 @@
 #include "asterisk/astobj2.h"
 #include "asterisk/sorcery.h"
 #include "asterisk/app.h"
+#include "asterisk/taskprocessor.h"
 
 #define BODY_SIZE 1024
 #define EVENT_TYPE_SIZE 50
@@ -53,6 +54,8 @@ struct exten_state_subscription {
        int id;
        /*! The SIP subscription */
        struct ast_sip_subscription *sip_sub;
+       /*! The serializer to use for notifications */
+       struct ast_taskprocessor *serializer;
        /*! Context in which subscription looks for updates */
        char context[AST_MAX_CONTEXT];
        /*! Extension within the context to receive updates from */
@@ -113,6 +116,7 @@ static void exten_state_subscription_destructor(void *obj)
 
        ast_free(sub->user_agent);
        ao2_cleanup(sub->sip_sub);
+       ast_taskprocessor_unreference(sub->serializer);
 }
 
 static char *get_user_agent(const struct ast_sip_subscription *sip_sub)
@@ -157,6 +161,13 @@ static struct exten_state_subscription *exten_state_subscription_alloc(
        }
 
        exten_state_sub->sip_sub = ao2_bump(sip_sub);
+
+       /* We keep our own reference to the serializer as there is no guarantee in state_changed
+        * that the subscription tree is still valid when it is called. This can occur when
+        * the subscription is terminated at around the same time as the state_changed
+        * callback is invoked.
+        */
+       exten_state_sub->serializer = ao2_bump(ast_sip_subscription_get_serializer(sip_sub));
        exten_state_sub->last_exten_state = INITIAL_LAST_EXTEN_STATE;
        exten_state_sub->last_presence_state = AST_PRESENCE_NOT_SET;
        exten_state_sub->user_agent = get_user_agent(sip_sub);
@@ -205,11 +216,6 @@ static struct notify_task_data *alloc_notify_task_data(char *exten, struct exten
        task_data->exten_state_data.device_state_info = ao2_bump(info->device_state_info);
        task_data->exten_state_data.sub = exten_state_sub->sip_sub;
 
-       ast_sip_subscription_get_local_uri(exten_state_sub->sip_sub,
-                       task_data->exten_state_data.local, sizeof(task_data->exten_state_data.local));
-       ast_sip_subscription_get_remote_uri(exten_state_sub->sip_sub,
-                       task_data->exten_state_data.remote, sizeof(task_data->exten_state_data.remote));
-
        if ((info->exten_state == AST_EXTENSION_DEACTIVATED) ||
            (info->exten_state == AST_EXTENSION_REMOVED)) {
                ast_verb(2, "Watcher for hint %s %s\n", exten, info->exten_state
@@ -228,6 +234,19 @@ static int notify_task(void *obj)
                .body_data = &task_data->exten_state_data,
        };
 
+       /* Terminated subscriptions are no longer associated with a valid tree, and sending
+        * NOTIFY messages on a subscription which has already been terminated won't work.
+        */
+       if (ast_sip_subscription_is_terminated(task_data->exten_state_sub->sip_sub)) {
+               return 0;
+       }
+
+       /* All access to the subscription must occur within a task executed within its serializer */
+       ast_sip_subscription_get_local_uri(task_data->exten_state_sub->sip_sub,
+                       task_data->exten_state_data.local, sizeof(task_data->exten_state_data.local));
+       ast_sip_subscription_get_remote_uri(task_data->exten_state_sub->sip_sub,
+                       task_data->exten_state_data.remote, sizeof(task_data->exten_state_data.remote));
+
        /* Pool allocation has to happen here so that we allocate within a PJLIB thread */
        task_data->exten_state_data.pool = pjsip_endpt_create_pool(ast_sip_get_pjsip_endpoint(),
                        "exten_state", 1024, 1024);
@@ -263,8 +282,8 @@ static int state_changed(char *context, char *exten,
 
        /* safe to push this async since we copy the data from info and
           add a ref for the device state info */
-       if (ast_sip_push_task(ast_sip_subscription_get_serializer(task_data->exten_state_sub->sip_sub),
-                             notify_task, task_data)) {
+       if (ast_sip_push_task(task_data->exten_state_sub->serializer, notify_task,
+               task_data)) {
                ao2_cleanup(task_data);
                return -1;
        }
index ebc43d1..1ecb17d 100644 (file)
@@ -2185,6 +2185,11 @@ const char *ast_sip_subscription_get_resource_name(struct ast_sip_subscription *
        return sub->resource;
 }
 
+int ast_sip_subscription_is_terminated(const struct ast_sip_subscription *sub)
+{
+       return sub->subscription_state == PJSIP_EVSUB_STATE_TERMINATED ? 1 : 0;
+}
+
 static int sip_subscription_accept(struct sip_subscription_tree *sub_tree, pjsip_rx_data *rdata, int response)
 {
        pjsip_hdr res_hdr;
index 2a6b75f..58702d6 100644 (file)
@@ -37,6 +37,7 @@
                LINKER_SYMBOL_PREFIXast_sip_subscription_get_local_uri;
                LINKER_SYMBOL_PREFIXast_sip_subscription_get_remote_uri;
                LINKER_SYMBOL_PREFIXast_sip_subscription_get_header;
+               LINKER_SYMBOL_PREFIXast_sip_subscription_is_terminated;
        local:
                *;
 };
index eae1881..bd45b0c 100644 (file)
@@ -42,14 +42,6 @@ ASTERISK_REGISTER_FILE()
 #include "asterisk/module.h"
 #include "asterisk/vector.h"
 
-#define test_validate_cleanup(condition) ({ \
-       if (!(condition)) {     \
-               ast_test_status_update((test), "%s: %s\n", "Condition failed", #condition); \
-               rc = AST_TEST_FAIL; \
-               goto cleanup; \
-       } \
-})
-
 static int cleanup_count;
 
 static void cleanup(char *element)
@@ -57,6 +49,10 @@ static void cleanup(char *element)
        cleanup_count++;
 }
 
+#define STRING_CMP(a, b) ({ \
+       ((a) == NULL || (b) == NULL) ? -1 : (strcmp((a), (b)) == 0); \
+})
+
 AST_TEST_DEFINE(basic_ops)
 {
        AST_VECTOR(test_struct, char *) sv1;
@@ -65,6 +61,7 @@ AST_TEST_DEFINE(basic_ops)
        char *AAA = "AAA";
        char *BBB = "BBB";
        char *CCC = "CCC";
+       char *YYY = "YYY";
        char *ZZZ = "ZZZ";
 
        switch (cmd) {
@@ -79,105 +76,131 @@ AST_TEST_DEFINE(basic_ops)
        }
 
        ast_test_validate(test, AST_VECTOR_INIT(&sv1, 3) == 0);
-       test_validate_cleanup(sv1.max == 3);
-       test_validate_cleanup(AST_VECTOR_SIZE(&sv1) == 0);
-
-       test_validate_cleanup(AST_VECTOR_APPEND(&sv1, AAA) == 0);
-       test_validate_cleanup(AST_VECTOR_APPEND(&sv1, BBB) == 0);
-       test_validate_cleanup(AST_VECTOR_APPEND(&sv1, CCC) == 0);
-       test_validate_cleanup(AST_VECTOR_SIZE(&sv1) == 3);
-       test_validate_cleanup(sv1.max == 3);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 0) == AAA);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 1) == BBB);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 2) == CCC);
-
-       test_validate_cleanup(AST_VECTOR_INSERT_AT(&sv1, 1, ZZZ) == 0);
-       test_validate_cleanup(sv1.max >= 4);
-       test_validate_cleanup(AST_VECTOR_SIZE(&sv1) == 4);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 0) == AAA);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 1) == ZZZ);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 2) == BBB);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 3) == CCC);
-
-       test_validate_cleanup(*(char **)AST_VECTOR_GET_CMP(&sv1, "AAA", 0 == strcmp) == AAA);
-       test_validate_cleanup(*(char **)AST_VECTOR_GET_CMP(&sv1, "ZZZ", 0 == strcmp) == ZZZ);
-
+       ast_test_validate_cleanup(test, sv1.max == 3, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_SIZE(&sv1) == 0, rc, cleanup);
+       /* there should be no vector growth for the 3 appends */
+       ast_test_validate_cleanup(test, AST_VECTOR_APPEND(&sv1, AAA) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, sv1.max == 3, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_APPEND(&sv1, BBB) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, sv1.max == 3, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_APPEND(&sv1, CCC) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_SIZE(&sv1) == 3, rc, cleanup);
+       ast_test_validate_cleanup(test, sv1.max >= 3, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 0) == AAA, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 1) == BBB, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 2) == CCC, rc, cleanup);
+       ast_test_validate_cleanup(test, sv1.max == sv1.current, rc, cleanup);
+
+       ast_test_validate_cleanup(test, AST_VECTOR_INSERT_AT(&sv1, 1, ZZZ) == 0, rc, cleanup);
+       /* The vector should have grown */
+       ast_test_validate_cleanup(test, sv1.max == 8, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_SIZE(&sv1) == 4, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 0) == AAA, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 1) == ZZZ, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 2) == BBB, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 3) == CCC, rc, cleanup);
+
+       /* Test inserting > current but < max */
+       ast_test_validate_cleanup(test, AST_VECTOR_INSERT_AT(&sv1, 6, YYY) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, sv1.current == 7, rc, cleanup);
+       /* The vector should not have grown */
+       ast_test_validate_cleanup(test, sv1.max == 8, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 6) == YYY, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 4) == NULL, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 5) == NULL, rc, cleanup);
+       ast_test_validate_cleanup(test, *(char **)AST_VECTOR_GET_CMP(&sv1, "AAA", STRING_CMP) == AAA, rc, cleanup);
+       ast_test_validate_cleanup(test, *(char **)AST_VECTOR_GET_CMP(&sv1, "ZZZ", STRING_CMP) == ZZZ, rc, cleanup);
+
+       /* Test inserting > max */
+       ast_test_validate_cleanup(test, AST_VECTOR_INSERT_AT(&sv1, 12, AAA) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, sv1.current == 13, rc, cleanup);
+       /* The vector should have grown */
+       ast_test_validate_cleanup(test, sv1.max == 26, rc, cleanup);
+
+       /* RESET */
        AST_VECTOR_FREE(&sv1);
        ast_test_validate(test, sv1.elems == NULL);
        ast_test_validate(test, sv1.current == 0);
        ast_test_validate(test, sv1.max == 0);
 
+       /* Test with initial size = 0 */
        ast_test_validate(test, AST_VECTOR_INIT(&sv1, 0) == 0);
-       test_validate_cleanup(sv1.max == 0);
-       test_validate_cleanup(AST_VECTOR_SIZE(&sv1) == 0);
+       ast_test_validate_cleanup(test, sv1.max == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_SIZE(&sv1) == 0, rc, cleanup);
 
-       test_validate_cleanup(AST_VECTOR_APPEND(&sv1, AAA) == 0);
-       test_validate_cleanup(AST_VECTOR_APPEND(&sv1, BBB) == 0);
-       test_validate_cleanup(AST_VECTOR_APPEND(&sv1, CCC) == 0);
-       test_validate_cleanup(sv1.max >= 3);
-       test_validate_cleanup(AST_VECTOR_SIZE(&sv1) == 3);
+       ast_test_validate_cleanup(test, AST_VECTOR_APPEND(&sv1, AAA) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_APPEND(&sv1, BBB) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_APPEND(&sv1, CCC) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, sv1.max >= 3, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_SIZE(&sv1) == 3, rc, cleanup);
 
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 0) == AAA);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 1) == BBB);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 2) == CCC);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 0) == AAA, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 1) == BBB, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 2) == CCC, rc, cleanup);
 
        /* Overwrite index 1 */
-       test_validate_cleanup(AST_VECTOR_REPLACE(&sv1, 1, ZZZ) == 0);
-       test_validate_cleanup(AST_VECTOR_SIZE(&sv1) == 3);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 0) == AAA);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 1) == ZZZ);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 2) == CCC);
-
-       /* Remove index 0 and bring the last entry into it's empty slot */
-       test_validate_cleanup(AST_VECTOR_REMOVE_UNORDERED(&sv1, 0) == AAA);
-       test_validate_cleanup(AST_VECTOR_SIZE(&sv1) == 2);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 0) == CCC);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 1) == ZZZ);
+       ast_test_validate_cleanup(test, AST_VECTOR_REPLACE(&sv1, 1, ZZZ) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, sv1.current == 3, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 0) == AAA, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 1) == ZZZ, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 2) == CCC, rc, cleanup);
+
+       /* Replace beyond current */
+       ast_test_validate_cleanup(test, AST_VECTOR_REPLACE(&sv1, 10, YYY) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, sv1.current == 11, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 0) == AAA, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 1) == ZZZ, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 2) == CCC, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 5) == NULL, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 10) == YYY, rc, cleanup);
+
+       /* Replace beyond max */
+       ast_test_validate_cleanup(test, AST_VECTOR_REPLACE(&sv1, 100, YYY) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, sv1.current == 101, rc, cleanup);
+       ast_test_validate_cleanup(test, sv1.max >= 101, rc, cleanup);
+
+       /* Remove index 0 and bring the last entry (10/YYY) into it's empty slot */
+       ast_test_validate_cleanup(test, AST_VECTOR_REMOVE_UNORDERED(&sv1, 0) == AAA, rc, cleanup);
+       ast_test_validate_cleanup(test, sv1.current == 100, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 0) == YYY, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 1) == ZZZ, rc, cleanup);
 
        /* Replace 0 and 2 leaving 1 alone */
-       test_validate_cleanup(AST_VECTOR_REPLACE(&sv1, 0, AAA) == 0);
-       test_validate_cleanup(AST_VECTOR_REPLACE(&sv1, 2, CCC) == 0);
-       test_validate_cleanup(AST_VECTOR_SIZE(&sv1) == 3);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 0) == AAA);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 1) == ZZZ);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 2) == CCC);
+       ast_test_validate_cleanup(test, AST_VECTOR_REPLACE(&sv1, 0, AAA) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_REPLACE(&sv1, 2, CCC) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, sv1.current == 100, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 0) == AAA, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 1) == ZZZ, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 2) == CCC, rc, cleanup);
 
        /* Remove 1 and compact preserving order */
-       test_validate_cleanup(AST_VECTOR_REMOVE_ORDERED(&sv1, 1) == ZZZ);
-       test_validate_cleanup(AST_VECTOR_SIZE(&sv1) == 2);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 0) == AAA);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 1) == CCC);
+       ast_test_validate_cleanup(test, AST_VECTOR_REMOVE_ORDERED(&sv1, 1) == ZZZ, rc, cleanup);
+       ast_test_validate_cleanup(test, sv1.current == 99, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 0) == AAA, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 1) == CCC, rc, cleanup);
 
-       /* Equivalent of APPEND */
-       test_validate_cleanup(AST_VECTOR_REPLACE(&sv1, 2, ZZZ) == 0);
-       test_validate_cleanup(AST_VECTOR_SIZE(&sv1) == 3);
+       ast_test_validate_cleanup(test, AST_VECTOR_INSERT_AT(&sv1, 0, ZZZ) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, sv1.current == 100, rc, cleanup);
 
        /* This should fail because comparison is by pointer */
-       test_validate_cleanup(AST_VECTOR_REMOVE_ELEM_ORDERED(&sv1, "ZZZ", cleanup) != 0);
+       ast_test_validate_cleanup(test, AST_VECTOR_REMOVE_ELEM_ORDERED(&sv1, "ZZZ", cleanup) != 0, rc, cleanup);
 
        /* This should work because we passing in the specific object to be removed */
        cleanup_count = 0;
-       test_validate_cleanup(AST_VECTOR_REMOVE_ELEM_ORDERED(&sv1, ZZZ, cleanup) == 0);
-       test_validate_cleanup(AST_VECTOR_SIZE(&sv1) == 2);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 0) == AAA);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 1) == CCC);
-       test_validate_cleanup(cleanup_count == 1);
+       ast_test_validate_cleanup(test, AST_VECTOR_REMOVE_ELEM_ORDERED(&sv1, ZZZ, cleanup) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, sv1.current == 99, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 0) == AAA, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 1) == CCC, rc, cleanup);
+       ast_test_validate_cleanup(test, cleanup_count == 1, rc, cleanup);
 
        /* If we want a comparison by value, we need to pass in a comparison
-        * function.  The comparison looks weird but that's what it takes.
+        * function.
         */
        cleanup_count = 0;
-       test_validate_cleanup(AST_VECTOR_REMOVE_CMP_ORDERED(&sv1, "AAA", 0 == strcmp, cleanup) == 0);
-       test_validate_cleanup(AST_VECTOR_SIZE(&sv1) == 1);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 0) == CCC);
-       test_validate_cleanup(cleanup_count == 1);
-
-       /* This element is gone so we shouldn't be able to find it or delete it again. */
-       test_validate_cleanup(AST_VECTOR_GET_CMP(&sv1, "AAA", 0 == strcmp) == NULL);
-       test_validate_cleanup(AST_VECTOR_REMOVE_CMP_ORDERED(&sv1, "AAA", 0 == strcmp, cleanup) != 0);
-
-       /* CCC should still be there though */
-       test_validate_cleanup(*(char **)AST_VECTOR_GET_CMP(&sv1, "CCC", 0 == strcmp) == CCC);
+       ast_test_validate_cleanup(test, AST_VECTOR_REMOVE_CMP_ORDERED(&sv1, "AAA", STRING_CMP, cleanup) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_SIZE(&sv1) == 98, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 0) == CCC, rc, cleanup);
+       ast_test_validate_cleanup(test, cleanup_count == 1, rc, cleanup);
 
 cleanup:
        AST_VECTOR_FREE(&sv1);
@@ -211,28 +234,30 @@ AST_TEST_DEFINE(basic_ops_integer)
        }
 
        ast_test_validate(test, AST_VECTOR_INIT(&sv1, 3) == 0);
-       test_validate_cleanup(sv1.max == 3);
-       test_validate_cleanup(AST_VECTOR_SIZE(&sv1) == 0);
-
-       test_validate_cleanup(AST_VECTOR_APPEND(&sv1, AAA) == 0);
-       test_validate_cleanup(AST_VECTOR_APPEND(&sv1, BBB) == 0);
-       test_validate_cleanup(AST_VECTOR_APPEND(&sv1, CCC) == 0);
-       test_validate_cleanup(AST_VECTOR_SIZE(&sv1) == 3);
-       test_validate_cleanup(sv1.max == 3);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 0) == AAA);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 1) == BBB);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 2) == CCC);
-
-       test_validate_cleanup(AST_VECTOR_INSERT_AT(&sv1, 1, ZZZ) == 0);
-       test_validate_cleanup(sv1.max >= 4);
-       test_validate_cleanup(AST_VECTOR_SIZE(&sv1) == 4);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 0) == AAA);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 1) == ZZZ);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 2) == BBB);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 3) == CCC);
-
-       test_validate_cleanup(*(int *)AST_VECTOR_GET_CMP(&sv1, AAA,  AST_VECTOR_ELEM_DEFAULT_CMP) == AAA);
-       test_validate_cleanup(*(int *)AST_VECTOR_GET_CMP(&sv1, ZZZ, AST_VECTOR_ELEM_DEFAULT_CMP) == ZZZ);
+       ast_test_validate_cleanup(test, sv1.max == 3, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_SIZE(&sv1) == 0, rc, cleanup);
+
+       ast_test_validate_cleanup(test, AST_VECTOR_APPEND(&sv1, AAA) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, sv1.max == 3, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_APPEND(&sv1, BBB) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, sv1.max == 3, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_APPEND(&sv1, CCC) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_SIZE(&sv1) == 3, rc, cleanup);
+       ast_test_validate_cleanup(test, sv1.max == 3, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 0) == AAA, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 1) == BBB, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 2) == CCC, rc, cleanup);
+
+       ast_test_validate_cleanup(test, AST_VECTOR_INSERT_AT(&sv1, 1, ZZZ) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, sv1.max >= 4, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_SIZE(&sv1) == 4, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 0) == AAA, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 1) == ZZZ, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 2) == BBB, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 3) == CCC, rc, cleanup);
+
+       ast_test_validate_cleanup(test, *(int *)AST_VECTOR_GET_CMP(&sv1, AAA,  AST_VECTOR_ELEM_DEFAULT_CMP) == AAA, rc, cleanup);
+       ast_test_validate_cleanup(test, *(int *)AST_VECTOR_GET_CMP(&sv1, ZZZ, AST_VECTOR_ELEM_DEFAULT_CMP) == ZZZ, rc, cleanup);
 
        AST_VECTOR_FREE(&sv1);
        ast_test_validate(test, sv1.elems == NULL);
@@ -240,94 +265,104 @@ AST_TEST_DEFINE(basic_ops_integer)
        ast_test_validate(test, sv1.max == 0);
 
        ast_test_validate(test, AST_VECTOR_INIT(&sv1, 0) == 0);
-       test_validate_cleanup(sv1.max == 0);
-       test_validate_cleanup(AST_VECTOR_SIZE(&sv1) == 0);
+       ast_test_validate_cleanup(test, sv1.max == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_SIZE(&sv1) == 0, rc, cleanup);
 
-       test_validate_cleanup(AST_VECTOR_APPEND(&sv1, AAA) == 0);
-       test_validate_cleanup(AST_VECTOR_APPEND(&sv1, BBB) == 0);
-       test_validate_cleanup(AST_VECTOR_APPEND(&sv1, CCC) == 0);
-       test_validate_cleanup(sv1.max >= 3);
-       test_validate_cleanup(AST_VECTOR_SIZE(&sv1) == 3);
+       ast_test_validate_cleanup(test, AST_VECTOR_APPEND(&sv1, AAA) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_APPEND(&sv1, BBB) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_APPEND(&sv1, CCC) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, sv1.max >= 3, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_SIZE(&sv1) == 3, rc, cleanup);
 
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 0) == AAA);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 1) == BBB);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 2) == CCC);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 0) == AAA, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 1) == BBB, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 2) == CCC, rc, cleanup);
 
        /* Overwrite index 1 */
-       test_validate_cleanup(AST_VECTOR_REPLACE(&sv1, 1, ZZZ) == 0);
-       test_validate_cleanup(AST_VECTOR_SIZE(&sv1) == 3);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 0) == AAA);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 1) == ZZZ);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 2) == CCC);
+       ast_test_validate_cleanup(test, AST_VECTOR_REPLACE(&sv1, 1, ZZZ) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_SIZE(&sv1) == 3, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 0) == AAA, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 1) == ZZZ, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 2) == CCC, rc, cleanup);
 
        /* Remove index 0 and bring the last entry into it's empty slot */
-       test_validate_cleanup(AST_VECTOR_REMOVE_UNORDERED(&sv1, 0) == 1);
-       test_validate_cleanup(AST_VECTOR_SIZE(&sv1) == 2);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 0) == CCC);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 1) == ZZZ);
+       ast_test_validate_cleanup(test, AST_VECTOR_REMOVE_UNORDERED(&sv1, 0) == 1, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_SIZE(&sv1) == 2, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 0) == CCC, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 1) == ZZZ, rc, cleanup);
 
        /* Replace 0 and 2 leaving 1 alone */
-       test_validate_cleanup(AST_VECTOR_REPLACE(&sv1, 0, AAA) == 0);
-       test_validate_cleanup(AST_VECTOR_REPLACE(&sv1, 2, CCC) == 0);
-       test_validate_cleanup(AST_VECTOR_SIZE(&sv1) == 3);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 0) == AAA);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 1) == ZZZ);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 2) == CCC);
+       ast_test_validate_cleanup(test, AST_VECTOR_REPLACE(&sv1, 0, AAA) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_REPLACE(&sv1, 2, CCC) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_SIZE(&sv1) == 3, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 0) == AAA, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 1) == ZZZ, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 2) == CCC, rc, cleanup);
 
        /* Remove 1 and compact preserving order */
-       test_validate_cleanup(AST_VECTOR_REMOVE_ORDERED(&sv1, 1) == ZZZ);
-       test_validate_cleanup(AST_VECTOR_SIZE(&sv1) == 2);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 0) == AAA);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 1) == CCC);
+       ast_test_validate_cleanup(test, AST_VECTOR_REMOVE_ORDERED(&sv1, 1) == ZZZ, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_SIZE(&sv1) == 2, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 0) == AAA, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 1) == CCC, rc, cleanup);
 
        /* Equivalent of APPEND */
-       test_validate_cleanup(AST_VECTOR_REPLACE(&sv1, 2, ZZZ) == 0);
-       test_validate_cleanup(AST_VECTOR_SIZE(&sv1) == 3);
+       ast_test_validate_cleanup(test, AST_VECTOR_REPLACE(&sv1, 2, ZZZ) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_SIZE(&sv1) == 3, rc, cleanup);
 
        /* This should work because we passing in the specific object to be removed */
        cleanup_count = 0;
-       test_validate_cleanup(AST_VECTOR_REMOVE_ELEM_ORDERED(&sv1, ZZZ, cleanup_int) == 0);
-       test_validate_cleanup(AST_VECTOR_SIZE(&sv1) == 2);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 0) == AAA);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 1) == CCC);
-       test_validate_cleanup(cleanup_count == 1);
+       ast_test_validate_cleanup(test, AST_VECTOR_REMOVE_ELEM_ORDERED(&sv1, ZZZ, cleanup_int) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_SIZE(&sv1) == 2, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 0) == AAA, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 1) == CCC, rc, cleanup);
+       ast_test_validate_cleanup(test, cleanup_count == 1, rc, cleanup);
 
        /* If we want a comparison by value, we need to pass in a comparison
         * function.
         */
        cleanup_count = 0;
-       test_validate_cleanup(AST_VECTOR_REMOVE_CMP_ORDERED(&sv1, AAA, AST_VECTOR_ELEM_DEFAULT_CMP, cleanup_int) == 0);
-       test_validate_cleanup(AST_VECTOR_SIZE(&sv1) == 1);
-       test_validate_cleanup(AST_VECTOR_GET(&sv1, 0) == CCC);
-       test_validate_cleanup(cleanup_count == 1);
+       ast_test_validate_cleanup(test, AST_VECTOR_REMOVE_CMP_ORDERED(&sv1, AAA, AST_VECTOR_ELEM_DEFAULT_CMP, cleanup_int) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_SIZE(&sv1) == 1, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 0) == CCC, rc, cleanup);
+       ast_test_validate_cleanup(test, cleanup_count == 1, rc, cleanup);
 
        /* This element is gone so we shouldn't be able to find it or delete it again. */
-       test_validate_cleanup(AST_VECTOR_GET_CMP(&sv1, AAA, AST_VECTOR_ELEM_DEFAULT_CMP) == NULL);
-       test_validate_cleanup(AST_VECTOR_REMOVE_CMP_ORDERED(&sv1, AAA, AST_VECTOR_ELEM_DEFAULT_CMP, cleanup_int) != 0);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET_CMP(&sv1, AAA, AST_VECTOR_ELEM_DEFAULT_CMP) == NULL, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_REMOVE_CMP_ORDERED(&sv1, AAA, AST_VECTOR_ELEM_DEFAULT_CMP, cleanup_int) != 0, rc, cleanup);
 
        /* CCC should still be there though */
-       test_validate_cleanup(*(int *)AST_VECTOR_GET_CMP(&sv1, CCC, AST_VECTOR_ELEM_DEFAULT_CMP) == CCC);
+       ast_test_validate_cleanup(test, *(int *)AST_VECTOR_GET_CMP(&sv1, CCC, AST_VECTOR_ELEM_DEFAULT_CMP) == CCC, rc, cleanup);
 
 cleanup:
        AST_VECTOR_FREE(&sv1);
        return rc;
 }
 
+static int visits;
 
-static int cb(void *obj, void *arg, void *data)
+static int cb_match(void *obj, void *arg)
 {
-       return strcmp(arg, "ARG") == 0 ? 0 : CMP_STOP;
+       visits++;
+       return strcmp(arg, obj) == 0 ? CMP_MATCH : 0;
 }
 
-static int cb_first(void *obj, void *arg, void *data)
+static int cb_visits(void *obj, int v)
 {
-       return data == arg ? CMP_STOP : 0;
+       visits++;
+       return visits == v ? CMP_STOP : 0;
 }
 
 AST_TEST_DEFINE(callbacks)
 {
        AST_VECTOR(, char *) sv1;
+       typeof(sv1) *sv2 = NULL;
+
        int rc = AST_TEST_PASS;
+       char *AAA = "AAA";
+       char *AAA2 = "AAA";
+       char *BBB = "BBB";
+       char *CCC = "CCC";
+       char *DEF = "default_value";
 
        switch (cmd) {
        case TEST_INIT:
@@ -342,16 +377,47 @@ AST_TEST_DEFINE(callbacks)
 
        AST_VECTOR_INIT(&sv1, 32);
 
-       AST_VECTOR_APPEND(&sv1, "AAA");
-       AST_VECTOR_APPEND(&sv1, "BBB");
-       AST_VECTOR_APPEND(&sv1, "CCC");
+       AST_VECTOR_APPEND(&sv1, AAA);
+       AST_VECTOR_APPEND(&sv1, BBB);
+       AST_VECTOR_APPEND(&sv1, CCC);
+       AST_VECTOR_APPEND(&sv1, AAA2);
+
+       visits = 0;
+       ast_test_validate_cleanup(test, AST_VECTOR_CALLBACK(&sv1, cb_match, DEF, "AAA") == AAA, rc, cleanup);
+       ast_test_validate_cleanup(test, visits == 1, rc, cleanup);
+
+       visits = 0;
+       ast_test_validate_cleanup(test, AST_VECTOR_CALLBACK(&sv1, cb_match, DEF, "XYZ") == DEF, rc, cleanup);
+       ast_test_validate_cleanup(test, visits == 4, rc, cleanup);
+
+       visits = 0;
+       ast_test_validate_cleanup(test, AST_VECTOR_CALLBACK(&sv1, cb_visits, DEF, 2) == DEF, rc, cleanup);
+       ast_test_validate_cleanup(test, visits == 2, rc, cleanup);
+
+
+       sv2 = AST_VECTOR_CALLBACK_MULTIPLE(&sv1, AST_VECTOR_MATCH_ALL);
+       ast_test_validate_cleanup(test, AST_VECTOR_SIZE(sv2) == 4, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(sv2, 0) == AAA, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(sv2, 1) == BBB, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(sv2, 2) == CCC, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(sv2, 3) == AAA2, rc, cleanup);
+
+       AST_VECTOR_PTR_FREE(sv2);
 
-       test_validate_cleanup(AST_VECTOR_CALLBACK(&sv1, cb, "ARG", test) == 3);
+       AST_VECTOR_APPEND(&sv1, AAA);
+       AST_VECTOR_APPEND(&sv1, BBB);
+       AST_VECTOR_APPEND(&sv1, CCC);
+       ast_test_validate_cleanup(test, AST_VECTOR_SIZE(&sv1) == 7, rc, cleanup);
 
-       test_validate_cleanup(AST_VECTOR_CALLBACK(&sv1, cb_first, test, test) == 1);
+       sv2 = AST_VECTOR_CALLBACK_MULTIPLE(&sv1, cb_match, "AAA");
+       ast_test_validate_cleanup(test, AST_VECTOR_SIZE(sv2) == 3, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(sv2, 0) == AAA, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(sv2, 1) == AAA2, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_GET(sv2, 2) == AAA, rc, cleanup);
 
 cleanup:
        AST_VECTOR_FREE(&sv1);
+       AST_VECTOR_PTR_FREE(sv2);
 
        return rc;
 }
@@ -379,25 +445,25 @@ AST_TEST_DEFINE(locks)
 
        AST_VECTOR_RW_INIT(&sv1, 0);
 
-       test_validate_cleanup(AST_VECTOR_RW_RDLOCK(&sv1) == 0);
-       test_validate_cleanup(AST_VECTOR_RW_UNLOCK(&sv1) == 0);
-       test_validate_cleanup(AST_VECTOR_RW_WRLOCK(&sv1) == 0);
-       test_validate_cleanup(AST_VECTOR_RW_UNLOCK(&sv1) == 0);
+       ast_test_validate_cleanup(test, AST_VECTOR_RW_RDLOCK(&sv1) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_RW_UNLOCK(&sv1) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_RW_WRLOCK(&sv1) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_RW_UNLOCK(&sv1) == 0, rc, cleanup);
 
-       test_validate_cleanup(AST_VECTOR_RW_RDLOCK_TRY(&sv1) == 0);
-       test_validate_cleanup(AST_VECTOR_RW_WRLOCK_TRY(&sv1) != 0);
-       test_validate_cleanup(AST_VECTOR_RW_UNLOCK(&sv1) == 0);
-       test_validate_cleanup(AST_VECTOR_RW_WRLOCK_TRY(&sv1) == 0);
-       test_validate_cleanup(AST_VECTOR_RW_UNLOCK(&sv1) == 0);
+       ast_test_validate_cleanup(test, AST_VECTOR_RW_RDLOCK_TRY(&sv1) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_RW_WRLOCK_TRY(&sv1) != 0, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_RW_UNLOCK(&sv1) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_RW_WRLOCK_TRY(&sv1) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_RW_UNLOCK(&sv1) == 0, rc, cleanup);
 
        ts.tv_nsec = 0;
        ts.tv_sec = 2;
 
-       test_validate_cleanup(AST_VECTOR_RW_RDLOCK_TIMED(&sv1, &ts) == 0);
-       test_validate_cleanup(AST_VECTOR_RW_WRLOCK_TIMED(&sv1, &ts) != 0);
-       test_validate_cleanup(AST_VECTOR_RW_UNLOCK(&sv1) == 0);
-       test_validate_cleanup(AST_VECTOR_RW_WRLOCK_TIMED(&sv1, &ts) == 0);
-       test_validate_cleanup(AST_VECTOR_RW_UNLOCK(&sv1) == 0);
+       ast_test_validate_cleanup(test, AST_VECTOR_RW_RDLOCK_TIMED(&sv1, &ts) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_RW_WRLOCK_TIMED(&sv1, &ts) != 0, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_RW_UNLOCK(&sv1) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_RW_WRLOCK_TIMED(&sv1, &ts) == 0, rc, cleanup);
+       ast_test_validate_cleanup(test, AST_VECTOR_RW_UNLOCK(&sv1) == 0, rc, cleanup);
 
 cleanup:
        AST_VECTOR_RW_FREE(&sv1);