[svn-commits] mmichelson: branch mmichelson/authenticate r380815 - in /team/mmichelson/auth...

SVN commits to the Digium repositories svn-commits at lists.digium.com
Mon Feb 4 13:47:24 CST 2013


Author: mmichelson
Date: Mon Feb  4 13:47:23 2013
New Revision: 380815

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=380815
Log:
Revert the backports that were added previously.

I found the change not to be very helpful and found a different
way to do what I needed.


Modified:
    team/mmichelson/authenticate/res/pjproject/pjsip/include/pjsip/sip_auth.h
    team/mmichelson/authenticate/res/pjproject/pjsip/src/pjsip/sip_auth_server.c

Modified: team/mmichelson/authenticate/res/pjproject/pjsip/include/pjsip/sip_auth.h
URL: http://svnview.digium.com/svn/asterisk/team/mmichelson/authenticate/res/pjproject/pjsip/include/pjsip/sip_auth.h?view=diff&rev=380815&r1=380814&r2=380815
==============================================================================
--- team/mmichelson/authenticate/res/pjproject/pjsip/include/pjsip/sip_auth.h (original)
+++ team/mmichelson/authenticate/res/pjproject/pjsip/include/pjsip/sip_auth.h Mon Feb  4 13:47:23 2013
@@ -275,36 +275,6 @@
 					    const pj_str_t *acc_name,
 					    pjsip_cred_info *cred_info );
 
-/** 
- * This structure describes input param for credential lookup. 
- */ 
-typedef struct pjsip_auth_lookup_cred_param 
-{ 
-    pj_str_t realm;         /**< Realm to find the account.             */ 
-    pj_str_t acc_name;      /**< Account name to look for.              */ 
-    pjsip_rx_data *rdata;   /**< Incoming request to be authenticated.  */ 
- 
-} pjsip_auth_lookup_cred_param; 
- 
- 
-/** 
- * Type of function to lookup credential for the specified name. 
- * 
- * @param pool          Pool to initialize the credential info. 
- * @param param         The input param for credential lookup. 
- * @param cred_info     The structure to put the credential when it's found. 
- * 
- * @return              The function MUST return PJ_SUCCESS when it found 
- *                      a correct credential for the specified account and 
- *                      realm. Otherwise it may return PJSIP_EAUTHACCNOTFOUND 
- *                      or PJSIP_EAUTHACCDISABLED. 
- */ 
-typedef pj_status_t pjsip_auth_lookup_cred2( 
-                                pj_pool_t *pool, 
-                                const pjsip_auth_lookup_cred_param *param, 
-                                pjsip_cred_info *cred_info ); 
- 
-
 /** Flag to specify that server is a proxy. */
 #define PJSIP_AUTH_SRV_IS_PROXY	    1
 
@@ -316,8 +286,7 @@
     pj_str_t		     realm;	/**< Realm to serve.		    */
     pj_bool_t		     is_proxy;	/**< Will issue 407 instead of 401  */
     pjsip_auth_lookup_cred  *lookup;	/**< Lookup function.		    */
-    pjsip_auth_lookup_cred2 *lookup2;   /**< Lookup function with additional 
-                                             info in its input param.       */ 
+
 } pjsip_auth_srv;
 
 
@@ -463,48 +432,6 @@
 					  pjsip_auth_lookup_cred *lookup,
 					  unsigned options );
 
