David Hinkle wrote:
> Maybe fixing this will become trivial for someone with more experience
> First invalid write:
>
> ==30539== Invalid write of size 1
> ==30539== at 0x401BD48: memcpy (in
> /usr/local/lib/valgrind/x86-linux/vgpreload_memcheck.so)
> ==30539== by 0x4064E74: SPF_dns_resolv_lookup (spf_dns_resolv.c:404)
Hmm, I wonder if it would help to change src/libspf2/spf_dns_resolv.c
line 399 from
while ( rdlen > 0 )
to
while ( rdlen > 1 )
I need to check it when I have time...
Actually, I would *very* much like to have a check added after line 401,
something like
if (len >= rdlen) {
complan_very_very_loudly();
len=rdlen-1;
}
Eugene
-------------------------------------------
-----------------------------------------------------------------------
To unsubscribe, change your address, or temporarily deactivate your
subscription,
please go to http://v2.listbox.com/member/?member_id=1311533&id_secret=57044523-85f1f8
Powered by Listbox: http://www.listbox.com
> Maybe fixing this will become trivial for someone with more experience
> First invalid write:
>
> ==30539== Invalid write of size 1
> ==30539== at 0x401BD48: memcpy (in
> /usr/local/lib/valgrind/x86-linux/vgpreload_memcheck.so)
> ==30539== by 0x4064E74: SPF_dns_resolv_lookup (spf_dns_resolv.c:404)
Hmm, I wonder if it would help to change src/libspf2/spf_dns_resolv.c
line 399 from
while ( rdlen > 0 )
to
while ( rdlen > 1 )
I need to check it when I have time...
Actually, I would *very* much like to have a check added after line 401,
something like
if (len >= rdlen) {
complan_very_very_loudly();
len=rdlen-1;
}
Eugene
-------------------------------------------
-----------------------------------------------------------------------
To unsubscribe, change your address, or temporarily deactivate your
subscription,
please go to http://v2.listbox.com/member/?member_id=1311533&id_secret=57044523-85f1f8
Powered by Listbox: http://www.listbox.com