LithApp / Lith

Multiplatform, mobile-focused WeeChat relay client
https://lith.app
GNU General Public License v2.0
80 stars 7 forks source link

segfault @ protocol.c:Protocol::convertColorsToHtml #61

Closed chinese-soup closed 3 years ago

chinese-soup commented 3 years ago

Line 471 data = "*|05~05c

image

This doesn't always happen (as you can see in the video it took a second try) but is reproducible quite a lot! Do not test on a buffer you aren't afraid to clear, as this crashes the client (almost) everytime the buffer with the characters is opened.

Video (with steps to reproduce): http://povidam.ahoj.club/2020-11-15_23-33-33.mp4

Steps to write a bg+fg message: in weechat press: Ctrl-C then type: 3,3c

where "c" is any message you want to write with green text and green background

On an unrelated note: as you can see, even if it doesn't crash, the message gets mangled as I assume we don't parse background colors at all?

chinese-soup commented 3 years ago

Also other stuff around text formatting is slightly broken still (but only background color seems to crash the client):

image

Steps to reproduce the above screenshot: