[svn-commits] rmudgett: branch 13 r432787 - in /branches/13: include/asterisk/ res/res_pjsi...

SVN commits to the Digium repositories svn-commits at lists.digium.com
Wed Mar 11 11:38:23 CDT 2015


Author: rmudgett
Date: Wed Mar 11 11:38:20 2015
New Revision: 432787

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=432787
Log:
res_pjsip: Move internal init/destroy prototypes to private header file.

Done as a separate commit from a finding in
https://reviewboard.asterisk.org/r/4467/

Modified:
    branches/13/include/asterisk/res_pjsip.h
    branches/13/res/res_pjsip/config_domain_aliases.c
    branches/13/res/res_pjsip/include/res_pjsip_private.h

Modified: branches/13/include/asterisk/res_pjsip.h
URL: http://svnview.digium.com/svn/asterisk/branches/13/include/asterisk/res_pjsip.h?view=diff&rev=432787&r1=432786&r2=432787
==============================================================================
--- branches/13/include/asterisk/res_pjsip.h (original)
+++ branches/13/include/asterisk/res_pjsip.h Wed Mar 11 11:38:20 2015
@@ -842,46 +842,6 @@
 struct ast_sorcery *ast_sip_get_sorcery(void);
 
 /*!
- * \brief Initialize transport support on a sorcery instance
- *
- * \retval -1 failure
- * \retval 0 success
- */
-int ast_sip_initialize_sorcery_transport(void);
-
-/*!
- * \brief Destroy transport support on a sorcery instance
- *
- * \retval -1 failure
- * \retval 0 success
- */
-int ast_sip_destroy_sorcery_transport(void);
-
-/*!
- * \brief Initialize qualify support on a sorcery instance
- *
- * \retval -1 failure
- * \retval 0 success
- */
-int ast_sip_initialize_sorcery_qualify(void);
-
-/*!
- * \brief Initialize location support on a sorcery instance
- *
- * \retval -1 failure
- * \retval 0 success
- */
-int ast_sip_initialize_sorcery_location(void);
-
-/*!
- * \brief Destroy location support on a sorcery instance
- *
- * \retval -1 failure
- * \retval 0 success
- */
-int ast_sip_destroy_sorcery_location(void);
-
-/*!
  * \brief Retrieve a named AOR
  *
  * \param aor_name Name of the AOR
@@ -975,30 +935,6 @@
 int ast_sip_location_delete_contact(struct ast_sip_contact *contact);
 
 /*!
- * \brief Initialize domain aliases support on a sorcery instance
- *
- * \retval -1 failure
- * \retval 0 success
- */
-int ast_sip_initialize_sorcery_domain_alias(void);
-
-/*!
- * \brief Initialize authentication support on a sorcery instance
- *
- * \retval -1 failure
- * \retval 0 success
- */
-int ast_sip_initialize_sorcery_auth(void);
-
-/*!
- * \brief Destroy authentication support on a sorcery instance
- *
- * \retval -1 failure
- * \retval 0 success
- */
-int ast_sip_destroy_sorcery_auth(void);
-
-/*!
  * \brief Callback called when an outbound request with authentication credentials is to be sent in dialog
  *
  * This callback will have the created request on it. The callback's purpose is to do any extra
@@ -1032,26 +968,6 @@
  */
 int ast_sip_dialog_setup_outbound_authentication(pjsip_dialog *dlg, const struct ast_sip_endpoint *endpoint,
 		ast_sip_dialog_outbound_auth_cb cb, void *user_data);
-
-/*!
- * \brief Initialize the distributor module
- *
- * The distributor module is responsible for taking an incoming
- * SIP message and placing it into the threadpool. Once in the threadpool,
- * the distributor will perform endpoint lookups and authentication, and
- * then distribute the message up the stack to any further modules.
- *
- * \retval -1 Failure
- * \retval 0 Success
- */
-int ast_sip_initialize_distributor(void);
-
-/*!
- * \brief Destruct the distributor module.
- *
- * Unregisters pjsip modules and cleans up any allocated resources.
- */
-void ast_sip_destroy_distributor(void);
 
 /*!
  * \brief Retrieves a reference to the artificial auth.
@@ -1619,28 +1535,8 @@
  */
 void ast_sip_report_mem_limit(struct ast_sip_endpoint *endpoint, pjsip_rx_data *rdata);
 
-void ast_sip_initialize_global_headers(void);
-void ast_sip_destroy_global_headers(void);
-
 int ast_sip_add_global_request_header(const char *name, const char *value, int replace);
 int ast_sip_add_global_response_header(const char *name, const char *value, int replace);
