Changeset 8667 for code/branches/presentation/src/modules/questsystem/QuestEffectBeacon.cc

Timestamp:

May 29, 2011, 6:44:45 PM (13 years ago)

Author:

dafrick

Message:

Possible fix for segfaults due to player being NULL.

File:

• code/branches/presentation/src/modules/questsystem/QuestEffectBeacon.cc (modified) (2 diffs)

code/branches/presentation/src/modules/questsystem/QuestEffectBeacon.cc

@@ -113 +113 @@
 
         PlayerTrigger* pTrigger = orxonox_cast<PlayerTrigger*>(trigger);
-        Pawn* pawn = NULL;
+        PlayerInfo* player = NULL;
 
         // If the trigger is a PlayerTrigger.
@@ -121 +121 @@
                 return false;
             else
-                pawn = pTrigger->getTriggeringPlayer();
+                player = pTrigger->getTriggeringPlayer();
         }
         else
             return false;
 
-        if(pawn == NULL)
+        if(player == NULL)
         {
             COUT(4) << "The QuestEffectBeacon was triggered by an entity other than a Pawn. (" << trigger->getIdentifier()->getName() << ")" << std::endl;
-            return false;
-        }
-
-        // Extract the PlayerInfo from the Pawn.
-        PlayerInfo* player = pawn->getPlayer();
-
-        if(player == NULL)
-        {
-            COUT(3) << "The PlayerInfo* is NULL." << std::endl;
             return false;
         }