[Release] Sicherheits Fixxes

[Grening]

In diesem Thread release ich alle Hack/Dupe Fixxes, sowie andere Dinge die zur Serversicherheit beitragen.
Den Thread werde ich immer updaten, sobald ich Zeit habe.
Solltet ihr Fragen haben, so stehe ich euch gerne zur Verfügung.

Vorwort: Die Fixxes stammen nicht von mir! Ich habe sie lediglich aus anderen Sourcen exportiert und teile sie der Allgemeinheit mit. Sollte jemand ein Problem damit haben, so kann er mir das gerne per PN mitteilen.


Char Overwrite Fix

 Spoiler

Geht in die DbManager.cpp des Database Servers und sucht nach dieser Funktion

void CDbManager::RemovePlayer( CQuery *qry, LPDB_OVERLAPPED_PLUS lpDbOverlappedPlus )

Scrollt dann etwas hoch bis ihr zum

#endif // __2ND_PASSWORD_SYSTEM

kommt und fügt unter der

}

bis zum

else

das hier ein

#ifdef __CHAROVERWRITE_FIX
		else if( nError == 2 )
		{
			Error("CDbManager::CreatePlayer() : Try overwrite player: account: %s", lpDbOverlappedPlus->AccountInfo.szAccount);
			g_dpLoginSrvr.SendError( ERROR_USER_EXISTS, lpDbOverlappedPlus->dpid );
		}
#endif // __CHAROVERWRITE_FIX

das sollte dann so aussehen

				WriteLog( "%s, %d\t%s", __FILE__, __LINE__, szQuery );
				FreeRequest( lpDbOverlappedPlus );
				return;
			}
#endif // __2ND_PASSWORD_SYSTEM
		}
#ifdef __CHAROVERWRITE_FIX
		else if( nError == 2 )
		{
			Error("CDbManager::CreatePlayer() : Try overwrite player: account: %s", lpDbOverlappedPlus->AccountInfo.szAccount);
			g_dpLoginSrvr.SendError( ERROR_USER_EXISTS, lpDbOverlappedPlus->dpid );
		}
#endif // __CHAROVERWRITE_FIX
		else
		{
			// ij¸¯ÅÍ Áߺ¹
			g_dpLoginSrvr.SendError( ERROR_USER_EXISTS, lpDbOverlappedPlus->dpid );
		}
	}
	FreeRequest( lpDbOverlappedPlus );

In alle Versioncommon.h das hier eintragen

#define __CHAROVERWRITE_FIX

Guildbank Dupe Fix #1

 Spoiler

Geht in DPsrvr.cpp des Worldservers und sucht nach

void CDPSrvr::OnPutItemGuildBank( CAr & ar, DPID dpidCache, DPID dpidUser, LPBYTE lpBuf, u_long uBufSize )

Scrollt dann runter bis ihr das hier findet

		//	GUILD_BANK_STR 'S1','000000','01' 
		//	GUILD BANK Àüü ºÒ·¯¿À±â ex ) GUILD_BANK_STR 'S1',@im_idGuild,@iserverindex GUILD_BANK_STR 'S1','000000','01'  
		//	GUILD BANK ÀúÀåÇϱâ ex ) GUILD_BANK_STR 'U1',@im_idGuild,@iserverindex,@im_nGoldGuild,@im_apIndex,@im_dwObjIndex,@im_GuildBank GUILD_BANK_STR 'U1','000001','01',0,'$','$','$'

Dadrüber fügt ihr dann ein

#ifdef __GUILDBANK_DUPEFIX1	
		if( (short)( nItemNum ) > pItemElem->m_nItemNum )
			nItemNum = pItemElem->m_nItemNum;
		if( (short)(nItemNum) < 1 )
			nItemNum = 1;
#else // __GUILDBANK_DUPEFIX1
		if( (int)( nItemNum ) > pItemElem->m_nItemNum )
			nItemNum = pItemElem->m_nItemNum;
		if( nItemNum < 1 )
			nItemNum = 1;
#endif // __GUILDBANK_DUPEFIX1

Tragt dann in alle Versioncommon.h das hier ein

#define __GUILDBANK_DUPEFIX1             // CDPSrvr::OnPutItemGuildBank()

Guildbank Dupe Fix #2

 Spoiler

Geht in DPsrvr.cpp des Worldservers und sucht nach

void CDPSrvr::OnGetItemGuildBank( CAr & ar, DPID dpidCache, DPID dpidUser, LPBYTE lpBuf, u_long uBufSize )

