Cherry/3D2D-Textscreens

https://github.com/Cherry/3D2D-TextScreens/blob/master/lua/weapons/gmod_tool/stools/textscreen.lua#L64

notcake opened this issue · 2 comments

This seems accurate.

While we do check for textscreen count for player <= limit before spawning, the count isn't incremented before after the error occurs. It can be reproduced by nulling one of local convars. By doing so, it will error out before ownership, and count is applied, and so it will always think that the player in question has zero textscreens, allowing you to proceed.

Should fix both by avoiding nil, and also by moving count increment, and ownership to right after spawn

Thanks. Fixed in 5f44284.