You can not select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
114 lines
4.2 KiB
114 lines
4.2 KiB
5 years ago
|
diff --git a/modules/aaa/mod_auth_digest.c b/modules/aaa/mod_auth_digest.c
|
||
|
|
index b50bcf9..5bfec82 100644
|
||
|
|
--- a/modules/aaa/mod_auth_digest.c
|
||
|
|
+++ b/modules/aaa/mod_auth_digest.c
|
||
|
|
@@ -92,7 +92,6 @@ typedef struct digest_config_struct {
|
||
|
|
int check_nc;
|
||
|
|
const char *algorithm;
|
||
|
|
char *uri_list;
|
||
|
|
- const char *ha1;
|
||
|
|
} digest_config_rec;
|
||
|
|
|
||
|
|
|
||
|
|
@@ -153,6 +152,7 @@ typedef struct digest_header_struct {
|
||
|
|
apr_time_t nonce_time;
|
||
|
|
enum hdr_sts auth_hdr_sts;
|
||
|
|
int needed_auth;
|
||
|
|
+ const char *ha1;
|
||
|
|
client_entry *client;
|
||
|
|
} digest_header_rec;
|
||
|
|
|
||
|
|
@@ -1295,7 +1295,7 @@ static int hook_note_digest_auth_failure(request_rec *r, const char *auth_type)
|
||
|
|
*/
|
||
|
|
|
||
|
|
static authn_status get_hash(request_rec *r, const char *user,
|
||
|
|
- digest_config_rec *conf)
|
||
|
|
+ digest_config_rec *conf, const char **rethash)
|
||
|
|
{
|
||
|
|
authn_status auth_result;
|
||
|
|
char *password;
|
||
|
|
@@ -1347,7 +1347,7 @@ static authn_status get_hash(request_rec *r, const char *user,
|
||
|
|
} while (current_provider);
|
||
|
|
|
||
|
|
if (auth_result == AUTH_USER_FOUND) {
|
||
|
|
- conf->ha1 = password;
|
||
|
|
+ *rethash = password;
|
||
|
|
}
|
||
|
|
|
||
|
|
return auth_result;
|
||
|
|
@@ -1474,25 +1474,24 @@ static int check_nonce(request_rec *r, digest_header_rec *resp,
|
||
|
|
|
||
|
|
/* RFC-2069 */
|
||
|
|
static const char *old_digest(const request_rec *r,
|
||
|
|
- const digest_header_rec *resp, const char *ha1)
|
||
|
|
+ const digest_header_rec *resp)
|
||
|
|
{
|
||
|
|
const char *ha2;
|
||
|
|
|
||
|
|
ha2 = ap_md5(r->pool, (unsigned char *)apr_pstrcat(r->pool, resp->method, ":",
|
||
|
|
resp->uri, NULL));
|
||
|
|
return ap_md5(r->pool,
|
||
|
|
- (unsigned char *)apr_pstrcat(r->pool, ha1, ":", resp->nonce,
|
||
|
|
- ":", ha2, NULL));
|
||
|
|
+ (unsigned char *)apr_pstrcat(r->pool, resp->ha1, ":",
|
||
|
|
+ resp->nonce, ":", ha2, NULL));
|
||
|
|
}
|
||
|
|
|
||
|
|
/* RFC-2617 */
|
||
|
|
static const char *new_digest(const request_rec *r,
|
||
|
|
- digest_header_rec *resp,
|
||
|
|
- const digest_config_rec *conf)
|
||
|
|
+ digest_header_rec *resp)
|
||
|
|
{
|
||
|
|
const char *ha1, *ha2, *a2;
|
||
|
|
|
||
|
|
- ha1 = conf->ha1;
|
||
|
|
+ ha1 = resp->ha1;
|
||
|
|
|
||
|
|
a2 = apr_pstrcat(r->pool, resp->method, ":", resp->uri, NULL);
|
||
|
|
ha2 = ap_md5(r->pool, (const unsigned char *)a2);
|
||
|
|
@@ -1505,7 +1504,6 @@ static const char *new_digest(const request_rec *r,
|
||
|
|
NULL));
|
||
|
|
}
|
||
|
|
|
||
|
|
-
|
||
|
|
static void copy_uri_components(apr_uri_t *dst,
|
||
|
|
apr_uri_t *src, request_rec *r) {
|
||
|
|
if (src->scheme && src->scheme[0] != '\0') {
|
||
|
|
@@ -1742,7 +1740,7 @@ static int authenticate_digest_user(request_rec *r)
|
||
|
|
return HTTP_UNAUTHORIZED;
|
||
|
|
}
|
||
|
|
|
||
|
|
- return_code = get_hash(r, r->user, conf);
|
||
|
|
+ return_code = get_hash(r, r->user, conf, &resp->ha1);
|
||
|
|
|
||
|
|
if (return_code == AUTH_USER_NOT_FOUND) {
|
||
|
|
ap_log_rerror(APLOG_MARK, APLOG_ERR, 0, r, APLOGNO(01790)
|
||
|
|
@@ -1772,7 +1770,7 @@ static int authenticate_digest_user(request_rec *r)
|
||
|
|
|
||
|
|
if (resp->message_qop == NULL) {
|
||
|
|
/* old (rfc-2069) style digest */
|
||
|
|
- if (strcmp(resp->digest, old_digest(r, resp, conf->ha1))) {
|
||
|
|
+ if (strcmp(resp->digest, old_digest(r, resp))) {
|
||
|
|
ap_log_rerror(APLOG_MARK, APLOG_ERR, 0, r, APLOGNO(01792)
|
||
|
|
"user %s: password mismatch: %s", r->user,
|
||
|
|
r->uri);
|
||
|
|
@@ -1802,7 +1800,7 @@ static int authenticate_digest_user(request_rec *r)
|
||
|
|
return HTTP_UNAUTHORIZED;
|
||
|
|
}
|
||
|
|
|
||
|
|
- exp_digest = new_digest(r, resp, conf);
|
||
|
|
+ exp_digest = new_digest(r, resp);
|
||
|
|
if (!exp_digest) {
|
||
|
|
/* we failed to allocate a client struct */
|
||
|
|
return HTTP_INTERNAL_SERVER_ERROR;
|
||
|
|
@@ -1886,7 +1884,7 @@ static int add_auth_info(request_rec *r)
|
||
|
|
|
||
|
|
/* calculate rspauth attribute
|
||
|
|
*/
|
||
|
|
- ha1 = conf->ha1;
|
||
|
|
+ ha1 = resp->ha1;
|
||
|
|
|
||
|
|
a2 = apr_pstrcat(r->pool, ":", resp->uri, NULL);
|
||
|
|
ha2 = ap_md5(r->pool, (const unsigned char *)a2);
|