Scrollt runter bis

		else if (mode == 1) // ¾ÆÀÌÅÛÀ» ±æµåâ°í¿¡¼* »©³¾¶§
		{
			CGuild*			pGuild = pUser->GetGuild();
			if (pGuild && pGuild->IsGetItem(pUser->m_idPlayer))
			{
				CItemElem* pItemElem	= pGuild->m_GuildBank.GetAtId( nId );
				
				if( NULL == pItemElem )
					return;

und fügt das hier drunter ein

#ifdef __GUILDBANK_DUPEFIX2				
				if( (short)( dwItemNum ) > pItemElem->m_nItemNum )
					dwItemNum = pItemElem->m_nItemNum;
				if( (short)(dwItemNum) < 1 )
					dwItemNum	= 1;
#else // __GUILDBANK_DUPEFIX2
				if( (int)( dwItemNum ) > pItemElem->m_nItemNum )
					dwItemNum = pItemElem->m_nItemNum;
				if( dwItemNum < 1 )
					dwItemNum	= 1;
#endif // __GUILDBANK_DUPEFIX2

Tragt dann in alle Versioncommon.h das hier ein

#define __GUILDBANK_DUPEFIX2				// CDPSrvr::OnGetItemGuildBank()

Veteranen Source:

[Schulzie]

Gute Idee (wenn der Thread auch weiter geführt wird),
aber da fehlen ja noch ein paaaar Fixxes =D
Hau rein...°

[Spraystar]

Nice hoffe es kommen mehr

[DerHeiligeJesus]

Ist das normal das meine danach nachdem man diese Eingebaut hat keine Items mehr in das Gildenlager legen kann oder liegt das am RoC Source?

[Grening]

Originally Posted by DerHeiligeJesus

Ist das normal das meine danach nachdem man diese Eingebaut hat keine Items mehr in das Gildenlager legen kann oder liegt das am RoC Source?

Ich habe es eben nochmal getestet und bei mir klappt es. Ist das auch wirklich die richtige Stelle in der du das eingebaut hast?

[DerHeiligeJesus]

ja 100%

[Grening]

Bei dem Guild fix#1 hast du es unter oder über den drei auskommentierten Sachen eingebaut?

Hast du beide Fixxes eingebaut?
Soll ich mal über TV rüber gucken?

[- DK]

Wenn du mehr Releasen willst, dann zieh dir die Veteran Source und easy.

[DerHeiligeJesus]

wo gibt es die source?

[Eightb4ll]

Erst einmal nette Idee, aber Fix #1 ist ziemlich senseless bzw. nutzlos.
Fix #2 & #3 da würde ich mal schauen ob es nicht klüger ist den Datentypen direkt auf short zu setzen. (Wobei ich nebenbei gesagt, auch hier bezweifle das irgendwas gefixt wird)

[Grening]

Bei dem Fix #2 & #3 ist der doch auf4 short gesetzt.

[Xylenu]

Zu Fix #1,
der is schon recht nutzlos.
Warum frägt man in der Removeplayer nach nError == 2, wenn man es doch direkt in der Createplayer abfragen lassen kann. Hätte zumindest in meinen Augen mehr Sinn.
Außerdem würde ich das ganze direkt in der DB fixen -> Character Str, da braucht man nix im Src editieren^^

[xTwiLightx]

Originally Posted by Xylenu

Zu Fix #1,
der is schon recht nutzlos.
Warum frägt man in der Removeplayer nach nError == 2, wenn man es doch direkt in der Createplayer abfragen lassen kann. Hätte zumindest in meinen Augen mehr Sinn.
Außerdem würde ich das ganze direkt in der DB fixen -> Character Str, da braucht man nix im Src editieren^^

Mal davon abgesehen, dass die Prozedur auch den nError 2 zurückgeben muss, damit der "Fix" auch greift. Stattdessen macht man die Abfrage komplett in der Prozedur und gibt 0 zurück, was den Charakter auch nicht erstellt.

[Eightb4ll]

Originally Posted by Grening

Bei dem Fix #2 & #3 ist der doch auf4 short gesetzt.

Nein du castest auf short, was aber keinen Sinn ergibt.
Weil die Vergleichsoperatoren zwischen unsigned Long und short auch ohne Casts wunderbar funktionieren. Warum Offi castet kann ich mir nur erklären weil sie den globalen max. Stapel begrenzen wollten. Dann kann man aber wie bereits erwähnt die Variable direkt als short deklarieren.

Originally Posted by Xylenu

Zu Fix #1,
der is schon recht nutzlos.
Warum frägt man in der Removeplayer nach nError == 2, wenn man es doch direkt in der Createplayer abfragen lassen kann. Hätte zumindest in meinen Augen mehr Sinn.
Außerdem würde ich das ganze direkt in der DB fixen -> Character Str, da braucht man nix im Src editieren^^

Der "Fix" wird in der CreatePlayer eingefügt, aber trotzdem ist es reiner Schwachsinn.

[Grening]

Da ich kein Bock mehr auf Flyff habe, gebe ich euch einfach mal die Veteranen Source von dort aus könnt ihr euch die ganzen Fixxes ziehen. Kann ich euch nur empfehlen.

#Ist auch im First Post geaddet.


Vielleicht Release ich auch noch die Homepage etc.