You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by ni...@apache.org on 2005/10/18 11:01:06 UTC

svn commit: r326054 - /httpd/httpd/trunk/modules/aaa/mod_authn_dbd.c

Author: niq
Date: Tue Oct 18 02:01:02 2005
New Revision: 326054

URL: http://svn.apache.org/viewcvs?rev=326054&view=rev
Log:
Faffing about with colon-delimiters (originally copied from mod_authn_dbm)
is deeply superfluous in an SQL context.

Modified:
    httpd/httpd/trunk/modules/aaa/mod_authn_dbd.c

Modified: httpd/httpd/trunk/modules/aaa/mod_authn_dbd.c
URL: http://svn.apache.org/viewcvs/httpd/httpd/trunk/modules/aaa/mod_authn_dbd.c?rev=326054&r1=326053&r2=326054&view=diff
==============================================================================
--- httpd/httpd/trunk/modules/aaa/mod_authn_dbd.c (original)
+++ httpd/httpd/trunk/modules/aaa/mod_authn_dbd.c Tue Oct 18 02:01:02 2005
@@ -88,7 +88,6 @@
 {
     apr_status_t rv;
     const char *dbd_password = NULL;
-    char *colon_pw;
     apr_dbd_prepared_t *statement;
     apr_dbd_results_t *res = NULL;
     apr_dbd_row_t *row = NULL;
@@ -136,11 +135,6 @@
         return AUTH_USER_NOT_FOUND;
     }
 
-    colon_pw = ap_strchr(dbd_password, ':');
-    if (colon_pw) {
-        *colon_pw = '\0';
-    }
-
     rv = apr_password_validate(password, dbd_password);
 
     if (rv != APR_SUCCESS) {
@@ -154,7 +148,6 @@
 {
     apr_status_t rv;
     const char *dbd_hash = NULL;
-    char *colon_hash;
     apr_dbd_prepared_t *statement;
     apr_dbd_results_t *res = NULL;
     apr_dbd_row_t *row = NULL;
@@ -200,13 +193,7 @@
         return AUTH_USER_NOT_FOUND;
     }
 
-    colon_hash = ap_strchr(dbd_hash, ':');
-    if (colon_hash) {
-        *colon_hash = '\0';
-    }
-
     *rethash = apr_pstrdup(r->pool, dbd_hash);
-
     return AUTH_USER_FOUND;
 }
 static void authn_dbd_hooks(apr_pool_t *p)