[ozrich]

Hi Folks,

I'm not a good coder. In fact, I only understand the basics so wanted some help.

I'm using Daimyo21's Insurgency plugin and receive the below error in console:

L 01/08/2017 - 04:29:47: [SM] Exception reported: Client index 0 is invalid
L 01/08/2017 - 04:29:47: [SM] Blaming: c_dy_respawn_naong.smx
L 01/08/2017 - 04:29:47: [SM] Call stack trace:
L 01/08/2017 - 04:29:47: [SM] [0] GetClientTeam
L 01/08/2017 - 04:29:47: [SM] [1] Line 6201, <drive\path\>c_dy_respawn_naong.sp::GrenadeSc reamCheckTimer

I understand I could comment it out, and it appears to work, I don't know, but would like to fix it.

[404UserNotFound]

Client index 0 is always the console. It's never a client.

I see multiple checks to make sure the client is valid and is not 0...yet the thing still fired off with an error.

Actually, it could possibly be GetClientTeam(owner) == 3. Try uncommenting things out to bring the plugin back to normal functions, then change line 6201 from

PHP Code:

if (client > 0 && IsPlayerAlive(client) && GetClientTeam(client) == 2 && GetClientTeam(owner) == 3) 


to

PHP Code:

if (client > 0 && IsPlayerAlive(client) && GetClientTeam(client) == 2 /* && GetClientTeam(owner) == 3*/) 


Compile and give it a try. I'm wondering if it's line 6108 (owner = GetEntPropEnt(entity, Prop_Send, "m_hOwnerEntity");) that is setting the owner of the entity to index 0 for some reason.

[ozrich]

Thanks the error no longer appears.

Is it ok to leave this commented out? I'm assuming the owner is the server (client index 0 as you say) thus giving the error and doesn't need to be there?

[404UserNotFound]

It *should* be safe to leave commented out.

You could always go the experimental route and try something like this...

PHP Code:

if (client > 0 && IsPlayerAlive(client) && GetClientTeam(client) == 2 && owner > 0 && GetClientTeam(owner) == 3) 


All I've done here is added "owner > 0". Not sure if this'll help or hurt, but it's always worth a shot.

If it gets worse, you can always remove "owner > 0" and leave GetClientTeam(owner) commented out.

[ozrich]

Thanks, I'll try that and let you know.

[ozrich]

Thanks for all your help, that worked.

Just one last error I'm receiving AFAIK. I contacted the developer of the plugin and he said he can't see any issues... Updated source attached.

L 01/14/2017 - 15:40:00: [SM] Exception reported: Client index 560163172 is invalid
L 01/14/2017 - 15:40:00: [SM] Blaming: c_dy_respawn_naong new new.smx
L 01/14/2017 - 15:40:00: [SM] Call stack trace:
L 01/14/2017 - 15:40:00: [SM] [0] PrintToChat
L 01/14/2017 - 15:40:00: [SM] [1] Line 3683, D:\Insurgency Server\Insurgency-dy-sourcemod-c51a2fdbae8d1fe32675a5623f3ebe6b8ecd6bf2\scri pting\c_dy_respawn_naong new new.sp::Event_PlayerDeath
L 01/14/2017 - 15:40:00: [SM] [3] ForcePlayerSuicide
L 01/14/2017 - 15:40:00: [SM] [4] Line 1631, D:\Insurgency Server\Insurgency-dy-sourcemod-c51a2fdbae8d1fe32675a5623f3ebe6b8ecd6bf2\scri pting\c_dy_respawn_naong new new.sp::Timer_CheckEnemyStatic

[ThatOneGuy]

Originally Posted by ozrich Thanks for all your help, that worked. Just one last error I'm receiving AFAIK. I contacted the developer of the plugin and he said he can't see any issues... Updated source attached. L 01/14/2017 - 15:40:00: [SM] Exception reported: Client index 560163172 is invalid L 01/14/2017 - 15:40:00: [SM] Blaming: c_dy_respawn_naong new new.smx L 01/14/2017 - 15:40:00: [SM] Call stack trace: L 01/14/2017 - 15:40:00: [SM] [0] PrintToChat L 01/14/2017 - 15:40:00: [SM] [1] Line 3683, D:\Insurgency Server\Insurgency-dy-sourcemod-c51a2fdbae8d1fe32675a5623f3ebe6b8ecd6bf2\scri pting\c_dy_respawn_naong new new.sp::Event_PlayerDeath L 01/14/2017 - 15:40:00: [SM] [3] ForcePlayerSuicide L 01/14/2017 - 15:40:00: [SM] [4] Line 1631, D:\Insurgency Server\Insurgency-dy-sourcemod-c51a2fdbae8d1fe32675a5623f3ebe6b8ecd6bf2\scri pting\c_dy_respawn_naong new new.sp::Timer_CheckEnemyStatic

Try changing line 3538 to

PHP Code:

// Check client valid
if (!IsValidClient(client)) return Plugin_Continue; 


It is currently:

PHP Code:

// Check client valid
if (!IsClientInGame(client)) return Plugin_Continue; 


Which can allow invalid client indexes. I assume IsValidClient() is defined in your includes, since it isnt in the code provided, and I assume it checks for IsClientInGame AND for valid client indexes. If so, that change should fix your error.

[ozrich]

Thanks for your help, I did that and I'm receiving the below errors (which I don't think I was receiving before). The error lines were for 3683 and 1631 but you mentioned to change 3538. Just confirming that was correct?

L 01/15/2017 - 10:195: [SM] Exception reported: Array index is out of bounds
L 01/15/2017 - 10:195: [SM] Blaming: c_dy_respawn_naong new new.smx
L 01/15/2017 - 10:195: [SM] Call stack trace:
L 01/15/2017 - 10:195: [SM] [1] Line 2117, D:\Insurgency Server\Insurgency-dy-sourcemod-c51a2fdbae8d1fe32675a5623f3ebe6b8ecd6bf2\scri pting\c_dy_respawn_naong new new.sp::CheckSpawnPointPlayers
L 01/15/2017 - 10:195: [SM] [2] Line 2241, D:\Insurgency Server\Insurgency-dy-sourcemod-c51a2fdbae8d1fe32675a5623f3ebe6b8ecd6bf2\scri pting\c_dy_respawn_naong new new.sp::Event_Spawn

[ThatOneGuy]

Yes, and now you have a new error. That plugin is just really buggy. Why not send it back to the original coder?

[ozrich]

I suspected that was the case but the code doesn't feel there's anything wrong with it.