httpd-cvs mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From yla...@apache.org
Subject svn commit: r1585434 - in /httpd/httpd/trunk: docs/log-message-tags/next-number modules/aaa/mod_auth_form.c
Date Mon, 07 Apr 2014 09:10:51 GMT
Author: ylavic
Date: Mon Apr  7 09:10:51 2014
New Revision: 1585434

URL: http://svn.apache.org/r1585434
Log:
mod_auth_form: revert r1585431 (don't crash when no AuthFormProvider is configured).
This cannot happen in trunk, 2.4.x needs authenticate_form_post_config() to be merged.

Modified:
    httpd/httpd/trunk/docs/log-message-tags/next-number
    httpd/httpd/trunk/modules/aaa/mod_auth_form.c

Modified: httpd/httpd/trunk/docs/log-message-tags/next-number
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/docs/log-message-tags/next-number?rev=1585434&r1=1585433&r2=1585434&view=diff
==============================================================================
--- httpd/httpd/trunk/docs/log-message-tags/next-number (original)
+++ httpd/httpd/trunk/docs/log-message-tags/next-number Mon Apr  7 09:10:51 2014
@@ -1 +1 @@
-2620
+2615

Modified: httpd/httpd/trunk/modules/aaa/mod_auth_form.c
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/aaa/mod_auth_form.c?rev=1585434&r1=1585433&r2=1585434&view=diff
==============================================================================
--- httpd/httpd/trunk/modules/aaa/mod_auth_form.c (original)
+++ httpd/httpd/trunk/modules/aaa/mod_auth_form.c Mon Apr  7 09:10:51 2014
@@ -909,14 +909,6 @@ static int authenticate_form_authn(reque
         return HTTP_INTERNAL_SERVER_ERROR;
     }
 
-    /* We need an authentication provider. */
-    if (!conf->providers) {
-        ap_log_rerror(APLOG_MARK, APLOG_ERR,
-                      0, r, APLOGNO(02617) "need AuthFormProvider: %s",
-                      r->uri);
-        return HTTP_INTERNAL_SERVER_ERROR;
-    }
-
     /* We need an authentication realm. */
     if (!ap_auth_name(r)) {
         ap_log_rerror(APLOG_MARK, APLOG_ERR,
@@ -1290,7 +1282,7 @@ static int authenticate_form_post_config
         ap_session_get_fn = APR_RETRIEVE_OPTIONAL_FN(ap_session_get);
         ap_session_set_fn = APR_RETRIEVE_OPTIONAL_FN(ap_session_set);
         if (!ap_session_load_fn || !ap_session_get_fn || !ap_session_set_fn) {
-            ap_log_error(APLOG_MARK, APLOG_CRIT, 0, NULL, APLOGNO(02618)
+            ap_log_error(APLOG_MARK, APLOG_CRIT, 0, NULL, APLOGNO()
                     "You must load mod_session to enable the mod_auth_form "
                                        "functions");
             return !OK;
@@ -1301,7 +1293,7 @@ static int authenticate_form_post_config
         ap_request_insert_filter_fn = APR_RETRIEVE_OPTIONAL_FN(ap_request_insert_filter);
         ap_request_remove_filter_fn = APR_RETRIEVE_OPTIONAL_FN(ap_request_remove_filter);
         if (!ap_request_insert_filter_fn || !ap_request_remove_filter_fn) {
-            ap_log_error(APLOG_MARK, APLOG_CRIT, 0, NULL, APLOGNO(02619)
+            ap_log_error(APLOG_MARK, APLOG_CRIT, 0, NULL, APLOGNO()
                     "You must load mod_request to enable the mod_auth_form "
                                        "functions");
             return !OK;



Mime
View raw message