kruton / tinyfugue

Tinyfugue MUD client (tf) with UTF-8 patches for Unicode MUDs.
http://tinyfugue.sourceforge.net/
Other
36 stars 21 forks source link

remote crash, reproducible #12

Closed warmstarter closed 3 years ago

warmstarter commented 3 years ago

I can reliably remotely crash this version of TF. There's already one version patched, but I would like to reliably disclose this to you, please tell me how best to contact.

warmstarter commented 3 years ago

This is fixed in Sketch's fork of this repo. I'll try to get in touch with him about sending a pull request, and if not I'll work to make one. Managed to patch the same issue on TF-rebirth using the modifications on Sketch's fork, and it worked out well despite being a very diverged target. Should be relatively easy for here.

warmstarter commented 3 years ago

Also note, this relates to the backspace bug reported in 2012. That's creating strange characters that can cause a crash either locally or remotely. Usually it just makes garbled text, but I can reliably make it crash.

kruton commented 3 years ago

Sure. I don't think anyone has sent a pull request here ever.

warmstarter commented 3 years ago

This is resolved in the Pull Request that just got merge.