Re: Status of 1.3.18 release

[William A. Rowe, Jr.]

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?