diff --git a/src/detection/battery/battery_apple.c b/src/detection/battery/battery_apple.c index 53a2431e5a..3af71bf56e 100644 --- a/src/detection/battery/battery_apple.c +++ b/src/detection/battery/battery_apple.c @@ -13,7 +13,7 @@ const char* ffDetectBatteryImpl(FFinstance* instance, FFlist* results) return "IOServiceMatching(\"AppleSmartBattery\") failed"; io_iterator_t iterator; - if(IOServiceGetMatchingServices(0, matchDict, &iterator) != kIOReturnSuccess) + if(IOServiceGetMatchingServices(MACH_PORT_NULL, matchDict, &iterator) != kIOReturnSuccess) return "IOServiceGetMatchingServices() failed"; io_registry_entry_t registryEntry; @@ -26,18 +26,19 @@ const char* ffDetectBatteryImpl(FFinstance* instance, FFlist* results) continue; } - int intValue; bool boolValue; BatteryResult* battery = ffListAdd(results); ffStrbufInit(&battery->capacity); - if(ffCfDictGetInt(properties, "CurrentCapacity", &intValue)) - ffStrbufAppendF(&battery->capacity, "%d", intValue); + int currentCapacity, maxCapacity; + if(ffCfDictGetInt(properties, CFSTR("CurrentCapacity"), ¤tCapacity) && + ffCfDictGetInt(properties, CFSTR("MaxCapacity"), &maxCapacity)) + ffStrbufAppendF(&battery->capacity, "%.0f", currentCapacity * 100.0 / maxCapacity); ffStrbufInit(&battery->manufacturer); ffStrbufInit(&battery->modelName); ffStrbufInit(&battery->technology); - if (ffCfDictGetBool(properties, "built-in", &boolValue) && boolValue) + if (ffCfDictGetBool(properties, CFSTR("built-in"), &boolValue) && boolValue) { ffStrbufAppendS(&battery->manufacturer, "Apple Inc."); ffStrbufAppendS(&battery->modelName, "Builtin"); @@ -51,9 +52,9 @@ const char* ffDetectBatteryImpl(FFinstance* instance, FFlist* results) } ffStrbufInit(&battery->status); - if (ffCfDictGetBool(properties, "FullyCharged", &boolValue) && boolValue) + if (ffCfDictGetBool(properties, CFSTR("FullyCharged"), &boolValue) && boolValue) ffStrbufAppendS(&battery->status, "Fully charged"); - else if (ffCfDictGetBool(properties, "IsCharging", &boolValue) && boolValue) + else if (ffCfDictGetBool(properties, CFSTR("IsCharging"), &boolValue) && boolValue) ffStrbufAppendS(&battery->status, "Charging"); else ffStrbufAppendS(&battery->status, ""); diff --git a/src/detection/battery/battery_linux.c b/src/detection/battery/battery_linux.c index db6aee1629..909ed493f0 100644 --- a/src/detection/battery/battery_linux.c +++ b/src/detection/battery/battery_linux.c @@ -78,10 +78,7 @@ const char* ffDetectBatteryImpl(FFinstance* instance, FFlist* results) { if(instance->config.batteryDir.length > 0) { ffStrbufAppend(&baseDir, &instance->config.batteryDir); - if (!ffStrbufEndsWithC(&baseDir, '/')) - { - ffStrbufAppendC(&baseDir, '/'); - } + ffStrbufEnsureEndsWithC(&baseDir, '/'); } else { diff --git a/src/detection/gpu/gpu_apple.c b/src/detection/gpu/gpu_apple.c index b8a2b34e69..e756b9467f 100644 --- a/src/detection/gpu/gpu_apple.c +++ b/src/detection/gpu/gpu_apple.c @@ -11,7 +11,7 @@ const char* ffDetectGPUImpl(FFlist* gpus, const FFinstance* instance) CFMutableDictionaryRef matchDict = IOServiceMatching(kIOAcceleratorClassName); io_iterator_t iterator; - if(IOServiceGetMatchingServices(0, matchDict, &iterator) != kIOReturnSuccess) + if(IOServiceGetMatchingServices(MACH_PORT_NULL, matchDict, &iterator) != kIOReturnSuccess) return "IOServiceGetMatchingServices() failed"; io_registry_entry_t registryEntry; @@ -26,16 +26,31 @@ const char* ffDetectGPUImpl(FFlist* gpus, const FFinstance* instance) FFGPUResult* gpu = ffListAdd(gpus); - ffStrbufInit(&gpu->name); - ffCfDictGetString(properties, "model", &gpu->name); - - if (!ffCfDictGetInt(properties, "gpu-core-count", &gpu->coreCount)) - gpu->coreCount = FF_GPU_CORE_COUNT_UNSET; - ffStrbufInitA(&gpu->vendor, 0); ffStrbufInit(&gpu->driver); - ffCfDictGetString(properties, "CFBundleIdentifier", &gpu->driver); + ffCfDictGetString(properties, CFSTR("CFBundleIdentifier"), &gpu->driver); + + ffStrbufInit(&gpu->name); + //IOAccelerator returns model property for Apple Silicon, but not for Intel Iris GPUs. + //Still needs testing for AMD's + if(!ffCfDictGetString(properties, CFSTR("model"), &gpu->name)) + { + CFRelease(properties); + + io_registry_entry_t parentEntry; + IORegistryEntryGetParentEntry(registryEntry, kIOServicePlane, &parentEntry); + if(IORegistryEntryCreateCFProperties(parentEntry, &properties, kCFAllocatorDefault, kNilOptions) != kIOReturnSuccess) + { + IOObjectRelease(parentEntry); + IOObjectRelease(registryEntry); + continue; + } + ffCfDictGetString(properties, CFSTR("model"), &gpu->name); + } + + if(!ffCfDictGetInt(properties, CFSTR("gpu-core-count"), &gpu->coreCount)) + gpu->coreCount = FF_GPU_CORE_COUNT_UNSET; gpu->temperature = FF_GPU_TEMP_UNSET; diff --git a/src/detection/vulkan.c b/src/detection/vulkan.c index 108101174b..944a5c8edf 100644 --- a/src/detection/vulkan.c +++ b/src/detection/vulkan.c @@ -191,6 +191,7 @@ static const char* detectVulkan(const FFinstance* instance, FFVulkanResult* resu //No way to detect those using vulkan ffStrbufInit(&gpu->vendor); ffStrbufInit(&gpu->driver); + gpu->coreCount = FF_GPU_CORE_COUNT_UNSET; gpu->temperature = FF_GPU_TEMP_UNSET; } diff --git a/src/util/apple/cfdict_helpers.c b/src/util/apple/cfdict_helpers.c index 9da7ca917f..f03c1dcc45 100644 --- a/src/util/apple/cfdict_helpers.c +++ b/src/util/apple/cfdict_helpers.c @@ -1,42 +1,53 @@ #include "cfdict_helpers.h" -const void* ffCfDictGetValue(CFMutableDictionaryRef dict, const char* key) +bool ffCfDictGetString(CFMutableDictionaryRef dict, CFStringRef key, FFstrbuf* result) { - CFStringRef cfKey = CFStringCreateWithCStringNoCopy(NULL, key, kCFStringEncodingASCII, kCFAllocatorNull); - return CFDictionaryGetValue(dict, cfKey); -} - -bool ffCfDictGetString(CFMutableDictionaryRef dict, const char* key, FFstrbuf* result) -{ - CFStringRef cf = (CFStringRef)ffCfDictGetValue(dict, key); - if(cf == NULL || CFGetTypeID(cf) != CFStringGetTypeID()) + CFTypeRef cf = (CFTypeRef)CFDictionaryGetValue(dict, key); + if(cf == NULL) return false; - uint32_t length = (uint32_t)CFStringGetLength(cf); - ffStrbufEnsureFree(result, length + 1); - if(CFStringGetCString(cf, result->chars, length + 1, kCFStringEncodingASCII)) + if(CFGetTypeID(cf) == CFStringGetTypeID()) + { + CFStringRef cfStr = (CFStringRef)cf; + uint32_t length = (uint32_t)CFStringGetLength(cfStr); + ffStrbufEnsureFree(result, length + 1); + if(CFStringGetCString(cfStr, result->chars, length + 1, kCFStringEncodingASCII)) + { + result->length = length; + // CFStringGetCString ensures the buffer is NUL terminated + // https://developer.apple.com/documentation/corefoundation/1542721-cfstringgetcstring + } + } + else if(CFGetTypeID(cf) == CFDataGetTypeID()) + { + CFDataRef cfData = (CFDataRef)cf; + uint32_t length = (uint32_t)CFDataGetLength(cfData); + ffStrbufEnsureFree(result, length + 1); + CFDataGetBytes(cfData, CFRangeMake(0, length), (uint8_t*)result->chars); + result->length = (uint32_t)strnlen(result->chars, length); + result->chars[result->length] = '\0'; + } + else { - result->length = length; - // CFStringGetCString ensures the buffer is NUL terminated - // https://developer.apple.com/documentation/corefoundation/1542721-cfstringgetcstring + return false; } return true; } -bool ffCfDictGetBool(CFMutableDictionaryRef dict, const char* key, bool* result) +bool ffCfDictGetBool(CFMutableDictionaryRef dict, CFStringRef key, bool* result) { - CFBooleanRef cf = (CFBooleanRef)ffCfDictGetValue(dict, key); - if(cf == NULL || CFGetTypeID(cf) != CFNumberGetTypeID()) + CFBooleanRef cf = (CFBooleanRef)CFDictionaryGetValue(dict, key); + if(cf == NULL || CFGetTypeID(cf) != CFBooleanGetTypeID()) return false; *result = CFBooleanGetValue(cf); return true; } -bool ffCfDictGetInt(CFMutableDictionaryRef dict, const char* key, int* result) +bool ffCfDictGetInt(CFMutableDictionaryRef dict, CFStringRef key, int* result) { - CFNumberRef cf = (CFNumberRef)ffCfDictGetValue(dict, key); - if (cf == NULL || CFGetTypeID(cf) != CFStringGetTypeID()) + CFNumberRef cf = (CFNumberRef)CFDictionaryGetValue(dict, key); + if (cf == NULL || CFGetTypeID(cf) != CFNumberGetTypeID()) return false; if(!CFNumberGetValue(cf, kCFNumberSInt32Type, result)) diff --git a/src/util/apple/cfdict_helpers.h b/src/util/apple/cfdict_helpers.h index a00522e740..cfb1e3c81d 100644 --- a/src/util/apple/cfdict_helpers.h +++ b/src/util/apple/cfdict_helpers.h @@ -6,9 +6,8 @@ #include "fastfetch.h" #include -const void* ffCfDictGetValue(CFMutableDictionaryRef dict, const char* str); -bool ffCfDictGetString(CFMutableDictionaryRef dict, const char* key, FFstrbuf* result); -bool ffCfDictGetBool(CFMutableDictionaryRef dict, const char* key, bool* result); -bool ffCfDictGetInt(CFMutableDictionaryRef dict, const char* key, int* result); +bool ffCfDictGetString(CFMutableDictionaryRef dict, CFStringRef key, FFstrbuf* result); +bool ffCfDictGetBool(CFMutableDictionaryRef dict, CFStringRef key, bool* result); +bool ffCfDictGetInt(CFMutableDictionaryRef dict, CFStringRef key, int* result); #endif