search

flackr / circ

An IRC packaged chrome app
BSD 3-Clause "New" or "Revised" License
387 stars 79 forks source link

Complex unicode text crashes CIRC #362

Open korfuri opened 8 years ago

korfuri commented 8 years ago

I just found out that complex unicode ("zalgo" text) can cause CIRC 0.6.6 to crash. I haven't witnessed the behavior on the client-side, but here's a reproducible test case:

05:04 < me> Mͬ̊͛̈Yͥ̽̉ͣ ͋ͧͪ͋Eͭ̑̂́Ÿ̈̅ͪȆ͂ͪ̔Sͧ̐̀ͨ ́͊͐̑A̓̓̔ͫR̄̐ͮ̍E͆̂̈́̎ ̀̇ͪ́M͛̄͛͗E̽̃̈́̐L̍̍̃͑T͂͒̽͑İ͋̓̉N̈́ͫ̔̚Ǵ͑͊̑ 05:04 -!- person1 has quit [Input/output error] 05:04 -!- person2 has quit [Input/output error] 05:04 -!- person3 has quit [Input/output error]

Hopefully github won't clober my unicode too much. If it does, feel free to generate zalgo text on any generator, it's pretty easy to generate text that causes that.

derakon commented 7 years ago

I can confirm that this happens even for simple cases like a trademark symbol (™).