From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from smtp1.rz.uni-karlsruhe.de (Debian-exim@smtp1.rz.uni-karlsruhe.de [129.13.185.217])
	by krisdoz.my.domain (8.14.3/8.14.3) with ESMTP id oBBH7GxB013842
	for <tech@mdocml.bsd.lv>; Sat, 11 Dec 2010 12:07:16 -0500 (EST)
Received: from hekate.usta.de (asta-nat.asta.uni-karlsruhe.de [172.22.63.82])
	by smtp1.rz.uni-karlsruhe.de with esmtp (Exim 4.63 #1)
	id 1PRSuh-0007NU-1g; Sat, 11 Dec 2010 18:07:15 +0100
Received: from donnerwolke.usta.de ([172.24.96.3])
	by hekate.usta.de with esmtp (Exim 4.72)
	(envelope-from <schwarze@usta.de>)
	id 1PRSuh-0006ws-0P
	for tech@mdocml.bsd.lv; Sat, 11 Dec 2010 18:07:15 +0100
Received: from iris.usta.de ([172.24.96.5] helo=usta.de)
	by donnerwolke.usta.de with esmtp (Exim 4.69)
	(envelope-from <schwarze@usta.de>)
	id 1PRSug-0002RH-Vn
	for tech@mdocml.bsd.lv; Sat, 11 Dec 2010 18:07:14 +0100
Received: from schwarze by usta.de with local (Exim 4.72)
	(envelope-from <schwarze@usta.de>)
	id 1PRSug-0006rD-V6
	for tech@mdocml.bsd.lv; Sat, 11 Dec 2010 18:07:14 +0100
Date: Sat, 11 Dec 2010 18:07:14 +0100
From: Ingo Schwarze <schwarze@usta.de>
To: tech@mdocml.bsd.lv
Subject: Re: Remaining patches
Message-ID: <20101211170714.GG27691@iris.usta.de>
References: <20101201145258.GA18473@iris.usta.de>
 <4CF662C5.8070806@bsd.lv>
 <20101202200205.GA12188@iris.usta.de>
 <4CF82337.2060203@bsd.lv>
 <20101202232111.GE12188@iris.usta.de>
 <4CFCE8A6.7000101@bsd.lv>
 <4CFCE997.6000700@bsd.lv>
 <20101206142051.GA6999@iris.usta.de>
 <4CFD0AE3.8050502@bsd.lv>
 <20101211170236.GF27691@iris.usta.de>
X-Mailinglist: mdocml-tech
Reply-To: tech@mdocml.bsd.lv
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20101211170236.GF27691@iris.usta.de>
User-Agent: Mutt/1.5.21 (2010-09-15)

> Index: main.c
> ===================================================================
> RCS file: /cvs/src/usr.bin/mandoc/main.c,v
> retrieving revision 1.61
> diff -u -p -r1.61 main.c
> --- main.c	9 Dec 2010 23:01:18 -0000	1.61
> +++ main.c	11 Dec 2010 17:01:40 -0000
> @@ -375,7 +375,7 @@ static void
>  resize_buf(struct buf *buf, size_t initial)
>  {
>  
> -	buf->sz = buf->sz ? 2 * buf->sz : initial;
> +	buf->sz = buf->sz >= initial ? 2 * buf->sz : initial;
>  	buf->buf = realloc(buf->buf, buf->sz);
>  	if (NULL == buf->buf) {
>  		perror(NULL);

Gah, that's wrong as well.
It only moves the bug from sz = 1 to sz = initial-1.

It must be

  buf->sz = buf->sz > initial/2 ? 2 * buf->sz : initial;
--
 To unsubscribe send an email to tech+unsubscribe@mdocml.bsd.lv