On Thu, Jan 03, 2002 at 09:43:48AM -0500, Chris Williams wrote: > I found a bug in flood_round_robin.c. The apr_pstrcat on line 146 should > have NULL as the last argument. I am new to submitting patches so if > someone could let me know the correct way to do it, I will repost. Without > this, you get a bunch of garbage in the cookie string if there is more that > one cookie.
Thanks for the find! Typically patches are posted as context diffs. If you can manage it with your mail, it's best to attach the patches inline (as opposed to a mime/uuencoded attachment). There's some good info on this site for developers: http://dev.apache.org/ You'll be particular interested in: http://dev.apache.org/patches.html So is this what you meant? Index: flood_round_robin.c =================================================================== RCS file: /home/cvs/httpd-test/flood/flood_round_robin.c,v retrieving revision 1.19 diff -u -u -r1.19 flood_round_robin.c --- flood_round_robin.c 3 Oct 2001 01:24:01 -0000 1.19 +++ flood_round_robin.c 3 Jan 2002 14:55:43 -0000 @@ -143,7 +143,7 @@ while (cook) { if (cook != p->cookie) - cookies = apr_pstrcat(p->pool, cookies, ";"); + cookies = apr_pstrcat(p->pool, cookies, ";", NULL); cookies = apr_pstrcat(p->pool, cookies, cook->name, "=", cook->value, NULL);
