bug#39047: macOS Catalina stack overflow patch

["Väinö Järvelä" <vaino@jarve.la>]

[-- Attachment #1: Type: text/plain, Size: 248 bytes --]

Hi,

After updating to macOS Catalina, Emacs started crashing when opening
any file, even a file that doesn't exist.

I traced it to a stack overflow from a fix to bug #38618 (commit
bfc54230). A fix for the bug is included.

-- 
Väinö

[-- Attachment #2: 0001-Fix-stack-overflow-on-macOS-Catalina.patch --]
[-- Type: application/octet-stream, Size: 1276 bytes --]

From c83cd49fa166ab87276146939eaf8ee3e0fd9a07 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?V=C3=A4in=C3=B6=20J=C3=A4rvel=C3=A4?= <vaino@jarve.la>
Date: Thu, 9 Jan 2020 08:01:59 +0200
Subject: [PATCH] Fix stack overflow on macOS Catalina

NSURL.getResourceValue:forKey:error: for key NSURLIsUbiquitousItemKey
writes an NSNumber to the pointer, not a BOOL. This results in a stack
overflow that can be reproduced on macOS Catalina when opening any file
with ns-use-proxy-icon enabled.
---
 src/nsfns.m | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/nsfns.m b/src/nsfns.m
index f2857c022e..42bd88eeb4 100644
--- a/src/nsfns.m
+++ b/src/nsfns.m
@@ -499,11 +499,11 @@ Turn the input menu (an NSMenu) into a lisp list for tracking on lisp side.
 #if defined (NS_IMPL_COCOA) && defined (MAC_OS_X_VERSION_10_7)
   /* Work around for Mach port leaks on macOS 10.15 (bug#38618).  */
   NSURL *fileURL = [NSURL fileURLWithPath:fstr isDirectory:NO];
-  BOOL isUbiquitousItem = YES;
+  NSNumber *isUbiquitousItem = @YES;
   [fileURL getResourceValue:(id *)&isUbiquitousItem
                      forKey:NSURLIsUbiquitousItemKey
                       error:nil];
-  if (isUbiquitousItem)
+  if ([isUbiquitousItem boolValue])
     fstr = @"";
 #endif
 
-- 
2.24.1