Modified code changes to address the review comments.

Author: mamabusi

CoreFoundation/Base.subproj/CFFileUtilities.c

@@ -1224,11 +1224,11 @@ CFArrayRef _CFXDGCreateDataDirectoriesPaths(void) {
     if ((dataDirectoriesPaths == NULL) || (dataDirectoriesPaths[0] == '\0')) {
         // Environmental variable not set. Return default value.
         CFStringRef defaultPath[2];
-        defaultPath[0] = CFStringCreateWithCString(kCFAllocatorSystemDefault, "/usr/local/share/", _kCFXDGStringEncoding);
-        defaultPath[1] = CFStringCreateWithCString(kCFAllocatorSystemDefault, "/usr/share/", _kCFXDGStringEncoding);
+        defaultPath[0] = CFSTR("/usr/local/share/");
+        defaultPath[1] = CFSTR("/usr/share/");
         return CFArrayCreate(kCFAllocatorSystemDefault, (const void **)defaultPath, 2, &kCFTypeArrayCallBacks);
     }
-    return _CFTokenizeStringToCFArrayOfCFStrings(dataDirectoriesPaths, ':');
+    return _CFCreateCFArrayByTokenizingString(dataDirectoriesPaths, ':');
 }
 
 
@@ -1241,10 +1241,10 @@ CFArrayRef _CFXDGCreateConfigDirectoriesPaths(void) {
     if ((configDirectoriesPaths == NULL) || (configDirectoriesPaths[0] == '\0')) {
         //Environmental variable not set. Return default value.
         CFStringRef defaultPath[1];
-        defaultPath[0] = CFStringCreateWithCString(kCFAllocatorSystemDefault, "/etc/xdg", _kCFXDGStringEncoding);
+        defaultPath[0] = CFSTR("/etc/xdg");
         return CFArrayCreate(kCFAllocatorSystemDefault, (const void **)defaultPath, 1, &kCFTypeArrayCallBacks);
     }
-    return _CFTokenizeStringToCFArrayOfCFStrings(configDirectoriesPaths, ':');
+    return _CFCreateCFArrayByTokenizingString(configDirectoriesPaths, ':');
 }
 
 /// a single base directory relative to which user-specific non-essential (cached) data should be written. This directory is defined by the environment variable $XDG_CACHE_HOME.
@@ -1276,7 +1276,7 @@ CFStringRef _CFXDGCreateRuntimeDirectoryPath(void) {
 }
 
 
-CF_PRIVATE CFArrayRef _CFTokenizeStringToCFArrayOfCFStrings(const char *values, char delimiter) {
+CF_PRIVATE CFArrayRef _CFCreateCFArrayByTokenizingString(const char *values, char delimiter) {
     size_t pathCount = 0;
     char* tmpDirectoriesPaths = (char*)values;
     char* last_colon = 0;
@@ -1304,16 +1304,23 @@ CF_PRIVATE CFArrayRef _CFTokenizeStringToCFArrayOfCFStrings(const char *values,
         while (path)
         {
             assert(validPathCount < pathCount);
-            CFStringRef dirPath = CFStringCreateWithCString(kCFAllocatorSystemDefault, strdup(path), _kCFXDGStringEncoding);
+            char* pathString  = strdup(path);
+            CFStringRef dirPath = CFStringCreateWithCString(kCFAllocatorSystemDefault, pathString, _kCFXDGStringEncoding);
             CFStringRef slash = CFSTR("/");
             CFStringRef tilde = CFSTR("~");
             // Check for absolutePath, if not ignore.
             if (CFStringHasPrefix(dirPath, slash) || CFStringHasPrefix(dirPath, tilde) ) {
                 pathList[validPathCount++] = dirPath;
             }
-            path = strtok(0, delimiterStr);
+            path = strtok(NULL, delimiterStr);
+            free(pathString);
         }
-        return CFArrayCreate(kCFAllocatorSystemDefault, (const void **)pathList, validPathCount, &kCFTypeArrayCallBacks);
+        free(copyDirPath);
+        CFArrayRef pathArray = CFArrayCreate(kCFAllocatorSystemDefault, (const void **)pathList  , validPathCount, &kCFTypeArrayCallBacks);
+        for(int i = 0; i < pathCount; i ++) {
+            CFRelease(pathList[i]);
+        }
+        return pathArray;
     }
     return CFArrayCreate(kCFAllocatorSystemDefault, NULL, 0, &kCFTypeArrayCallBacks);
 }

CoreFoundation/Base.subproj/CFInternal.h

@@ -599,7 +599,7 @@ CF_PRIVATE CFIndex _CFLengthAfterDeletingPathExtension2(CFStringRef path);
 CF_EXPORT CFIndex _CFStartOfPathExtension(UniChar *unichars, CFIndex length);
 CF_PRIVATE CFIndex _CFStartOfPathExtension2(CFStringRef path);
 CF_EXPORT CFIndex _CFLengthAfterDeletingPathExtension(UniChar *unichars, CFIndex length);
-CF_PRIVATE CFArrayRef _CFTokenizeStringToCFArrayOfCFStrings(const char *values, char delimiter);
+CF_PRIVATE CFArrayRef _CFCreateCFArrayByTokenizingString(const char *values, char delimiter);
 
 #if __BLOCKS__
 #if DEPLOYMENT_TARGET_WINDOWS

TestFoundation/XDGTestHelper.swift

@@ -11,9 +11,10 @@ let simpleCookie = HTTPCookie(properties: [
 let rawValue = getenv("XDG_CONFIG_HOME")        
 let xdg_config_home = String(utf8String: rawValue!)
 storage.setCookie(simpleCookie)
-XCTAssertEqual(storage.cookies!.count, 1)
 let fm = FileManager.default
 let destPath = xdg_config_home! + "/.cookies.shared"
 var isDir = false
 let exists = fm.fileExists(atPath: destPath, isDirectory: &isDir) 
-XCTAssertTrue(exists)
+if (!exists) {
+   exit(-1)
+}