Your patch does this:
ap_log_rerror(APLOG_MARK, APLOG_NOERRNO|APLOG_ERR, 0, r, "DB user %s: authentication
failure for \"%s\": %s", r->user, r->uri, ap_strerror(invalid_pw, error_buf, 1024));
You can accomplish pretty much the same thing (in a more normal Apache fashion) with:
ap_log_rerror(APLOG_MARK, APLOG_ERR, invalid_pw, r, "DB user %s: authentication failure
for \"%s\", r->user, r->uri);
If you specify APLOG_ERR then pass in the status code, ap_log_rerror will call ap_strerror
under the covers.
Bill
----- Original Message -----
From: <bmartin@penguincomputing.com>
To: <new-httpd@apache.org>
Sent: Tuesday, May 02, 2000 7:46 PM
Subject: Doh!! scrap that last patch
> Doh!!
>
> It seems I forgot to change ap_strerror to use a local buffer.
> I should pay more attention :)
>
> Here is the corrected patch.
>
> Brian
|