[Mini-Fix] Teleport Crash

[Iban24]

Datei: World.cpp

Sucht nach:

 		SAFE_DELETE( pObj );

ersetzen mit:

#ifndef __CLIENT
		SAFE_DELETE( pObj );
#endif

Nun crasht euer Client nicht mehr wenn ihr euch mit einem Pet Teleportiert.

[andreitajai]

When this bug happen?

[Chriѕ]

When you teleport with a pet.

[/-/µR€]

Originally Posted by Iban24

Datei: World.cpp

Sucht nach:

 		SAFE_DELETE( pObj );

ersetzen mit:

#ifndef __CLIENT
		SAFE_DELETE( pObj );
#endif

Nun crasht euer Client nicht mehr wenn ihr euch mit einem Pet Teleportiert.

In World.cpp gibt es 2 mal SAFE_DELETE( pObj );

[Wanetrain]

Originally Posted by /-/µR€

In World.cpp gibt es 2 mal SAFE_DELETE( pObj );

Sollte man 1. Wissen was SAFE_DELETE macht und 2. was dannach gemacht wird, wa?

1. SAFE_DELETE löscht einen Pointer und setzt diesen auf NULL.
2. Wenn nach SAFE_DELETE( pObj ) im Client noch der Pointer pObj aufgerufen wird ist es klar dass es Crasht, da er nicht vorhanden ist, von daher können keine Informationen aus ihm "gezogen" werden, also ist der Fixx einerseits dumm und anderseits auch nicht.

Dumm ist er weil niemand weiß ob der Pointer noch für wichtige sachen benötigt wird oder nicht.

Schlau wiederrum: es behebt ZWANGSWEIßE UND NICHT ZUM VORTEIL den Crash.

Wenn man die ganze Teleport sache schlauer gemacht hätte, würde man den Fixx nicht brauchen, ich sag nur: Ticket Function, damit kannst du egal ob du Pet hast oder net raus ein rein aus Cash welten kommen, ohne crash.

[FlyCraft.TobiLap]

Originally Posted by Wanetrain

Dumm ist er weil niemand weiß ob der Pointer noch für wichtige sachen benötigt wird oder nicht.

Schlau wiederrum: es behebt ZWANGSWEIßE UND NICHT ZUM VORTEIL den Crash.

Falls du dir den crash mal anschaus merkste dass safedelete aufgrund einer nullpointer exeption stirbt. ist schon der richtige fix so hab den so ähnlich schon seit ca einem jahr kurz nach flycraft down.

[Wanetrain]

Originally Posted by FlyCraft.TobiLap

Falls du dir den crash mal anschaus merkste dass safedelete aufgrund einer nullpointer exeption stirbt. ist schon der richtige fix so hab den so ähnlich schon seit ca einem jahr kurz nach flycraft down.

Der Richtige Fixx ansich ja, allerdings würde man es besser hinbekommen. ;>

[FlyCraft.TobiLap]

Originally Posted by Wanetrain

Der Richtige Fixx ansich ja, allerdings würde man es besser hinbekommen. ;>

oops, hab den fix nur überflogen, xD

so währe der besser:

PHP Code:

if(pObj )
{
     SAFE_DELETE( pObj );
} 


so ists besser (bzw so hab ichs)

[Sedrika]

Originally Posted by FlyCraft.TobiLap

oops, hab den fix nur überflogen, xD

so währe der besser:

PHP Code:

if(pObj )
{
     SAFE_DELETE( pObj );
} 


so ists besser (bzw so hab ichs)

SAFE_DELETE hat bereits eine abfrage ob dieser valide ist oder nicht, so machst du das ganze nur doppelt.

[Velmore]

Originally Posted by Sago​

SAFE_DELETE hat bereits eine abfrage ob dieser valide ist oder nicht, so machst du das ganze nur doppelt.

Und sagen wir so das hilft auch nicht gegen den Crash.

Ich glaube für die meisten Server reicht es einfach das SAFE_DELETE auszunehmen.

[FlyCraft.TobiLap]

Originally Posted by Sago​

SAFE_DELETE hat bereits eine abfrage ob dieser valide ist oder nicht, so machst du das ganze nur doppelt.

dachte ich auch, nur scheisse, dass es trotzdem crashed wenn das pObj nen nullpointer ist. erstmal testen bevor man dumme kommentare macht.

[Velmore]

Originally Posted by FlyCraft.TobiLap

dachte ich auch, nur scheisse, dass es trotzdem crashed wenn das pObj nen nullpointer ist. erstmal testen bevor man dumme kommentare macht.

Ich hatte das mit der Abfrage auch mal gemacht aber es ist noch gecrasht.

[Sedrika]

Originally Posted by FlyCraft.TobiLap

dachte ich auch, nur scheisse, dass es trotzdem crashed wenn das pObj nen nullpointer ist. erstmal testen bevor man dumme kommentare macht.

Jemanden dumm anmachen ist dumm, dein Verhalten ist schlimmer als jemand der debil ist.

#define SAFE_DELETE(p)       { if(p) { safe_delete(p);     (p)=NULL; } }

Merkste was?

[Serenity-.]

Ich weiß nicht ob ich was falsch gemacht habe aber sobald ich das mache was iban geschrieben habe verschwinden npc´s immer wieder xDDD
Wenn man ein Ticket benutzt usw.

Hat jemand einen anständigen fix dafür xD