From 5c126c3207edd6143c27c93630b265cc288cdcc6 Mon Sep 17 00:00:00 2001
From: utox39 <francescomoccaldi39@gmail.com>
Date: Thu, 13 Mar 2025 02:11:08 +0100
Subject: [PATCH] refactor(macos): use errdefer to deallocate in case of error
 instead of doing it manually

---
 src/macos/macos.zig | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/macos/macos.zig b/src/macos/macos.zig
index ee2eb3e..da8806b 100644
--- a/src/macos/macos.zig
+++ b/src/macos/macos.zig
@@ -142,10 +142,10 @@ pub fn getCpuInfo(allocator: std.mem.Allocator) !CpuInfo {
     }
 
     const cpu_name: []u8 = try allocator.alloc(u8, size - 1);
+    errdefer allocator.free(cpu_name);
 
     // Second call to sysctlbyname to get the CPU name
     if (c_sysctl.sysctlbyname("machdep.cpu.brand_string", cpu_name.ptr, &size, null, 0) != 0) {
-        allocator.free(cpu_name);
         return error.FailedToGetCpuName;
     }
 
@@ -315,10 +315,10 @@ pub fn getKernelInfo(allocator: std.mem.Allocator) !KernelInfo {
     }
 
     const kernel_type: []u8 = try allocator.alloc(u8, size - 1);
+    errdefer allocator.free(kernel_type);
 
     // Second call to sysctlbyname to get the kernel name
     if (c_sysctl.sysctlbyname("kern.ostype", kernel_type.ptr, &size, null, 0) != 0) {
-        allocator.free(kernel_type);
         return error.FailedToGetKernelName;
     }
 
@@ -329,10 +329,10 @@ pub fn getKernelInfo(allocator: std.mem.Allocator) !KernelInfo {
     }
 
     const os_release: []u8 = try allocator.alloc(u8, size - 1);
+    errdefer allocator.free(os_release);
 
     // Second call to sysctlbyname to get the kernel release
     if (c_sysctl.sysctlbyname("kern.osrelease", os_release.ptr, &size, null, 0) != 0) {
-        allocator.free(os_release);
         return error.FailedToGetKernelRelease;
     }