]> git.ipfire.org Git - thirdparty/asterisk.git/commitdiff
Merged revisions 187599 via svnmerge from
authorTilghman Lesher <tilghman@meg.abyt.es>
Fri, 10 Apr 2009 03:56:37 +0000 (03:56 +0000)
committerTilghman Lesher <tilghman@meg.abyt.es>
Fri, 10 Apr 2009 03:56:37 +0000 (03:56 +0000)
https://origsvn.digium.com/svn/asterisk/trunk

........
  r187599 | tilghman | 2009-04-09 22:55:27 -0500 (Thu, 09 Apr 2009) | 2 lines

  Modify headers and macros, according to Russell's suggestions on the -dev list
........

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

include/asterisk/linkedlists.h
include/asterisk/lock.h
main/app.c
main/audiohook.c
main/channel.c
main/features.c
main/http.c
main/manager.c
main/pbx.c

index dd5e2f6ca03cc055fb499cce8d4709f72759211a..e9492cc168bd26b8dd829bb2e34cf4a1d376b393 100644 (file)
 #include "asterisk/lock.h"
 
 /*!
-  \file linkedlists.h
-  \brief A set of macros to manage forward-linked lists.
-*/
* \file linkedlists.h
* \brief A set of macros to manage forward-linked lists.
+ */
 
 /*!
-  \brief Locks a list.
-  \param head This is a pointer to the list head structure
-
-  This macro attempts to place an exclusive lock in the
-  list head structure pointed to by head.
-  \retval 0 on success
-  \retval non-zero on failure
-*/
* \brief Locks a list.
* \param head This is a pointer to the list head structure
+ *
* This macro attempts to place an exclusive lock in the
* list head structure pointed to by head.
* \retval 0 on success
* \retval non-zero on failure
+ */
 #define AST_LIST_LOCK(head)                                            \
        ast_mutex_lock(&(head)->lock)
 
 /*!
-  \brief Write locks a list.
-  \param head This is a pointer to the list head structure
-
-  This macro attempts to place an exclusive write lock in the
-  list head structure pointed to by head.
-  \retval 0 on success
-  \retval non-zero on failure
-*/
* \brief Write locks a list.
* \param head This is a pointer to the list head structure
+ *
* This macro attempts to place an exclusive write lock in the
* list head structure pointed to by head.
* \retval 0 on success
* \retval non-zero on failure
+ */
 #define AST_RWLIST_WRLOCK(head)                                         \
         ast_rwlock_wrlock(&(head)->lock)
 
 /*!
-  \brief Write locks a list, with timeout.
-  \param head This is a pointer to the list head structure
-  \param tv Pointer to a timeval structure
-
-  This macro attempts to place an exclusive write lock in the
-  list head structure pointed to by head.
-  \retval 0 on success
-  \retval non-zero on failure
-*/
-#define        AST_RWLIST_TIMEDWRLOCK(head,tv) ast_rwlock_timedwrlock(&(head)->lock, tv)
+ * \brief Write locks a list, with timeout.
+ * \param head This is a pointer to the list head structure
+ * \param ts Pointer to a timespec structure
+ *
+ * This macro attempts to place an exclusive write lock in the
+ * list head structure pointed to by head.
+ * \retval 0 on success
+ * \retval non-zero on failure
+ *
+ * \since 1.6.1
+ */
+#define        AST_RWLIST_TIMEDWRLOCK(head, ts)        ast_rwlock_timedwrlock(&(head)->lock, ts)
 
 /*!
-  \brief Read locks a list.
-  \param head This is a pointer to the list head structure
-
-  This macro attempts to place a read lock in the
-  list head structure pointed to by head.
-  \retval 0 on success
-  \retval non-zero on failure
-*/
* \brief Read locks a list.
* \param head This is a pointer to the list head structure
+ *
* This macro attempts to place a read lock in the
* list head structure pointed to by head.
* \retval 0 on success
* \retval non-zero on failure
+ */
 #define AST_RWLIST_RDLOCK(head)                                         \
         ast_rwlock_rdlock(&(head)->lock)
 
 /*!
-  \brief Read locks a list, with timeout.
-  \param head This is a pointer to the list head structure
-  \param tv Pointer to a timeval structure
-
-  This macro attempts to place a read lock in the
-  list head structure pointed to by head.
-  \retval 0 on success
-  \retval non-zero on failure
-*/
-#define AST_RWLIST_TIMEDRDLOCK(head,tv)                                 \
-        ast_rwlock_timedrdlock(&(head)->lock, tv)
+ * \brief Read locks a list, with timeout.
+ * \param head This is a pointer to the list head structure
+ * \param ts Pointer to a timespec structure
+ *
+ * This macro attempts to place a read lock in the
+ * list head structure pointed to by head.
+ * \retval 0 on success
+ * \retval non-zero on failure
+ *
+ * \since 1.6.1
+ */
+#define AST_RWLIST_TIMEDRDLOCK(head, ts)                                 \
+       ast_rwlock_timedrdlock(&(head)->lock, ts)
 
 /*!
-  \brief Locks a list, without blocking if the list is locked.
-  \param head This is a pointer to the list head structure
-
-  This macro attempts to place an exclusive lock in the
-  list head structure pointed to by head.
-  \retval 0 on success
-  \retval non-zero on failure
-*/
* \brief Locks a list, without blocking if the list is locked.
* \param head This is a pointer to the list head structure
+ *
* This macro attempts to place an exclusive lock in the
* list head structure pointed to by head.
* \retval 0 on success
* \retval non-zero on failure
+ */
 #define AST_LIST_TRYLOCK(head)                                         \
        ast_mutex_trylock(&(head)->lock)
 
 /*!
-  \brief Write locks a list, without blocking if the list is locked.
-  \param head This is a pointer to the list head structure
-
-  This macro attempts to place an exclusive write lock in the
-  list head structure pointed to by head.
-  \retval 0 on success
-  \retval non-zero on failure
-*/
* \brief Write locks a list, without blocking if the list is locked.
* \param head This is a pointer to the list head structure
+ *
* This macro attempts to place an exclusive write lock in the
* list head structure pointed to by head.
* \retval 0 on success
* \retval non-zero on failure
+ */
 #define AST_RWLIST_TRYWRLOCK(head)                                      \
         ast_rwlock_trywrlock(&(head)->lock)
 
 /*!
-  \brief Read locks a list, without blocking if the list is locked.
-  \param head This is a pointer to the list head structure
-
-  This macro attempts to place a read lock in the
-  list head structure pointed to by head.
-  \retval 0 on success
-  \retval non-zero on failure
-*/
* \brief Read locks a list, without blocking if the list is locked.
* \param head This is a pointer to the list head structure
+ *
* This macro attempts to place a read lock in the
* list head structure pointed to by head.
* \retval 0 on success
* \retval non-zero on failure
+ */
 #define AST_RWLIST_TRYRDLOCK(head)                                      \
         ast_rwlock_tryrdlock(&(head)->lock)
 
 /*!
-  \brief Attempts to unlock a list.
-  \param head This is a pointer to the list head structure
-
-  This macro attempts to remove an exclusive lock from the
-  list head structure pointed to by head. If the list
-  was not locked by this thread, this macro has no effect.
-*/
-#define AST_LIST_UNLOCK(head)                                          \
* \brief Attempts to unlock a list.
* \param head This is a pointer to the list head structure
+ *
* This macro attempts to remove an exclusive lock from the
* list head structure pointed to by head. If the list
* was not locked by this thread, this macro has no effect.
+ */
+#define AST_LIST_UNLOCK(head)                                          \
        ast_mutex_unlock(&(head)->lock)
 
 /*!
-  \brief Attempts to unlock a read/write based list.
-  \param head This is a pointer to the list head structure
-
-  This macro attempts to remove a read or write lock from the
-  list head structure pointed to by head. If the list
-  was not locked by this thread, this macro has no effect.
-*/
* \brief Attempts to unlock a read/write based list.
* \param head This is a pointer to the list head structure
+ *
* This macro attempts to remove a read or write lock from the
* list head structure pointed to by head. If the list
* was not locked by this thread, this macro has no effect.
+ */
 #define AST_RWLIST_UNLOCK(head)                                         \
         ast_rwlock_unlock(&(head)->lock)
 
 /*!
-  \brief Defines a structure to be used to hold a list of specified type.
-  \param name This will be the name of the defined structure.
-  \param type This is the type of each list entry.
-
-  This macro creates a structure definition that can be used
-  to hold a list of the entries of type \a type. It does not actually
-  declare (allocate) a structure; to do that, either follow this
-  macro with the desired name of the instance you wish to declare,
-  or use the specified \a name to declare instances elsewhere.
-
-  Example usage:
-  \code
-  static AST_LIST_HEAD(entry_list, entry) entries;
-  \endcode
-
-  This would define \c struct \c entry_list, and declare an instance of it named
-  \a entries, all intended to hold a list of type \c struct \c entry.
-*/
* \brief Defines a structure to be used to hold a list of specified type.
* \param name This will be the name of the defined structure.
* \param type This is the type of each list entry.
+ *
* This macro creates a structure definition that can be used
* to hold a list of the entries of type \a type. It does not actually
* declare (allocate) a structure; to do that, either follow this
* macro with the desired name of the instance you wish to declare,
* or use the specified \a name to declare instances elsewhere.
+ *
* Example usage:
* \code
* static AST_LIST_HEAD(entry_list, entry) entries;
* \endcode
+ *
* This would define \c struct \c entry_list, and declare an instance of it named
* \a entries, all intended to hold a list of type \c struct \c entry.
+ */
 #define AST_LIST_HEAD(name, type)                                      \
 struct name {                                                          \
        struct type *first;                                             \
@@ -173,24 +177,24 @@ struct name {                                                             \
 }
 
 /*!
-  \brief Defines a structure to be used to hold a read/write list of specified type.
-  \param name This will be the name of the defined structure.
-  \param type This is the type of each list entry.
-
-  This macro creates a structure definition that can be used
-  to hold a list of the entries of type \a type. It does not actually
-  declare (allocate) a structure; to do that, either follow this
-  macro with the desired name of the instance you wish to declare,
-  or use the specified \a name to declare instances elsewhere.
-
-  Example usage:
-  \code
-  static AST_RWLIST_HEAD(entry_list, entry) entries;
-  \endcode
-
-  This would define \c struct \c entry_list, and declare an instance of it named
-  \a entries, all intended to hold a list of type \c struct \c entry.
-*/
* \brief Defines a structure to be used to hold a read/write list of specified type.
* \param name This will be the name of the defined structure.
* \param type This is the type of each list entry.
+ *
* This macro creates a structure definition that can be used
* to hold a list of the entries of type \a type. It does not actually
* declare (allocate) a structure; to do that, either follow this
* macro with the desired name of the instance you wish to declare,
* or use the specified \a name to declare instances elsewhere.
+ *
* Example usage:
* \code
* static AST_RWLIST_HEAD(entry_list, entry) entries;
* \endcode
+ *
* This would define \c struct \c entry_list, and declare an instance of it named
* \a entries, all intended to hold a list of type \c struct \c entry.
+ */
 #define AST_RWLIST_HEAD(name, type)                                     \
 struct name {                                                           \
         struct type *first;                                             \
@@ -199,24 +203,24 @@ struct name {                                                           \
 }
 
 /*!
-  \brief Defines a structure to be used to hold a list of specified type (with no lock).
-  \param name This will be the name of the defined structure.
-  \param type This is the type of each list entry.
-
-  This macro creates a structure definition that can be used
-  to hold a list of the entries of type \a type. It does not actually
-  declare (allocate) a structure; to do that, either follow this
-  macro with the desired name of the instance you wish to declare,
-  or use the specified \a name to declare instances elsewhere.
-
-  Example usage:
-  \code
-  static AST_LIST_HEAD_NOLOCK(entry_list, entry) entries;
-  \endcode
-
-  This would define \c struct \c entry_list, and declare an instance of it named
-  \a entries, all intended to hold a list of type \c struct \c entry.
-*/
* \brief Defines a structure to be used to hold a list of specified type (with no lock).
* \param name This will be the name of the defined structure.
* \param type This is the type of each list entry.
+ *
* This macro creates a structure definition that can be used
* to hold a list of the entries of type \a type. It does not actually
* declare (allocate) a structure; to do that, either follow this
* macro with the desired name of the instance you wish to declare,
* or use the specified \a name to declare instances elsewhere.
+ *
* Example usage:
* \code
* static AST_LIST_HEAD_NOLOCK(entry_list, entry) entries;
* \endcode
+ *
* This would define \c struct \c entry_list, and declare an instance of it named
* \a entries, all intended to hold a list of type \c struct \c entry.
+ */
 #define AST_LIST_HEAD_NOLOCK(name, type)                               \
 struct name {                                                          \
        struct type *first;                                             \
@@ -224,8 +228,8 @@ struct name {                                                               \
 }
 
 /*!
-  \brief Defines initial values for a declaration of AST_LIST_HEAD
-*/
* \brief Defines initial values for a declaration of AST_LIST_HEAD
+ */
 #define AST_LIST_HEAD_INIT_VALUE       {               \
        .first = NULL,                                  \
        .last = NULL,                                   \
@@ -233,8 +237,8 @@ struct name {                                                               \
        }
 
 /*!
-  \brief Defines initial values for a declaration of AST_RWLIST_HEAD
-*/
* \brief Defines initial values for a declaration of AST_RWLIST_HEAD
+ */
 #define AST_RWLIST_HEAD_INIT_VALUE      {               \
         .first = NULL,                                  \
         .last = NULL,                                   \
@@ -242,30 +246,30 @@ struct name {                                                             \
         }
 
 /*!
-  \brief Defines initial values for a declaration of AST_LIST_HEAD_NOLOCK
-*/
* \brief Defines initial values for a declaration of AST_LIST_HEAD_NOLOCK
+ */
 #define AST_LIST_HEAD_NOLOCK_INIT_VALUE        {       \
        .first = NULL,                                  \
        .last = NULL,                                   \
        }
 
 /*!
-  \brief Defines a structure to be used to hold a list of specified type, statically initialized.
-  \param name This will be the name of the defined structure.
-  \param type This is the type of each list entry.
-
-  This macro creates a structure definition that can be used
-  to hold a list of the entries of type \a type, and allocates an instance
-  of it, initialized to be empty.
-
-  Example usage:
-  \code
-  static AST_LIST_HEAD_STATIC(entry_list, entry);
-  \endcode
-
-  This would define \c struct \c entry_list, intended to hold a list of
-  type \c struct \c entry.
-*/
* \brief Defines a structure to be used to hold a list of specified type, statically initialized.
* \param name This will be the name of the defined structure.
* \param type This is the type of each list entry.
+ *
* This macro creates a structure definition that can be used
* to hold a list of the entries of type \a type, and allocates an instance
* of it, initialized to be empty.
+ *
* Example usage:
* \code
* static AST_LIST_HEAD_STATIC(entry_list, entry);
* \endcode
+ *
* This would define \c struct \c entry_list, intended to hold a list of
* type \c struct \c entry.
+ */
 #if defined(AST_MUTEX_INIT_W_CONSTRUCTORS)
 #define AST_LIST_HEAD_STATIC(name, type)                               \
 struct name {                                                          \
@@ -292,22 +296,22 @@ struct name {                                                             \
 #endif
 
 /*!
-  \brief Defines a structure to be used to hold a read/write list of specified type, statically initialized.
-  \param name This will be the name of the defined structure.
-  \param type This is the type of each list entry.
-
-  This macro creates a structure definition that can be used
-  to hold a list of the entries of type \a type, and allocates an instance
-  of it, initialized to be empty.
-
-  Example usage:
-  \code
-  static AST_RWLIST_HEAD_STATIC(entry_list, entry);
-  \endcode
-
-  This would define \c struct \c entry_list, intended to hold a list of
-  type \c struct \c entry.
-*/
* \brief Defines a structure to be used to hold a read/write list of specified type, statically initialized.
* \param name This will be the name of the defined structure.
* \param type This is the type of each list entry.
+ *
* This macro creates a structure definition that can be used
* to hold a list of the entries of type \a type, and allocates an instance
* of it, initialized to be empty.
+ *
* Example usage:
* \code
* static AST_RWLIST_HEAD_STATIC(entry_list, entry);
* \endcode
+ *
* This would define \c struct \c entry_list, intended to hold a list of
* type \c struct \c entry.
+ */
 #ifndef HAVE_PTHREAD_RWLOCK_INITIALIZER
 #define AST_RWLIST_HEAD_STATIC(name, type)                              \
 struct name {                                                           \
@@ -334,10 +338,10 @@ struct name {                                                           \
 #endif
 
 /*!
-  \brief Defines a structure to be used to hold a list of specified type, statically initialized.
-
-  This is the same as AST_LIST_HEAD_STATIC, except without the lock included.
-*/
* \brief Defines a structure to be used to hold a list of specified type, statically initialized.
+ *
* This is the same as AST_LIST_HEAD_STATIC, except without the lock included.
+ */
 #define AST_LIST_HEAD_NOLOCK_STATIC(name, type)                                \
 struct name {                                                          \
        struct type *first;                                             \
@@ -345,13 +349,13 @@ struct name {                                                             \
 } name = AST_LIST_HEAD_NOLOCK_INIT_VALUE
 
 /*!
-  \brief Initializes a list head structure with a specified first entry.
-  \param head This is a pointer to the list head structure
-  \param entry pointer to the list entry that will become the head of the list
-
-  This macro initializes a list head structure by setting the head
-  entry to the supplied value and recreating the embedded lock.
-*/
* \brief Initializes a list head structure with a specified first entry.
* \param head This is a pointer to the list head structure
* \param entry pointer to the list entry that will become the head of the list
+ *
* This macro initializes a list head structure by setting the head
* entry to the supplied value and recreating the embedded lock.
+ */
 #define AST_LIST_HEAD_SET(head, entry) do {                            \
        (head)->first = (entry);                                        \
        (head)->last = (entry);                                         \
@@ -359,13 +363,13 @@ struct name {                                                             \
 } while (0)
 
 /*!
-  \brief Initializes an rwlist head structure with a specified first entry.
-  \param head This is a pointer to the list head structure
-  \param entry pointer to the list entry that will become the head of the list
-
-  This macro initializes a list head structure by setting the head
-  entry to the supplied value and recreating the embedded lock.
-*/
* \brief Initializes an rwlist head structure with a specified first entry.
* \param head This is a pointer to the list head structure
* \param entry pointer to the list entry that will become the head of the list
+ *
* This macro initializes a list head structure by setting the head
* entry to the supplied value and recreating the embedded lock.
+ */
 #define AST_RWLIST_HEAD_SET(head, entry) do {                           \
         (head)->first = (entry);                                        \
         (head)->last = (entry);                                         \
@@ -373,35 +377,35 @@ struct name {                                                             \
 } while (0)
 
 /*!
-  \brief Initializes a list head structure with a specified first entry.
-  \param head This is a pointer to the list head structure
-  \param entry pointer to the list entry that will become the head of the list
-
-  This macro initializes a list head structure by setting the head
-  entry to the supplied value.
-*/
* \brief Initializes a list head structure with a specified first entry.
* \param head This is a pointer to the list head structure
* \param entry pointer to the list entry that will become the head of the list
+ *
* This macro initializes a list head structure by setting the head
* entry to the supplied value.
+ */
 #define AST_LIST_HEAD_SET_NOLOCK(head, entry) do {                     \
        (head)->first = (entry);                                        \
        (head)->last = (entry);                                         \
 } while (0)
 
 /*!
-  \brief Declare a forward link structure inside a list entry.
-  \param type This is the type of each list entry.
-
-  This macro declares a structure to be used to link list entries together.
-  It must be used inside the definition of the structure named in
-  \a type, as follows:
-
-  \code
-  struct list_entry {
-       ...
-       AST_LIST_ENTRY(list_entry) list;
-  }
-  \endcode
-
-  The field name \a list here is arbitrary, and can be anything you wish.
-*/
* \brief Declare a forward link structure inside a list entry.
* \param type This is the type of each list entry.
+ *
* This macro declares a structure to be used to link list entries together.
* It must be used inside the definition of the structure named in
* \a type, as follows:
+ *
* \code
* struct list_entry {
+      ...
+      AST_LIST_ENTRY(list_entry) list;
* }
* \endcode
+ *
* The field name \a list here is arbitrary, and can be anything you wish.
+ */
 #define AST_LIST_ENTRY(type)                                           \
 struct {                                                               \
        struct type *next;                                              \
@@ -410,117 +414,117 @@ struct {                                                                \
 #define AST_RWLIST_ENTRY AST_LIST_ENTRY
 
 /*!
-  \brief Returns the first entry contained in a list.
-  \param head This is a pointer to the list head structure
* \brief Returns the first entry contained in a list.
* \param head This is a pointer to the list head structure
  */
 #define        AST_LIST_FIRST(head)    ((head)->first)
 
 #define AST_RWLIST_FIRST AST_LIST_FIRST
 
 /*!
-  \brief Returns the last entry contained in a list.
-  \param head This is a pointer to the list head structure
* \brief Returns the last entry contained in a list.
* \param head This is a pointer to the list head structure
  */
 #define        AST_LIST_LAST(head)     ((head)->last)
 
 #define AST_RWLIST_LAST AST_LIST_LAST
 
 /*!
-  \brief Returns the next entry in the list after the given entry.
-  \param elm This is a pointer to the current entry.
-  \param field This is the name of the field (declared using AST_LIST_ENTRY())
-  used to link entries of this list together.
-*/
* \brief Returns the next entry in the list after the given entry.
* \param elm This is a pointer to the current entry.
* \param field This is the name of the field (declared using AST_LIST_ENTRY())
* used to link entries of this list together.
+ */
 #define AST_LIST_NEXT(elm, field)      ((elm)->field.next)
 
 #define AST_RWLIST_NEXT AST_LIST_NEXT
 
 /*!
-  \brief Checks whether the specified list contains any entries.
-  \param head This is a pointer to the list head structure
-
-  \return non-zero if the list has entries
-  \return zero if not.
* \brief Checks whether the specified list contains any entries.
* \param head This is a pointer to the list head structure
+ *
* \return non-zero if the list has entries
* \return zero if not.
  */
 #define        AST_LIST_EMPTY(head)    (AST_LIST_FIRST(head) == NULL)
 
 #define AST_RWLIST_EMPTY AST_LIST_EMPTY
 
 /*!
-  \brief Loops over (traverses) the entries in a list.
-  \param head This is a pointer to the list head structure
-  \param var This is the name of the variable that will hold a pointer to the
-  current list entry on each iteration. It must be declared before calling
-  this macro.
-  \param field This is the name of the field (declared using AST_LIST_ENTRY())
-  used to link entries of this list together.
-
-  This macro is use to loop over (traverse) the entries in a list. It uses a
-  \a for loop, and supplies the enclosed code with a pointer to each list
-  entry as it loops. It is typically used as follows:
-  \code
-  static AST_LIST_HEAD(entry_list, list_entry) entries;
-  ...
-  struct list_entry {
-       ...
-       AST_LIST_ENTRY(list_entry) list;
-  }
-  ...
-  struct list_entry *current;
-  ...
-  AST_LIST_TRAVERSE(&entries, current, list) {
* \brief Loops over (traverses) the entries in a list.
* \param head This is a pointer to the list head structure
* \param var This is the name of the variable that will hold a pointer to the
* current list entry on each iteration. It must be declared before calling
* this macro.
* \param field This is the name of the field (declared using AST_LIST_ENTRY())
* used to link entries of this list together.
+ *
* This macro is use to loop over (traverse) the entries in a list. It uses a
* \a for loop, and supplies the enclosed code with a pointer to each list
* entry as it loops. It is typically used as follows:
* \code
* static AST_LIST_HEAD(entry_list, list_entry) entries;
* ...
* struct list_entry {
+      ...
+      AST_LIST_ENTRY(list_entry) list;
* }
* ...
* struct list_entry *current;
* ...
* AST_LIST_TRAVERSE(&entries, current, list) {
      (do something with current here)
-  }
-  \endcode
-  \warning If you modify the forward-link pointer contained in the \a current entry while
-  inside the loop, the behavior will be unpredictable. At a minimum, the following
-  macros will modify the forward-link pointer, and should not be used inside
-  AST_LIST_TRAVERSE() against the entry pointed to by the \a current pointer without
-  careful consideration of their consequences:
-  \li AST_LIST_NEXT() (when used as an lvalue)
-  \li AST_LIST_INSERT_AFTER()
-  \li AST_LIST_INSERT_HEAD()
-  \li AST_LIST_INSERT_TAIL()
-  \li AST_LIST_INSERT_SORTALPHA()
-*/
* }
* \endcode
* \warning If you modify the forward-link pointer contained in the \a current entry while
* inside the loop, the behavior will be unpredictable. At a minimum, the following
* macros will modify the forward-link pointer, and should not be used inside
* AST_LIST_TRAVERSE() against the entry pointed to by the \a current pointer without
* careful consideration of their consequences:
* \li AST_LIST_NEXT() (when used as an lvalue)
* \li AST_LIST_INSERT_AFTER()
* \li AST_LIST_INSERT_HEAD()
* \li AST_LIST_INSERT_TAIL()
* \li AST_LIST_INSERT_SORTALPHA()
+ */
 #define AST_LIST_TRAVERSE(head,var,field)                              \
        for((var) = (head)->first; (var); (var) = (var)->field.next)
 
 #define AST_RWLIST_TRAVERSE AST_LIST_TRAVERSE
 
 /*!
-  \brief Loops safely over (traverses) the entries in a list.
-  \param head This is a pointer to the list head structure
-  \param var This is the name of the variable that will hold a pointer to the
-  current list entry on each iteration. It must be declared before calling
-  this macro.
-  \param field This is the name of the field (declared using AST_LIST_ENTRY())
-  used to link entries of this list together.
-
-  This macro is used to safely loop over (traverse) the entries in a list. It
-  uses a \a for loop, and supplies the enclosed code with a pointer to each list
-  entry as it loops. It is typically used as follows:
-
-  \code
-  static AST_LIST_HEAD(entry_list, list_entry) entries;
-  ...
-  struct list_entry {
-       ...
-       AST_LIST_ENTRY(list_entry) list;
-  }
-  ...
-  struct list_entry *current;
-  ...
-  AST_LIST_TRAVERSE_SAFE_BEGIN(&entries, current, list) {
* \brief Loops safely over (traverses) the entries in a list.
* \param head This is a pointer to the list head structure
* \param var This is the name of the variable that will hold a pointer to the
* current list entry on each iteration. It must be declared before calling
* this macro.
* \param field This is the name of the field (declared using AST_LIST_ENTRY())
* used to link entries of this list together.
+ *
* This macro is used to safely loop over (traverse) the entries in a list. It
* uses a \a for loop, and supplies the enclosed code with a pointer to each list
* entry as it loops. It is typically used as follows:
+ *
* \code
* static AST_LIST_HEAD(entry_list, list_entry) entries;
* ...
* struct list_entry {
+      ...
+      AST_LIST_ENTRY(list_entry) list;
* }
* ...
* struct list_entry *current;
* ...
* AST_LIST_TRAVERSE_SAFE_BEGIN(&entries, current, list) {
      (do something with current here)
-  }
-  AST_LIST_TRAVERSE_SAFE_END;
-  \endcode
-
-  It differs from AST_LIST_TRAVERSE() in that the code inside the loop can modify
-  (or even free, after calling AST_LIST_REMOVE_CURRENT()) the entry pointed to by
-  the \a current pointer without affecting the loop traversal.
-*/
* }
* AST_LIST_TRAVERSE_SAFE_END;
* \endcode
+ *
* It differs from AST_LIST_TRAVERSE() in that the code inside the loop can modify
* (or even free, after calling AST_LIST_REMOVE_CURRENT()) the entry pointed to by
* the \a current pointer without affecting the loop traversal.
+ */
 #define AST_LIST_TRAVERSE_SAFE_BEGIN(head, var, field) {                               \
        typeof((head)) __list_head = head;                                              \
        typeof(__list_head->first) __list_next;                                         \
@@ -537,14 +541,14 @@ struct {                                                          \
 #define AST_RWLIST_TRAVERSE_SAFE_BEGIN AST_LIST_TRAVERSE_SAFE_BEGIN
 
 /*!
-  \brief Removes the \a current entry from a list during a traversal.
-  \param field This is the name of the field (declared using AST_LIST_ENTRY())
-  used to link entries of this list together.
-
-  \note This macro can \b only be used inside an AST_LIST_TRAVERSE_SAFE_BEGIN()
-  block; it is used to unlink the current entry from the list without affecting
-  the list traversal (and without having to re-traverse the list to modify the
-  previous entry, if any).
* \brief Removes the \a current entry from a list during a traversal.
* \param field This is the name of the field (declared using AST_LIST_ENTRY())
* used to link entries of this list together.
+ *
* \note This macro can \b only be used inside an AST_LIST_TRAVERSE_SAFE_BEGIN()
* block; it is used to unlink the current entry from the list without affecting
* the list traversal (and without having to re-traverse the list to modify the
* previous entry, if any).
  */
 #define AST_LIST_REMOVE_CURRENT(field) do { \
        __new_prev->field.next = NULL;                                                  \
@@ -554,7 +558,7 @@ struct {                                                            \
        else                                                                            \
                __list_head->first = __list_next;                                       \
        if (!__list_next)                                                               \
-               __list_head->last = __list_prev;                                        \
+               __list_head->last = __list_prev;                                        \
        } while (0)
 
 #define AST_RWLIST_REMOVE_CURRENT AST_LIST_REMOVE_CURRENT
@@ -568,13 +572,13 @@ struct {                                                          \
 #define AST_RWLIST_MOVE_CURRENT AST_LIST_MOVE_CURRENT
 
 /*!
-  \brief Inserts a list entry before the current entry during a traversal.
-  \param elm This is a pointer to the entry to be inserted.
-  \param field This is the name of the field (declared using AST_LIST_ENTRY())
-  used to link entries of this list together.
-
-  \note This macro can \b only be used inside an AST_LIST_TRAVERSE_SAFE_BEGIN()
-  block.
* \brief Inserts a list entry before the current entry during a traversal.
* \param elm This is a pointer to the entry to be inserted.
* \param field This is the name of the field (declared using AST_LIST_ENTRY())
* used to link entries of this list together.
+ *
* \note This macro can \b only be used inside an AST_LIST_TRAVERSE_SAFE_BEGIN()
* block.
  */
 #define AST_LIST_INSERT_BEFORE_CURRENT(elm, field) do {                        \
        if (__list_prev) {                                              \
@@ -590,19 +594,19 @@ struct {                                                          \
 #define AST_RWLIST_INSERT_BEFORE_CURRENT AST_LIST_INSERT_BEFORE_CURRENT
 
 /*!
-  \brief Closes a safe loop traversal block.
* \brief Closes a safe loop traversal block.
  */
 #define AST_LIST_TRAVERSE_SAFE_END  }
 
 #define AST_RWLIST_TRAVERSE_SAFE_END AST_LIST_TRAVERSE_SAFE_END
 
 /*!
-  \brief Initializes a list head structure.
-  \param head This is a pointer to the list head structure
-
-  This macro initializes a list head structure by setting the head
-  entry to \a NULL (empty list) and recreating the embedded lock.
-*/
* \brief Initializes a list head structure.
* \param head This is a pointer to the list head structure
+ *
* This macro initializes a list head structure by setting the head
* entry to \a NULL (empty list) and recreating the embedded lock.
+ */
 #define AST_LIST_HEAD_INIT(head) {                                     \
        (head)->first = NULL;                                           \
        (head)->last = NULL;                                            \
@@ -610,12 +614,12 @@ struct {                                                          \
 }
 
 /*!
-  \brief Initializes an rwlist head structure.
-  \param head This is a pointer to the list head structure
-
-  This macro initializes a list head structure by setting the head
-  entry to \a NULL (empty list) and recreating the embedded lock.
-*/
* \brief Initializes an rwlist head structure.
* \param head This is a pointer to the list head structure
+ *
* This macro initializes a list head structure by setting the head
* entry to \a NULL (empty list) and recreating the embedded lock.
+ */
 #define AST_RWLIST_HEAD_INIT(head) {                                    \
         (head)->first = NULL;                                           \
         (head)->last = NULL;                                            \
@@ -623,13 +627,13 @@ struct {                                                          \
 }
 
 /*!
-  \brief Destroys a list head structure.
-  \param head This is a pointer to the list head structure
-
-  This macro destroys a list head structure by setting the head
-  entry to \a NULL (empty list) and destroying the embedded lock.
-  It does not free the structure from memory.
-*/
* \brief Destroys a list head structure.
* \param head This is a pointer to the list head structure
+ *
* This macro destroys a list head structure by setting the head
* entry to \a NULL (empty list) and destroying the embedded lock.
* It does not free the structure from memory.
+ */
 #define AST_LIST_HEAD_DESTROY(head) {                                  \
        (head)->first = NULL;                                           \
        (head)->last = NULL;                                            \
@@ -637,13 +641,13 @@ struct {                                                          \
 }
 
 /*!
-  \brief Destroys an rwlist head structure.
-  \param head This is a pointer to the list head structure
-
-  This macro destroys a list head structure by setting the head
-  entry to \a NULL (empty list) and destroying the embedded lock.
-  It does not free the structure from memory.
-*/
* \brief Destroys an rwlist head structure.
* \param head This is a pointer to the list head structure
+ *
* This macro destroys a list head structure by setting the head
* entry to \a NULL (empty list) and destroying the embedded lock.
* It does not free the structure from memory.
+ */
 #define AST_RWLIST_HEAD_DESTROY(head) {                                 \
         (head)->first = NULL;                                           \
         (head)->last = NULL;                                            \
@@ -651,26 +655,26 @@ struct {                                                          \
 }
 
 /*!
-  \brief Initializes a list head structure.
-  \param head This is a pointer to the list head structure
-
-  This macro initializes a list head structure by setting the head
-  entry to \a NULL (empty list). There is no embedded lock handling
-  with this macro.
-*/
* \brief Initializes a list head structure.
* \param head This is a pointer to the list head structure
+ *
* This macro initializes a list head structure by setting the head
* entry to \a NULL (empty list). There is no embedded lock handling
* with this macro.
+ */
 #define AST_LIST_HEAD_INIT_NOLOCK(head) {                              \
        (head)->first = NULL;                                           \
        (head)->last = NULL;                                            \
 }
 
 /*!
-  \brief Inserts a list entry after a given entry.
-  \param head This is a pointer to the list head structure
-  \param listelm This is a pointer to the entry after which the new entry should
-  be inserted.
-  \param elm This is a pointer to the entry to be inserted.
-  \param field This is the name of the field (declared using AST_LIST_ENTRY())
-  used to link entries of this list together.
* \brief Inserts a list entry after a given entry.
* \param head This is a pointer to the list head structure
* \param listelm This is a pointer to the entry after which the new entry should
* be inserted.
* \param elm This is a pointer to the entry to be inserted.
* \param field This is the name of the field (declared using AST_LIST_ENTRY())
* used to link entries of this list together.
  */
 #define AST_LIST_INSERT_AFTER(head, listelm, elm, field) do {          \
        (elm)->field.next = (listelm)->field.next;                      \
@@ -682,11 +686,11 @@ struct {                                                          \
 #define AST_RWLIST_INSERT_AFTER AST_LIST_INSERT_AFTER
 
 /*!
-  \brief Inserts a list entry at the head of a list.
-  \param head This is a pointer to the list head structure
-  \param elm This is a pointer to the entry to be inserted.
-  \param field This is the name of the field (declared using AST_LIST_ENTRY())
-  used to link entries of this list together.
* \brief Inserts a list entry at the head of a list.
* \param head This is a pointer to the list head structure
* \param elm This is a pointer to the entry to be inserted.
* \param field This is the name of the field (declared using AST_LIST_ENTRY())
* used to link entries of this list together.
  */
 #define AST_LIST_INSERT_HEAD(head, elm, field) do {                    \
                (elm)->field.next = (head)->first;                      \
@@ -698,15 +702,15 @@ struct {                                                          \
 #define AST_RWLIST_INSERT_HEAD AST_LIST_INSERT_HEAD
 
 /*!
-  \brief Appends a list entry to the tail of a list.
-  \param head This is a pointer to the list head structure
-  \param elm This is a pointer to the entry to be appended.
-  \param field This is the name of the field (declared using AST_LIST_ENTRY())
-  used to link entries of this list together.
-
-  Note: The link field in the appended entry is \b not modified, so if it is
-  actually the head of a list itself, the entire list will be appended
-  temporarily (until the next AST_LIST_INSERT_TAIL is performed).
* \brief Appends a list entry to the tail of a list.
* \param head This is a pointer to the list head structure
* \param elm This is a pointer to the entry to be appended.
* \param field This is the name of the field (declared using AST_LIST_ENTRY())
* used to link entries of this list together.
+ *
* Note: The link field in the appended entry is \b not modified, so if it is
* actually the head of a list itself, the entire list will be appended
* temporarily (until the next AST_LIST_INSERT_TAIL is performed).
  */
 #define AST_LIST_INSERT_TAIL(head, elm, field) do {                    \
       if (!(head)->first) {                                            \
@@ -751,14 +755,14 @@ struct {                                                          \
 #define AST_RWLIST_INSERT_SORTALPHA    AST_LIST_INSERT_SORTALPHA
 
 /*!
-  \brief Appends a whole list to the tail of a list.
-  \param head This is a pointer to the list head structure
-  \param list This is a pointer to the list to be appended.
-  \param field This is the name of the field (declared using AST_LIST_ENTRY())
-  used to link entries of this list together.
-
-  Note: The source list (the \a list parameter) will be empty after
-  calling this macro (the list entries are \b moved to the target list).
* \brief Appends a whole list to the tail of a list.
* \param head This is a pointer to the list head structure
* \param list This is a pointer to the list to be appended.
* \param field This is the name of the field (declared using AST_LIST_ENTRY())
* used to link entries of this list together.
+ *
* Note: The source list (the \a list parameter) will be empty after
* calling this macro (the list entries are \b moved to the target list).
  */
 #define AST_LIST_APPEND_LIST(head, list, field) do {                   \
       if (!(head)->first) {                                            \
@@ -775,13 +779,13 @@ struct {                                                          \
 #define AST_RWLIST_APPEND_LIST AST_LIST_APPEND_LIST
 
 /*!
-  \brief Removes and returns the head entry from a list.
-  \param head This is a pointer to the list head structure
-  \param field This is the name of the field (declared using AST_LIST_ENTRY())
-  used to link entries of this list together.
-
-  Removes the head entry from the list, and returns a pointer to it.
-  This macro is safe to call on an empty list.
* \brief Removes and returns the head entry from a list.
* \param head This is a pointer to the list head structure
* \param field This is the name of the field (declared using AST_LIST_ENTRY())
* used to link entries of this list together.
+ *
* Removes the head entry from the list, and returns a pointer to it.
* This macro is safe to call on an empty list.
  */
 #define AST_LIST_REMOVE_HEAD(head, field) ({                           \
                typeof((head)->first) cur = (head)->first;              \
@@ -797,12 +801,12 @@ struct {                                                          \
 #define AST_RWLIST_REMOVE_HEAD AST_LIST_REMOVE_HEAD
 
 /*!
-  \brief Removes a specific entry from a list.
-  \param head This is a pointer to the list head structure
-  \param elm This is a pointer to the entry to be removed.
-  \param field This is the name of the field (declared using AST_LIST_ENTRY())
-  used to link entries of this list together.
-  \warning The removed entry is \b not freed nor modified in any way.
* \brief Removes a specific entry from a list.
* \param head This is a pointer to the list head structure
* \param elm This is a pointer to the entry to be removed.
* \param field This is the name of the field (declared using AST_LIST_ENTRY())
* used to link entries of this list together.
* \warning The removed entry is \b not freed nor modified in any way.
  */
 #define AST_LIST_REMOVE(head, elm, field) ({                           \
        __typeof(elm) __res = NULL; \
index 139f54cad0a599aaad5832b52a9203a58250938e..bba20c85df4f728e0cca9d45118553b360acdabc 100644 (file)
@@ -1335,6 +1335,7 @@ static inline int _ast_rwlock_wrlock(ast_rwlock_t *t, const char *name,
                if (lt->reentrancy) {
                        ast_reentrancy_lock(lt);
                        bt = &lt->backtrace[lt->reentrancy-1];
+                       ast_reentrancy_unlock(lt);
                } else {
                        bt = NULL;
                }
@@ -1353,7 +1354,7 @@ static inline int _ast_rwlock_wrlock(ast_rwlock_t *t, const char *name,
        return res;
 }
 
-#define ast_rwlock_timedrdlock(a,b) \
+#define ast_rwlock_timedrdlock(a, b) \
        _ast_rwlock_timedrdlock(a, # a, b, __FILE__, __LINE__, __PRETTY_FUNCTION__)
 
 static inline int _ast_rwlock_timedrdlock(ast_rwlock_t *t, const char *name,
@@ -1413,6 +1414,7 @@ static inline int _ast_rwlock_timedrdlock(ast_rwlock_t *t, const char *name,
                if (lt->reentrancy) {
                        ast_reentrancy_lock(lt);
                        bt = &lt->backtrace[lt->reentrancy-1];
+                       ast_reentrancy_unlock(lt);
                } else {
                        bt = NULL;
                }
@@ -1431,7 +1433,7 @@ static inline int _ast_rwlock_timedrdlock(ast_rwlock_t *t, const char *name,
        return res;
 }
 
-#define ast_rwlock_timedwrlock(a,b) \
+#define ast_rwlock_timedwrlock(a, b) \
        _ast_rwlock_timedwrlock(a, # a, b, __FILE__, __LINE__, __PRETTY_FUNCTION__)
 
 static inline int _ast_rwlock_timedwrlock(ast_rwlock_t *t, const char *name,
@@ -1491,6 +1493,7 @@ static inline int _ast_rwlock_timedwrlock(ast_rwlock_t *t, const char *name,
                if (lt->reentrancy) {
                        ast_reentrancy_lock(lt);
                        bt = &lt->backtrace[lt->reentrancy-1];
+                       ast_reentrancy_unlock(lt);
                } else {
                        bt = NULL;
                }
index 8528ab53a3f14585762309143f19d981183549c2..7e65d9568812ca5f7d6afdf9ad096fc780ff8ca4 100644 (file)
@@ -1033,7 +1033,7 @@ int ast_app_group_update(struct ast_channel *old, struct ast_channel *new)
                        ast_free(gi);
                }
        }
-       AST_RWLIST_TRAVERSE_SAFE_END
+       AST_RWLIST_TRAVERSE_SAFE_END;
        AST_RWLIST_UNLOCK(&groups);
 
        return 0;
index baf370dc7326c6d6e5ca7ca8f3c2a11ad035eb14..6f6e08b3a86c1545f2d38dbf6d72fc973eb9700e 100644 (file)
@@ -592,7 +592,7 @@ static struct ast_frame *audio_audiohook_write_list(struct ast_channel *chan, st
                ast_audiohook_write_frame(audiohook, direction, middle_frame);
                ast_audiohook_unlock(audiohook);
        }
-       AST_LIST_TRAVERSE_SAFE_END
+       AST_LIST_TRAVERSE_SAFE_END;
 
        /* If this frame is being written out to the channel then we need to use whisper sources */
        if (direction == AST_AUDIOHOOK_DIRECTION_WRITE && !AST_LIST_EMPTY(&audiohook_list->whisper_list)) {
@@ -615,7 +615,7 @@ static struct ast_frame *audio_audiohook_write_list(struct ast_channel *chan, st
                        }
                        ast_audiohook_unlock(audiohook);
                }
-               AST_LIST_TRAVERSE_SAFE_END
+               AST_LIST_TRAVERSE_SAFE_END;
                /* We take all of the combined whisper sources and combine them into the audio being written out */
                for (i = 0, data1 = middle_frame->data.ptr, data2 = combine_buf; i < samples; i++, data1++, data2++)
                        ast_slinear_saturated_add(data1, data2);
@@ -638,7 +638,7 @@ static struct ast_frame *audio_audiohook_write_list(struct ast_channel *chan, st
                        audiohook->manipulate_callback(audiohook, chan, middle_frame, direction);
                        ast_audiohook_unlock(audiohook);
                }
-               AST_LIST_TRAVERSE_SAFE_END
+               AST_LIST_TRAVERSE_SAFE_END;
                end_frame = middle_frame;
        }
 
index ff9c68f6008a7201c355969b9d82bbdff15cf5b2..3a19ee6f529e0dd4c113029f47c92a39113ca184 100644 (file)
@@ -2623,7 +2623,7 @@ static struct ast_frame *__ast_read(struct ast_channel *chan, int dropaudio)
                        AST_LIST_REMOVE_CURRENT(frame_list);
                        break;
                }
-               AST_LIST_TRAVERSE_SAFE_END
+               AST_LIST_TRAVERSE_SAFE_END;
                
                if (!f) {
                        /* There were no acceptable frames on the readq. */
index e0461dbb1296933999f2dfa394f492eeaec88168..cfa3fb03dd6e517ee7c0c331741412d7a60849b3 100644 (file)
@@ -3260,7 +3260,7 @@ static int park_exec_full(struct ast_channel *chan, void *data, struct ast_parki
                        break;
                }
        }
-       AST_LIST_TRAVERSE_SAFE_END
+       AST_LIST_TRAVERSE_SAFE_END;
        AST_LIST_UNLOCK(&parkinglot->parkings);
 
        if (pu) {
index 6dea2fed63b57d7d68eafc3508ed21882485a445..c520560acdd13d28ab8a54587960ec66f335d2cc 100644 (file)
@@ -399,7 +399,7 @@ void ast_http_uri_unlink_all_with_key(const char *key)
                        ast_free(urih);
                }
        }
-       AST_RWLIST_TRAVERSE_SAFE_END
+       AST_RWLIST_TRAVERSE_SAFE_END;
        AST_RWLIST_UNLOCK(&uris);
 }
 
index eb7e43f5fc74a7cf0f2c8994c4e3c2d42fc830ce..6008e89a7b87495a73ea814ec4b6d491df5cc737 100644 (file)
@@ -3352,7 +3352,7 @@ int ast_manager_unregister(char *action)
                        break;
                }
        }
-       AST_RWLIST_TRAVERSE_SAFE_END
+       AST_RWLIST_TRAVERSE_SAFE_END;
        AST_RWLIST_UNLOCK(&actions);
 
        return 0;
index 04cc1480358169af9cd5f52c1e4bb0bda6a313d1..b4f6599d9e9da6198118110535f9ca64f17978ea 100644 (file)
@@ -3473,7 +3473,7 @@ int ast_extension_state_del(int id, ast_state_cb_type callback)
                                break;
                        }
                }
-               AST_LIST_TRAVERSE_SAFE_END
+               AST_LIST_TRAVERSE_SAFE_END;
        } else { /* callback with extension, find the callback based on ID */
                struct ast_hint *hint;
                AST_RWLIST_TRAVERSE(&hints, hint, list) {
@@ -3483,7 +3483,7 @@ int ast_extension_state_del(int id, ast_state_cb_type callback)
                                        break;
                                }
                        }
-                       AST_LIST_TRAVERSE_SAFE_END
+                       AST_LIST_TRAVERSE_SAFE_END;
 
                        if (p_cur)
                                break;
@@ -7726,7 +7726,7 @@ void __ast_context_destroy(struct ast_context *list, struct ast_hashtab *context
                                        ast_free(sw);
                                }
                        }
-                       AST_LIST_TRAVERSE_SAFE_END
+                       AST_LIST_TRAVERSE_SAFE_END;
 
                        if (tmp->root_table) { /* it is entirely possible that the context is EMPTY */
                                exten_iter = ast_hashtab_start_traversal(tmp->root_table);