[FIXED] cPlayer with NULL pointer to cClientHandle

[FakeTruth]

Apparently sometimes a player still has a NULL pointer to its cClientHandle.

I noticed this in cFurnaceEntity::Tick line 187
It tries to send a cooking progress packet to a NULL pointer

This is just a reminder for me, I'll have to investigate later

[xoft]

cClientHandle's destructor sets cPlayer's m_ClientHandle to NULL just before detaching itself from the player object. This might be the cause.

I'd say, remove the cPlayer::SetClientHandle altogether and set the cPlayer's clienthandle to NULL in its Destroy() method. Then you'll see if there have been any more calls to SetClientHandle().
And of course you can put asserts everywhere

[FakeTruth]

These are all references to SetClientHandle, so really it's only called once from the cClientHandle destructor


You're right about removing it, it's called together with Destroy() anyway

[xoft]

Well, I've seen VS's "Call browser" fail too many a time, so I don't trust it anymore Removing the function implementation is the easiest way to tell

[FakeTruth]

It's Visual Assist's reference finder thingy Much better.