-/** 
- * This structure describes initialization settings of server authorization 
- * session. 
- */ 
-typedef struct pjsip_auth_srv_init_param 
-{ 
-    /** 
-     * Realm to be served by the server. 
-     */ 
-    const pj_str_t              *realm; 
- 
-    /** 
-     * Account lookup function. 
-     */ 
-    pjsip_auth_lookup_cred2     *lookup2; 
- 
-    /** 
-     * Options, bitmask of: 
-     * - PJSIP_AUTH_SRV_IS_PROXY: to specify that the server will authorize 
-     *   clients as a proxy server (instead of as UAS), which means that 
-     *   Proxy-Authenticate will be used instead of WWW-Authenticate. 
-     */ 
-    unsigned                     options; 
- 
-} pjsip_auth_srv_init_param; 
- 
- 
-/** 
- * Initialize server authorization session data structure to serve the  
- * specified realm and to use lookup_func function to look for the credential 
- * info.  
- * 
- * @param pool          Pool used to initialize the authentication server. 
- * @param auth_srv      The authentication server structure. 
- * @param param         The initialization param. 
- * 
- * @return              PJ_SUCCESS on success. 
- */ 
-PJ_DECL(pj_status_t) pjsip_auth_srv_init2( 
-                                    pj_pool_t *pool, 
-                                    pjsip_auth_srv *auth_srv, 
-                                    const pjsip_auth_srv_init_param *param);
 
 /**
  * Request the authorization server framework to verify the authorization 

Modified: team/mmichelson/authenticate/res/pjproject/pjsip/src/pjsip/sip_auth_server.c
URL: http://svnview.digium.com/svn/asterisk/team/mmichelson/authenticate/res/pjproject/pjsip/src/pjsip/sip_auth_server.c?view=diff&rev=380815&r1=380814&r2=380815
==============================================================================
--- team/mmichelson/authenticate/res/pjproject/pjsip/src/pjsip/sip_auth_server.c (original)
+++ team/mmichelson/authenticate/res/pjproject/pjsip/src/pjsip/sip_auth_server.c Mon Feb  4 13:47:23 2013
@@ -39,30 +39,11 @@
 					  unsigned options )
 {
     PJ_ASSERT_RETURN(pool && auth_srv && realm && lookup, PJ_EINVAL);
-    pj_bzero(auth_srv, sizeof(*auth_srv)); 
+
     pj_strdup( pool, &auth_srv->realm, realm);
     auth_srv->lookup = lookup;
     auth_srv->is_proxy = (options & PJSIP_AUTH_SRV_IS_PROXY);
- 
-    return PJ_SUCCESS; 
-} 
- 
-/* 
- * Initialize server authorization session data structure to serve the  
- * specified realm and to use lookup_func function to look for the credential  
- * info.  
- */ 
-PJ_DEF(pj_status_t) pjsip_auth_srv_init2( 
-                                    pj_pool_t *pool, 
-                                    pjsip_auth_srv *auth_srv, 
-                                    const pjsip_auth_srv_init_param *param) 
-{ 
-    PJ_ASSERT_RETURN(pool && auth_srv && param, PJ_EINVAL); 
- 
-    pj_bzero(auth_srv, sizeof(*auth_srv)); 
-    pj_strdup( pool, &auth_srv->realm, param->realm); 
-    auth_srv->lookup2 = param->lookup2; 
-    auth_srv->is_proxy = (param->options & PJSIP_AUTH_SRV_IS_PROXY);
+
     return PJ_SUCCESS;
 }
 
@@ -166,25 +147,12 @@
 	return PJSIP_EINVALIDAUTHSCHEME;
     }
 
-    if (auth_srv->lookup2) { 
-        pjsip_auth_lookup_cred_param param; 
- 
-        pj_bzero(&param, sizeof(param)); 
-        param.realm = auth_srv->realm; 
-        param.acc_name = acc_name; 
-        param.rdata = rdata; 
-        status = (*auth_srv->lookup2)(rdata->tp_info.pool, &param, &cred_info); 
-        if (status != PJ_SUCCESS) { 
-            *status_code = PJSIP_SC_FORBIDDEN; 
-            return status; 
-        }
-    } else { 
-        status = (*auth_srv->lookup)(rdata->tp_info.pool, &auth_srv->realm, 
-                                     &acc_name, &cred_info); 
-        if (status != PJ_SUCCESS) { 
-            *status_code = PJSIP_SC_FORBIDDEN; 
-            return status; 
-        } 
+    /* Find the credential information for the account. */
+    status = (*auth_srv->lookup)(rdata->tp_info.pool, &auth_srv->realm,
+				 &acc_name, &cred_info);
+    if (status != PJ_SUCCESS) {
+	*status_code = PJSIP_SC_FORBIDDEN;
+	return status;
     }
 
     /* Authenticate with the specified credential. */




More information about the svn-commits mailing list