From 1842a932d37638c910f4471d85e81554076c3d2c Mon Sep 17 00:00:00 2001 From: Rodger Combs Date: Sun, 1 Jul 2018 04:56:08 -0500 Subject: {mac,cocoa}: trim trailing null out of macosx_icon when loading it This prevents crashes when loading the application icon image. Suggested-by: Akemi --- video/out/cocoa_common.m | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'video/out') diff --git a/video/out/cocoa_common.m b/video/out/cocoa_common.m index abead0fb53..b4283bbe8d 100644 --- a/video/out/cocoa_common.m +++ b/video/out/cocoa_common.m @@ -172,8 +172,10 @@ static void set_application_icon(NSApplication *app) NSBundle *bundle = [NSBundle mainBundle]; if ([bundle pathForResource:@"icon" ofType:@"icns"]) return; + + // The C string contains a trailing null, so we strip it away NSData *icon_data = [NSData dataWithBytesNoCopy:(void *)macosx_icon - length:sizeof(macosx_icon) + length:sizeof(macosx_icon) - 1 freeWhenDone:NO]; NSImage *icon = [[NSImage alloc] initWithData:icon_data]; [app setApplicationIconImage:icon]; -- cgit v1.2.3