ServerActivate checks for player entities incorrectly

[SamVanheer]

ServerActivate checks for player entities incorrectly, resulting in it not calling Activate for worldspawn:

halflife/dlls/client.cpp

Lines 716 to 718 in c7240b9

	// Clients aren't necessarily initialized until ClientPutInServer()
	if ( i < clientMax || !pEdictList[i].pvPrivateData )
	continue;

This check also allows it to call Activate on the player in the slot matching maxplayers (last slot) which it shouldn't do.

This should be:

// Clients aren't necessarily initialized until ClientPutInServer()
if ( (i > 0 && i <= clientMax) || !pEdictList[i].pvPrivateData )
	continue;

Originally reported by vasiavasiavasia95.