rightclicking the left aligned timestamp "frame" crashes the client
Bug #1329948 reported by
Fenwryches
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Mudlet |
Fix Released
|
Critical
|
Chris |
Bug Description
Rightclicking on the timestamp "frame" causes the client to crash on win7 x32/64 and Ubuntu
Changed in mudlet: | |
status: | New → Confirmed |
Changed in mudlet: | |
assignee: | nobody → Chris (chrismudlet) |
Changed in mudlet: | |
status: | Confirmed → Fix Committed |
status: | Fix Committed → Confirmed |
Changed in mudlet: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
Got it! In TTextEdit: :mousePressEven t() 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.