From 7b9fc1c702584aad3204f8f8328d8ec257320120 Mon Sep 17 00:00:00 2001
From: Rose <83477269+AreaZR@users.noreply.github.com>
Date: Thu, 17 Oct 2024 15:31:35 -0400
Subject: [PATCH] Use CFStringCreateWithCStringNoCopy when the CFString doesn't
 outlast the char *

The object is freed before anything happens to the pointers.
---
 backend/usb-darwin.c | 4 ++--
 cups/dest.c          | 2 +-
 scheduler/colorman.c | 8 ++++----
 3 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/backend/usb-darwin.c b/backend/usb-darwin.c
index 5040c604e..ade1f1fbc 100644
--- a/backend/usb-darwin.c
+++ b/backend/usb-darwin.c
@@ -1950,7 +1950,7 @@ CFStringRef cfstr_create_trim(const char *cstr)
   CFStringRef		cfstr;
   CFMutableStringRef	cfmutablestr = NULL;
 
-  if ((cfstr = CFStringCreateWithCString(kCFAllocatorDefault, cstr, kCFStringEncodingUTF8)) != NULL)
+  if ((cfstr = CFStringCreateWithCStringNoCopy(kCFAllocatorDefault, cstr, kCFStringEncodingUTF8)) != NULL)
   {
     if ((cfmutablestr = CFStringCreateMutableCopy(kCFAllocatorDefault, 1024, cfstr)) != NULL)
       CFStringTrimWhitespace(cfmutablestr);
@@ -2071,7 +2071,7 @@ static void setup_cfLanguage(void)
     return;
   }
 
-  lang[0] = CFStringCreateWithCString(kCFAllocatorDefault, requestedLang, kCFStringEncodingUTF8);
+  lang[0] = CFStringCreateWithCStringNoCopy(kCFAllocatorDefault, requestedLang, kCFStringEncodingUTF8);
   langArray = CFArrayCreate(kCFAllocatorDefault, (const void **)lang, sizeof(lang) / sizeof(lang[0]), &kCFTypeArrayCallBacks);
 
   CFPreferencesSetValue(CFSTR("AppleLanguages"), langArray, kCFPreferencesCurrentApplication, kCFPreferencesAnyUser, kCFPreferencesAnyHost);
diff --git a/cups/dest.c b/cups/dest.c
index f0067100a..1f7fb35a6 100644
--- a/cups/dest.c
+++ b/cups/dest.c
@@ -2149,7 +2149,7 @@ cupsSetDests2(http_t      *http,	// I - Connection to server or @code CUPS_HTTP_
 
   if ((dest = cupsGetDest(NULL, NULL, num_dests, dests)) != NULL)
   {
-    CFStringRef name = CFStringCreateWithCString(kCFAllocatorDefault, dest->name, kCFStringEncodingUTF8);
+    CFStringRef name = CFStringCreateWithCStringNoCopy(kCFAllocatorDefault, dest->name, kCFStringEncodingUTF8);
 					// Default printer name
 
     if (name)
diff --git a/scheduler/colorman.c b/scheduler/colorman.c
index 5784c1c75..a7a9767ea 100644
--- a/scheduler/colorman.c
+++ b/scheduler/colorman.c
@@ -249,7 +249,7 @@ apple_init_profile(
     return;
   }
 
-  cftext = CFStringCreateWithCString(kCFAllocatorDefault, text,
+  cftext = CFStringCreateWithCStringNoCopy(kCFAllocatorDefault, text,
 				     kCFStringEncodingUTF8);
 
   if (cftext)
@@ -281,9 +281,9 @@ apple_init_profile(
 
       if (attr && attr->text[0])
       {
-	cflang = CFStringCreateWithCString(kCFAllocatorDefault, language,
+	cflang = CFStringCreateWithCStringNoCopy(kCFAllocatorDefault, language,
 					   kCFStringEncodingUTF8);
-	cftext = CFStringCreateWithCString(kCFAllocatorDefault, attr->text,
+	cftext = CFStringCreateWithCStringNoCopy(kCFAllocatorDefault, attr->text,
 					   kCFStringEncodingUTF8);
 
         if (cflang && cftext)
@@ -766,7 +766,7 @@ apple_register_profiles(
     device_name  = CFDictionaryCreateMutable(kCFAllocatorDefault, 0,
 					     &kCFTypeDictionaryKeyCallBacks,
 					     &kCFTypeDictionaryValueCallBacks);
-    printer_name = CFStringCreateWithCString(kCFAllocatorDefault,
+    printer_name = CFStringCreateWithCStringNoCopy(kCFAllocatorDefault,
                                              p->name, kCFStringEncodingUTF8);
 
     if (device_name && printer_name)