On Sun, Feb 18, 2001 at 06:31:03PM -0600, William A. Rowe, Jr. wrote:
> static void fix_hostname(request_rec *r)
> {
> char *host = ap_palloc(r->pool, strlen(r->hostname) + 1);
> const char *src;
> char *dst;
>
> /* check and copy the host part */
> src = r->hostname;
> /* unescape the hostname first */
> if (ap_unescape_url(src) != OK) {
>
>
> This looks like badness ... ap_unescape_url mucks src, but r->hostname
> and src are declared const. This is unhealthy, no?
Perhaps making a local copy, unescape that, and re-assign it to r->hostname
at the beginning? Dean?
Martin
--
<[EMAIL PROTECTED]> | Fujitsu Siemens
<[EMAIL PROTECTED]> | 81730 Munich, Germany
