gmod_src: Fixed crash when scripters try to get an entity by..

[garry]

[B]Revision 966[/B]
[INDENT]Fixed crash when scripters try to get an entity by using a network ID higher than the max number of entities[/INDENT][B]Changed Files:[/B]
[LIST]
[*]trunk/src_garrysmod/game/shared/entitylist_base.h
[/LIST]
Committed By [B]Garry Newman[/B]

[Wizard of Ass]

So

[lua]Entity(4097)[/lua]

Would have crashed the server?

[King Flawless]

While your at fixing bad coder crashes could you fix this

] lua_run umsg.End()
> umsg.End()...
[lua_run:1] Error: Calling umsg.End without a valid message!

But
[lua]lua_run umsg.Bool(1) umsg.End()[/lua]
Will crash you

[editline]7th May 2011[/editline]

Crash dump if you want it - [url]http://dumps.garrysmod.com/?view=2160835210[/url]

[Chewgum]

[QUOTE=King Flawless;29672145]While your at fixing bad coder crashes could you fix this

] lua_run umsg.End()
> umsg.End()...
[lua_run:1] Error: Calling umsg.End without a valid message!

But
[lua]lua_run umsg.Bool(1) umsg.End()[/lua]
Will crash you

[editline]7th May 2011[/editline]

Crash dump if you want it - [url]http://dumps.garrysmod.com/?view=2160835210[/url][/QUOTE]

You gotta be damn retarded if you do that anyway, so why 'fix' it?

[King Flawless]

[QUOTE=_Chewgum;29682277]You gotta be damn retarded if you do that anyway, so why 'fix' it?[/QUOTE]

He was fixing bad coder crashes so I was simply informing him of one, and now I feel retarded for doing it and crashing thanks :ohdear:.

[ColdFusion]

[QUOTE=Wizard of Ass;29672059]So

[lua]Entity(4097)[/lua]

Would have crashed the server?[/QUOTE]

Probably just client side.

[Grocel]

[QUOTE=_Chewgum;29682277]So why 'fix' it?[/QUOTE]

So why Garry fixed the entity crash, it also was caused by bad coding?

Lua is a scripting language, that means bad coding should not cause crashing. It should show an error Message if there is something wrong.