Bug when deactivating skins (with fix proposal)

This topic has 4 replies, 2 voices, and was last updated 11 years, 12 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.