Monotorrent may crush during change current date to some month back

[misterion]

Subj can be produced for after client torrent engine start if change current time for some month back. This is so becouse of bug in MonoTorrent\MonoTorrent.Common\TimeoutDispatcher.cs line 155:

if (interval < TimeSpan.Zero)
{
interval = TimeSpan.Zero;
}
must be:

if (interval < TimeSpan.Zero || interval.TotalMilliseconds > Int32.MaxValue)
{
interval = TimeSpan.Zero;
}

Please fix it :)

[alanmcgovern]

This code doesn't exist anymore so I can only assume that whatever the issue was, it's already fixed in git. Can you verify that the problematic code you mentioned is gone or refactored to not display the issue?

[misterion]

At least this file has still exist in master of mono torrent. We are using 0.9 version of mono torrent (mono-monotorrent-ee0b937) and this problem is actual in this version.

[alanmcgovern]

The file exists but the code you demonstrated doesn't any longer. If it does exist and I'm just not seeing it, can you fork monotorrent, make the required change and issue a pull request? http://help.github.com/pull-requests/

It's the easiest way to get changes such as this to me :)

[alanmcgovern]

I'm not sure if you closed this or I did, if I did then I didn't mean to close it yet ;)

I just checked, the issue has been fixed in git master but not in the 0.90 branch. Can you test with git master to verify the fix works as expected?