Re: [Pan-devel] newsrc corruption theory (Was: Pan 0.91 issues)

[Christophe Lambin]

On Thu, 06 Apr, 2006 at 20:51 +0200, Charles Kerr wrote:
> More ideas on this. 4294967295 is ULONG_MAX.
> 
> task-xover.cc sets _high to max (_high, number) on each xover line
> where number is gotten with strtoul() from the xover line.  strtoul()
> handles an overflow condition by returning ULONG_MAX and setting errno
> to ERANGE.  Currently the TaskXOver::on_nntp_line() doesn't have any
> errno checking...

That doesn't appear to be it.  The bug seems to be in
DataImpl::set_xover_low. Put some debugging info in there:

set_xover_low: group=easynet.test. low=1586. newsrc was 0-1641
set_xover_low: easynet.test. newsrc is now 0-4294967295

I suspect calling _read.mark_range (0, 1586, false) when the range is
0-1641 should result in a range of 1586-1641, right ?

Regards,
Christophe