Bug when deactivating skins (with fix proposal)

This topic has 4 replies, 2 voices, and was last updated 12 years, 9 months ago by Support.

Viewing 4 posts - 1 through 4 (of 4 total)

Author

Posts
May 28, 2013 at 10:32 pm #50559
wesson
Participant
Also same issue (out of bound access to dynamic array) occurs when switching between themes (ex: from BlueGauze to BlueGlass).

For the fix, there is an attached a patch file for sSkinProvider.pas

There's not a lot of range checks in the code… 😉
May 29, 2013 at 5:37 am #50560
Support
Keymaster
Thank you, I'll check it soon.

Btw, sSkinProvider.pas is not attached here 🙂
May 29, 2013 at 10:14 am #50561
wesson
Participant
'Support' wrote:

Thank you, I'll check it soon.

Btw, sSkinProvider.pas is not attached here 🙂

My mistake, I forgot to push the “attach this file” button.

It's based on Version 8.34, modifications are in lines 6793, 6827 and 6834
June 5, 2013 at 9:10 am #50567
Support
Keymaster
Thank you.
Author

Posts

Viewing 4 posts - 1 through 4 (of 4 total)

You must be logged in to reply to this topic.