If we're going to be passing a negative value for the size of a stringfield,

in order to indicate something, then using an UNSIGNED parameter is bad,
mmmmmkay?

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

diff --git a/include/asterisk/stringfields.h b/include/asterisk/stringfields.h
index 5fe0fc3..f842601 100644 (file)
--- a/include/asterisk/stringfields.h
+++ b/include/asterisk/stringfields.h
@@ -230,7 +230,7 @@ void __ast_string_field_ptr_build_va(struct ast_string_field_mgr *mgr,
 
 /*! \internal \brief internal version of ast_string_field_init */
 int __ast_string_field_init(struct ast_string_field_mgr *mgr,
-       struct ast_string_field_pool **pool_head, size_t needed);
+       struct ast_string_field_pool **pool_head, int needed);
 
 /*!
   \brief Set a field to a simple string value

diff --git a/main/utils.c b/main/utils.c
index 49e559d..d948891 100644 (file)
--- a/main/utils.c
+++ b/main/utils.c
@@ -1244,7 +1244,7 @@ static int add_string_pool(struct ast_string_field_mgr *mgr,
  *     This must be done before destroying the object.
  */
 int __ast_string_field_init(struct ast_string_field_mgr *mgr,
-       struct ast_string_field_pool **pool_head, size_t size)
+       struct ast_string_field_pool **pool_head, int size)
 {
        const char **p = (const char **)pool_head + 1;
        struct ast_string_field_pool *cur = *pool_head;