-
-/*!
- * \brief Initialize global type on a sorcery instance
- *
- * \retval -1 failure
- * \retval 0 success
- */
-int ast_sip_initialize_sorcery_global(void);
-
-/*!
- * \brief Destroy global type on a sorcery instance
- * \since 13.3.0
- *
- * \retval -1 failure
- * \retval 0 success
- */
-int ast_sip_destroy_sorcery_global(void);
 
 /*!
  * \brief Retrieves the value associated with the given key.

Modified: branches/13/res/res_pjsip/config_domain_aliases.c
URL: http://svnview.digium.com/svn/asterisk/branches/13/res/res_pjsip/config_domain_aliases.c?view=diff&rev=432787&r1=432786&r2=432787
==============================================================================
--- branches/13/res/res_pjsip/config_domain_aliases.c (original)
+++ branches/13/res/res_pjsip/config_domain_aliases.c Wed Mar 11 11:38:20 2015
@@ -21,6 +21,7 @@
 #include "pjsip.h"
 #include "pjlib.h"
 #include "asterisk/res_pjsip.h"
+#include "include/res_pjsip_private.h"
 #include "asterisk/logger.h"
 #include "asterisk/sorcery.h"
 

Modified: branches/13/res/res_pjsip/include/res_pjsip_private.h
URL: http://svnview.digium.com/svn/asterisk/branches/13/res/res_pjsip/include/res_pjsip_private.h?view=diff&rev=432787&r1=432786&r2=432787
==============================================================================
--- branches/13/res/res_pjsip/include/res_pjsip_private.h (original)
+++ branches/13/res/res_pjsip/include/res_pjsip_private.h Wed Mar 11 11:38:20 2015
@@ -1,5 +1,5 @@
 /*
- * res_pjsip.h
+ * res_pjsip_private.h
  *
  *  Created on: Jan 25, 2013
  *      Author: mjordan
@@ -8,6 +8,12 @@
 #ifndef RES_PJSIP_PRIVATE_H_
 #define RES_PJSIP_PRIVATE_H_
 
+/*!
+ * \todo XXX Functions prototyped in this file that begin with "ast_sip_"
+ * need to be renamed so res_pjsip.so does not export the names outside
+ * of the module.
+ */
+
 #include "asterisk/module.h"
 #include "asterisk/compat.h"
 
