[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#534641: [ptex:00353] Re: mendex bug



Kakuto-san

On Sa, 07 Sep 2013, Akira Kakuto wrote:
>> Ahh, maybe the intended rewrite was to have
>> sprintf(TAIL(tmpbuff),"%s%s%s",encap_prefix,ind.p[j].enc,encap_infix);
>> sprintf(TAIL(tmpbuff),"%s%s%s",ind.p[j].page,encap_suffix,delim_n);
>> then it would work ....
>>
>> I guess that is the better fix.
>
> I also think so.

I checked, no need. The tmpbuff used here is defined in the head
of the function as tmpbuff[4096], and the above lines are the only
lines writing to it. 

So one
	snprintf(tmpbuff, 4096, ...)
should be enough.

Thanks.

Norbert

------------------------------------------------------------------------
PREINING, Norbert                               http://www.preining.info
JAIST, Japan                                 TeX Live & Debian Developer
DSA: 0x09C5B094   fp: 14DF 2E6C 0307 BE6D AD76  A9C0 D2BF 4AA3 09C5 B094
------------------------------------------------------------------------


Reply to: