Use objectguid instead, hasmap lookups are fast, and this did not ensure that player's pointer were valid. Now it does