STABLE 20040522 MAJOR problems!

[The Doctor]

On Sun, May 23, 2004 at 05:07:19PM -0700, Russ Allbery wrote:
> Jeffrey M Vinocur <jeff at litech.org> writes:
> 
> > Committed just now to CURRENT.
> 
> > The patch didn't apply as is to STABLE, so I thought I'd better confirm 
> > it's applicable.  You're happy with this for STABLE?
> 
> >     if (fp->NeedHeader) {
> >       buffer_append(overview, hp->Name, hp->Size);
> >       buffer_append(overview, COLONSPACE, strlen(COLONSPACE));
> >     }
> >     if (overview->used + overview->left + len > overview->size)
> >         buffer_resize(overview, overview->size + len);
> > -    for (i = 0, q = &overview->data[overview->left] ; i < len ; p++, q++, i++) {
> > +    for (i = 0, q = &overview->data[overview->left] ; i < len ; p++, i++) {
> >       /* we can replace consecutive '\r', '\n' and '\r' with one ' ' here */
> >       if (*p == '\t' || *p == '\n' || *p == '\r')
> >         *q = ' ';
> >       else
> 
> That doesn't look right; that looks like the old code.  Are you sure your
> STABLE is up to date?
>

STABLE nnrpd was also affected.  Does the above address that?
 
> -- 
> Russ Allbery (rra at stanford.edu)             <http://www.eyrie.org/~eagle/>
> 
>     Please send questions to the list rather than mailing me directly.
>      <http://www.eyrie.org/~eagle/faqs/questions.html> explains why.
> 

-- 
Member - Liberal International	
This is doctor at nl2k.ab.ca	Ici doctor at nl2k.ab.ca
God Queen and country! Beware Anti-Christ rising!
Canada - Do not be conned by Mulroney Harper nor Layoffs Layton!! Liberal!!!