Jim Meyering wrote:
Actually, we need yet another byte at the end, and one more prior:
> When I built with ASAN and rawhide's gcc version 4.9.1 20140930
> (Red Hat 4.9.1-11) (GCC), using this command:
>
> make CFLAGS=-ggdb3 AM_CFLAGS=-fsanitize=address \
> AM_LDFLAGS='-fsanitize=address -st
On Sun, Oct 5, 2014 at 12:27 AM, Norihiro Tanaka wrote:
> Jim Meyering wrote:
> Actually, we need yet another byte at the end, and one more prior:
>
>> When I built with ASAN and rawhide's gcc version 4.9.1 20140930
>> (Red Hat 4.9.1-11) (GCC), using this command:
>>
>> make CFLAGS=-ggdb3 AM_CFL
Jim Meyering wrote:
> Thank you.
> I have added a test case to your post-transit_state buffer-length
> checking patch, and shrank the eolbytes buffer by one byte, now
> that I see this patch prevents the overrun. Here's the patch I'll
> push later today:
Thanks. Could you also add a test case in
On Sun, Oct 5, 2014 at 5:05 PM, Norihiro Tanaka wrote:
> Jim Meyering wrote:
>> Thank you.
>> I have added a test case to your post-transit_state buffer-length
>> checking patch, and shrank the eolbytes buffer by one byte, now
>> that I see this patch prevents the overrun. Here's the patch I'll
>
On Sun, Oct 5, 2014 at 5:05 PM, Norihiro Tanaka wrote:
> Jim Meyering wrote:
>> Thank you.
>> I have added a test case to your post-transit_state buffer-length
>> checking patch, and shrank the eolbytes buffer by one byte, now
>> that I see this patch prevents the overrun. Here's the patch I'll
>
Norihiro's latest fix is for a bug that is significant enough
that I would feel remiss not to make a release soon.
Here is one patch that looks worth including:
#17715 dfa: build struct dfamust on demand
Any other suggestions for changes to be included?
