aMule Bug Tracker - aMule
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0000731aMuleExternal Connpublic2005-12-14 08:372005-12-14 11:50
ReporterJacobo221 
Assigned ToKry 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionopen 
PlatformOSOS Version
Product Version 
Target VersionFixed in Version 
Fixed in Revision
Operating System
Summary0000731: wrong failed servers check
DescriptionCServerList::RemoveDeadServers() in ServerList.cpp checks:

cur_server->GetFailedCount() > thePrefs::GetDeadserverRetries()

where it should check

cur_server->GetFailedCount() >= thePrefs::GetDeadserverRetries()
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2005-12-14 08:37Jacobo221New Issue
2005-12-14 11:49KryStatusnew => assigned
2005-12-14 11:49KryAssigned To => Kry
2005-12-14 11:50KryStatusassigned => closed
2005-12-14 11:50KryNote Added: 0001666

Notes
(0001666)
Kry   
2005-12-14 11:50   
The check is ok, we only want to remove when we have MORE failed than the max failed.