#408 Group Member null crashes zone on SpellProcess

SpellProcess.cpp:2180

                    PlayerGroup* group = world.GetGroupManager()->GetGroup(((Entity*)caster)->GetGroupMemberInfo()->group_id);
                    if (group)
                    {
                        group->MGroupMembers.readlock(__FUNCTION__, __LINE__);
                        deque<GroupMemberInfo*>* members = group->GetMembers();
                        deque<GroupMemberInfo*>::iterator itr;

                        // iterate through list of group members
                        for (itr = members->begin(); itr != members->end(); itr++)
                        {
                            // get group member player info
                            Entity* group_member = (*itr)->member;

                            LogWrite(SPELL__DEBUG, 0, "Player", "%s is group member for spell %s", group_member->GetName(), luaspell->spell->GetName());
                            // if the group member is in the casters zone, and is alive

group_member can be null (while player is zoning)

Other areas where we pull GetMembers needs check for member being null

Other impacted code points:

Bot.cpp:310
Bot.cpp:399


BotCommands.cpp:50
BotCommands.cpp:113
BotCommands.cpp:187

client.cpp:5145

Combat.cpp:1223

SpellProcess.cpp:2352
SpellProcess.cpp:2398

Fixed pending git update. (original and other impacted code points)

54a61e0c39

I've seen, nor heard any further issues from this patch, closing related issues. I Will re-open if some problem pops back up.