rightclicking the left aligned timestamp "frame" crashes the client

[Fenwryches]

Rightclicking on the timestamp "frame" causes the client to crash on win7 x32/64 and Ubuntu

[Stephen Lyons (slysven)]

Got it! In TTextEdit::mousePressEvent() when the timestamps are enabled the normal lines of text are offset to the right by 13 spaces (so the time stamps can go at the start of the line) this means the character x-coordinates used to manipulate text selections in the console have to have 13 added to them to refer to the correct text - but if the mouse is pressed in the time stamp area the coordinate will be less than zero which causes a seg. fault (the time stamp bit of the the text lines is NOT selectable).

The code for the left mouse button press correctly limits the x coordinate to not less than zero but the boundary value checks have not been repeated for the right mouse button press code.

The attached commit is for the current Source Forge Master but should apply to pretty much any recent version. Unfortunately I got my local master branch out of step with my public one on GitHub and until I can get them synced without losing other stuff locally I'm not sure I can push this commit out to my public repository directly.

[Stephen Lyons (slysven)]

Chris

Sorry if I've dropped it onto to you a bit abruptly but can you get this two line patch up into the current SF release?

[Ahmed Charles (ahmedcharles)]

Fixed by: https://github.com/Mudlet/Mudlet/commit/dce42a2b4d2c6c741cecbabe560387a0d3f41fc7