httpd-test-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Chris Williams" <chriw...@cmguc.com>
Subject RE: Cookie Patch for Flood
Date Thu, 03 Jan 2002 15:02:31 GMT
Yes it is.  I will review those links for the next time.
Thanks!
Chris

> -----Original Message-----
> From: Aaron Bannert [mailto:aaron@clove.org]
> Sent: Thursday, January 03, 2002 9:56 AM
> To: test-dev@httpd.apache.org
> Subject: Re: Cookie Patch for Flood
>
>
> 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);
>
>
>


Mime
View raw message