wornvirtue
/
premiere-libtorrent
forked from premiere/premiere-libtorrent
1
0
Fork 0
Code Pull Requests Activity

fixed rare bug where a torrent could be put back into downloading state when finishing checking files if it already finished by then

This commit is contained in:
Arvid Norberg 2008-10-05 01:44:57 +00:00
parent c65919f43d
commit 8f665e949f
1 changed files with 16 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
src/torrent.cpp
View File

@ -3299,6 +3299,9 @@ namespace libtorrent
{ {
INVARIANT_CHECK; INVARIANT_CHECK;
TORRENT_ASSERT(is_finished());
TORRENT_ASSERT(m_state != torrent_status::finished && m_state != torrent_status::seeding);
if (alerts().should_post<torrent_finished_alert>()) if (alerts().should_post<torrent_finished_alert>())
{ {
alerts().post_alert(torrent_finished_alert( alerts().post_alert(torrent_finished_alert(
@ -3429,7 +3432,10 @@ namespace libtorrent
TORRENT_ASSERT(m_torrent_file->is_valid()); TORRENT_ASSERT(m_torrent_file->is_valid());
INVARIANT_CHECK; INVARIANT_CHECK;
set_state(torrent_status::downloading); // we might be finished already, in which case we should
// not switch to downloading mode.
if (m_state != torrent_status::finished)
set_state(torrent_status::downloading);
if (m_ses.m_alerts.should_post<torrent_checked_alert>()) if (m_ses.m_alerts.should_post<torrent_checked_alert>())
{ {
@ -4463,6 +4469,15 @@ namespace libtorrent
void torrent::set_state(torrent_status::state_t s) void torrent::set_state(torrent_status::state_t s)
{ {
#ifndef NDEBUG
if (s == torrent_status::seeding)
TORRENT_ASSERT(is_seed());
if (s == torrent_status::finished)
TORRENT_ASSERT(is_finished());
if (s == torrent_status::downloading && m_state == torrent_status::finished)
TORRENT_ASSERT(!is_finished());
#endif
if (m_state == s) return; if (m_state == s) return;
m_state = s; m_state = s;
if (m_ses.m_alerts.should_post<state_changed_alert>()) if (m_ses.m_alerts.should_post<state_changed_alert>())