Fix analyser warnings

acidanthera · Jul 4, 2018 · b5a0e77 · b5a0e77
1 parent d2f0487
commit b5a0e77
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/Lilu/Sources/kern_cpu.cpp b/Lilu/Sources/kern_cpu.cpp
@@ -148,6 +148,7 @@ uint32_t CPUInfo::getGpuPlatformId(IORegistryEntry *sect, bool *specified) {
 			else
 				source = "(not found)";
 		}
+		(void)source;
 		DBGLOG("cpu", "found %s with frame id %08x via %s", safeString(sect->getName()), platform, source);
 	} else {
 		DBGLOG("cpu", "failed to detect built-in GPU");

diff --git a/Lilu/Sources/kern_user.cpp b/Lilu/Sources/kern_user.cpp
@@ -555,8 +555,9 @@ void UserPatcher::patchSharedCache(vm_map_t taskPort, uint32_t slide, cpu_type_t
 									DBGLOG("user", "obtained write permssions\n");
 
 									r = orgVmMapWriteUser(taskPort, applyChanges ? patch.replace : patch.find, place, patch.size);
-
-									DBGLOG("user", "patching %llX -> res %d", place, r);
+
+									if (r != KERN_SUCCESS)
+										SYSLOG("user", "patching %llX -> res %d", place, r);
 
 									if (vm_protect(taskPort, (place & -PAGE_SIZE), PAGE_SIZE, FALSE, VM_PROT_READ|VM_PROT_EXECUTE) == KERN_SUCCESS) {
 										DBGLOG("user", "restored write permssions\n");