asgardius/virtualx-engine
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge pull request #79010 from bruvzg/set_native_icon_crash

Browse source 
[macOS] Fix `set_native_icon` crash with empty or invalid ICNS file.
This commit is contained in:
Rémi Verschelde 2023-07-07 08:32:56 +02:00
commit 9522ef67d5
No known key found for this signature in database
GPG key ID: C3336907360768E1
1 changed files with 11 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
platform/macos/display_server_macos.mm
View file

@ -3588,9 +3588,12 @@ void DisplayServerMacOS::set_native_icon(const String &p_filename) {
Vector<uint8_t> data;
uint64_t len = f->get_length();
ERR_FAIL_COND_MSG(len < 8, "Error reading icon data."); // "icns" + 32-bit length
data.resize(len);
f->get_buffer((uint8_t *)&data.write[0], len);
@try {
NSData *icon_data = [[NSData alloc] initWithBytes:&data.write[0] length:len];
ERR_FAIL_COND_MSG(!icon_data, "Error reading icon data.");
@ -3598,6 +3601,9 @@ void DisplayServerMacOS::set_native_icon(const String &p_filename) {
ERR_FAIL_COND_MSG(!icon, "Error loading icon.");
[NSApp setApplicationIconImage:icon];
} @catch (NSException *exception) {
ERR_FAIL_MSG("NSException: " + String::utf8([exception reason].UTF8String));
}
}
void DisplayServerMacOS::set_icon(const Ref<Image> &p_icon) {