@@ -15,21 +21,154 @@
 struct ast_threadpool_options;
 
 /*!
+ * \internal
  * \brief Initialize the configuration for res_pjsip
  */
 int ast_res_pjsip_initialize_configuration(const struct ast_module_info *ast_module_info);
 
 /*!
+ * \internal
  * \brief Annihilate the configuration objects
  */
 void ast_res_pjsip_destroy_configuration(void);
 
 /*!
+ * \internal
  * \brief Reload the configuration
  */
 int ast_res_pjsip_reload_configuration(void);
 
 /*!
+ * \internal
+ * \brief Initialize transport support on a sorcery instance
+ *
+ * \retval -1 failure
+ * \retval 0 success
+ */
+int ast_sip_initialize_sorcery_transport(void);
+
+/*!
+ * \internal
+ * \brief Destroy transport support on a sorcery instance
+ *
+ * \retval -1 failure
+ * \retval 0 success
+ */
+int ast_sip_destroy_sorcery_transport(void);
+
+/*!
+ * \internal
+ * \brief Initialize qualify support on a sorcery instance
+ *
+ * \retval -1 failure
+ * \retval 0 success
+ */
+int ast_sip_initialize_sorcery_qualify(void);
+
+/*!
+ * \internal
+ * \brief Initialize location support on a sorcery instance
+ *
+ * \retval -1 failure
+ * \retval 0 success
+ */
+int ast_sip_initialize_sorcery_location(void);
+
+/*!
+ * \internal
+ * \brief Destroy location support on a sorcery instance
+ *
+ * \retval -1 failure
+ * \retval 0 success
+ */
+int ast_sip_destroy_sorcery_location(void);
+
+/*!
+ * \internal
+ * \brief Initialize domain aliases support on a sorcery instance
+ *
+ * \retval -1 failure
+ * \retval 0 success
+ */
+int ast_sip_initialize_sorcery_domain_alias(void);
+
+/*!
+ * \internal
+ * \brief Initialize authentication support on a sorcery instance
+ *
+ * \retval -1 failure
+ * \retval 0 success
+ */
+int ast_sip_initialize_sorcery_auth(void);
+
+/*!
+ * \internal
+ * \brief Destroy authentication support on a sorcery instance
+ *
+ * \retval -1 failure
+ * \retval 0 success
+ */
+int ast_sip_destroy_sorcery_auth(void);
+
+/*!
+ * \internal
+ * \brief Initialize the distributor module
+ *
+ * The distributor module is responsible for taking an incoming
+ * SIP message and placing it into the threadpool. Once in the threadpool,
+ * the distributor will perform endpoint lookups and authentication, and
+ * then distribute the message up the stack to any further modules.
+ *
+ * \retval -1 Failure
+ * \retval 0 Success
+ */
+int ast_sip_initialize_distributor(void);
+
+/*!
+ * \internal
+ * \brief Destruct the distributor module.
+ *
+ * Unregisters pjsip modules and cleans up any allocated resources.
+ */
+void ast_sip_destroy_distributor(void);
+
+/*!
+ * \internal
+ * \brief Initialize global type on a sorcery instance
+ *
+ * \retval -1 failure
+ * \retval 0 success
+ */
+int ast_sip_initialize_sorcery_global(void);
+
+/*!
+ * \internal
+ * \brief Destroy global type on a sorcery instance
+ * \since 13.3.0
+ *
+ * \retval -1 failure
+ * \retval 0 success
+ */
+int ast_sip_destroy_sorcery_global(void);
+
+/*!
+ * \internal
+ * \brief Initialize global headers support
+ *
+ * \return Nothing
+ */
+void ast_sip_initialize_global_headers(void);
+
+/*!
+ * \internal
+ * \brief Destroy global headers support
+ *
+ * \return Nothing
+ */
+void ast_sip_destroy_global_headers(void);
+
+/*!
+ * \internal
  * \brief Initialize OPTIONS request handling.
  *
  * XXX This currently includes qualifying peers. It shouldn't.
@@ -44,6 +183,7 @@
 int ast_res_pjsip_init_options_handling(int reload);
 
 /*!
+ * \internal
  * \brief Initialize transport storage for contacts.
  *
  * \retval 0 on success
@@ -52,6 +192,7 @@
 int ast_res_pjsip_init_contact_transports(void);
 
 /*!
+ * \internal
  * \brief Initialize outbound authentication support
  *
  * \retval 0 Success
@@ -60,6 +201,7 @@
 int internal_sip_initialize_outbound_authentication(void);
 
 /*!
+ * \internal
  * \brief Destroy outbound authentication support
  *
  * \retval 0 Success
@@ -68,6 +210,7 @@
 void internal_sip_destroy_outbound_authentication(void);
 
 /*!
+ * \internal
  * \brief Initialize system configuration
  *
  * \retval 0 Success
@@ -76,16 +219,19 @@
 int ast_sip_initialize_system(void);
 
 /*!
+ * \internal
  * \brief Destroy system configuration
  */
 void ast_sip_destroy_system(void);
 
 /*!
+ * \internal
  * \brief Initialize nameserver configuration
  */
 void ast_sip_initialize_dns(void);
 
 /*!
+ * \internal
  * \brief Initialize global configuration
  *
  * \retval 0 Success
@@ -94,16 +240,19 @@
 int ast_sip_initialize_global(void);
 
 /*!
+ * \internal
  * \brief Clean up res_pjsip options handling
  */
 void ast_res_pjsip_cleanup_options_handling(void);
 
 /*!
+ * \internal
  * \brief Get threadpool options
  */
 void sip_get_threadpool_options(struct ast_threadpool_options *threadpool_options);
 
 /*!
+ * \internal
  * \brief Retrieve the name of the default outbound endpoint.
  *
  * \note This returns a memory allocated copy of the name that
@@ -115,28 +264,33 @@
 char *ast_sip_global_default_outbound_endpoint(void);
 
 /*!
+ * \internal
  * \brief Functions for initializing and destroying the CLI.
  */
 int ast_sip_initialize_cli(void);
 void ast_sip_destroy_cli(void);
 
 /*!
- * \internal \brief Used by res_pjsip.so to register a service without adding a self reference
+ * \internal
+ * \brief Used by res_pjsip.so to register a service without adding a self reference
  */
 int internal_sip_register_service(pjsip_module *module);
 
 /*!
- * \internal \brief Used by res_pjsip.so to unregister a service without removing a self reference
+ * \internal
+ * \brief Used by res_pjsip.so to unregister a service without removing a self reference
  */
 int internal_sip_unregister_service(pjsip_module *module);
 
 /*!
- * \internal \brief Used by res_pjsip.so to register an endpoint formatter without adding a self reference
+ * \internal
+ * \brief Used by res_pjsip.so to register an endpoint formatter without adding a self reference
  */
 void internal_sip_register_endpoint_formatter(struct ast_sip_endpoint_formatter *obj);
 
 /*!
- * \internal \brief Used by res_pjsip.so to unregister a endpoint formatter without removing a self reference
+ * \internal
+ * \brief Used by res_pjsip.so to unregister a endpoint formatter without removing a self reference
  */
 int internal_sip_unregister_endpoint_formatter(struct ast_sip_endpoint_formatter *obj);
 




More information about the svn-commits mailing list