Small fix for small problem

[volas]

No description provided.

[Menshin]

What was the non-standard character ?

Was there any proc that was affected by this infinite loop ? (I think this old sanitizing proc is still used somewhere in the code)

[volas]

What was the non-standard character ?

Maybe once it was a russian "я" letter ("я" = "Я"), but due to problems with encodings at some point it was "broken". Or maybe someone left it specially. Who knows.

Was there any proc that was affected by this infinite loop ? (I think this old sanitizing proc is still used somewhere in the code)

sanitize() use it, and sanitize() is used in many places. https://github.com/tgstation/-tg-station/search?utf8=%E2%9C%93&q=sanitize%28

[Menshin]

Yeah, apparently it's still used a lot...
I've a PR dealing with sanitizing procs opened already (#7471), to avoid merge conflicts i'll add your fix to it.

Thanks for pointing this out.

[volas]

Yeah, well, then I close my PR

[Deantwo]

What about goimg back in the commit history and see what the symbol was.
That is one of the reasons to use version-control after all.

But yeah I had been wondering what it was too.