On Thu, Feb 06, 2020 at 04:02:34PM +0100, Enrico Forestieri wrote:
> On Thu, Feb 06, 2020 at 12:59:53PM +0100, Pavel Sanda wrote:
> > I looked at the code and the buffer length seems properly handled in the 
> > reported
> > line (247) by the previous if clause.
> > What is not clear to me are two following lines, which add m+1 chars while 
> > the
> > check seem to properly handle only m chars and leaving no place for the 
> > final' \0'.
> > Anyone else can confirm?
> 
> I think the code is correct. It copies m+1 chars in order to be sure that
> the final '\0' is also copied. The fact that there is space for it is
> assured by the initial check that k+m+1 < MAX_WD_LEN.

I checked the code again and you are right.
Misread < (by <=) in the initial condition.

Pavel
-- 
lyx-devel mailing list
lyx-devel@lists.lyx.org
http://lists.lyx.org/mailman/listinfo/lyx-devel

Reply via email to