Was halltet ihr von dem hier:
Code:
SERIALNUMBER CDbManager::RemoveItem( CQuery *pQry, char* szSql, int nNo, CMover* pMover, char* szItemName, int nItemCount, int nAbilityOption, int nItemResist, int nResistAbilityOption, char chState )
{
SERIALNUMBER iSerialNumber = 0;
if(nNo < 0 || nItemCount < 0)
return iSerialNumber;
Man sollte sowiso immer alle Parameter überprüfen aber jetzt speziell zum Fixxen von diesem Bug reicht das hier aus oder gehts immernoch weiter mit dem dupen?
Wenn ja dann muss ich mir die Klassen nochmal genauer anschauen...
Eigentlich müsste es simpel sein in allen öffentlichen schnittstellen die Params checken und schon bringt das absenden von invaliden werten nichts mehr.
Statt dem Return einfach einen 30min Bann und schon kann man viele probleme beheben
Falls ich mich irre killt mich net kenne nichtmal nen achtel aller klassen im Flyff Sourcecode und auch die nur vom überfliegen...
MFG Andrej-Delany