aMule Bug Tracker - aMule
View Issue Details
0000849aMuleServerspublic2006-03-21 15:212006-05-24 03:55
eyalzo 
Kry 
normalminoralways
resolvedfixed 
2.1.1 
SVN 
0000849: Smart LowId never worked
I tested this very-old feature, and found that it does not work. Using iptables I blocked my TCP port, and found that it always connects with low-id to the first or second server.
Looking at eMule and aMule 2.1.1, I saw that it has never been changed. That's strange. People are talking about this feature all over.
Maybe I missed something?
Anyway, here is the simple fix. Seems like someone forgot to set the state to 1 before trying to connect a new server. This is why this feature does not work when starting aMule.
The fix:

void CServerConnect::ConnectToServer(CServer* server, bool multiconnect)
{
...
    //eyal 21-03-2006: fix smart-lowid
    if(thePrefs::GetSmartIdCheck())
    {
        thePrefs::SetSmartIdState(1);
    }
    
    newsocket->ConnectToServer(server);
...
}
No tags attached.
Issue History
2006-03-21 15:21eyalzoNew Issue
2006-03-22 20:43KryStatusnew => assigned
2006-03-22 20:43KryAssigned To => Kry
2006-05-24 03:55KryStatusassigned => resolved
2006-05-24 03:55KryFixed in Version => CVS
2006-05-24 03:55KryResolutionopen => fixed
2006-05-24 03:55KryNote Added: 0002000

Notes
(0002000)
Kry   
2006-05-24 03:55   
Commited.