httpd-test-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Aaron Bannert <aa...@clove.org>
Subject Re: [PATCH] flood: baseurl support
Date Fri, 13 Sep 2002 16:41:04 GMT
On Fri, Sep 13, 2002 at 06:06:55PM +0200, Jacek Prucia wrote:
> 
> Following patch makes such config possible:
> 
> <urllist>
>    <baseurl>http://www.site.com</baseurl>
>    <url>/</url>
>    <url>/foo.html</url>
>    <url>/cgi-bin/bar.cgi</url>
> </urllist>

Cool!

> Might be useful if you move/install your site/app around quite a lot.
> 
> I'm not sure if I got everything right (apr_pstrcat with overleaping
> strings -- is it safe?), but if there are no objections I'll comit this
> patch later today.

I believe it's safe, since apr_pstrcat basicly first makes a pass at
counting the total string length, then allocates the whole thing at
once and then makes a second pass a memcpy'ing each string into the
final one. So yeah.

-aaron

Mime
View raw message