httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bill Stoddard" <stodd...@raleigh.ibm.com>
Subject Re: Doh!! scrap that last patch
Date Wed, 03 May 2000 13:18:39 GMT
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


Mime
View raw message