ensure nil error is returned

pkg/drivers/kic/oci/info.go

@@ -37,11 +37,14 @@ var cachedSysInfoErr *error
 
 // CachedDaemonInfo will run and return a docker/podman info only once per minikube run time. to avoid performance
 func CachedDaemonInfo(ociBin string) (SysInfo, error) {
-	if cachedSysInfo == nil || cachedSysInfoErr != nil { // if cached daemon info has error, try to get a new one
+	if cachedSysInfo == nil { // if cached daemon info has error, try to get a new one
 		si, err := DaemonInfo(ociBin)
 		cachedSysInfo = &si
 		cachedSysInfoErr = &err
 	}
+	if cachedSysInfoErr == nil {
+		return *cachedSysInfo, nil
+	}
 	return *cachedSysInfo, *cachedSysInfoErr
 }
 

pkg/minikube/machine/info.go

@@ -111,11 +111,14 @@ var cachedSystemMemoryErr *error
 
 //  cachedSysMemLimit will return a cached limit for the system's virtual memory.
 func cachedSysMemLimit() (*mem.VirtualMemoryStat, error) {
-	if cachedSystemMemoryLimit == nil || cachedSystemMemoryErr != nil {
+	if cachedSystemMemoryLimit == nil {
 		v, err := mem.VirtualMemory()
 		cachedSystemMemoryLimit = v
 		cachedSystemMemoryErr = &err
 	}
+	if cachedSystemMemoryErr == nil {
+		return cachedSystemMemoryLimit, nil
+	}
 	return cachedSystemMemoryLimit, *cachedSystemMemoryErr
 }
 
@@ -124,11 +127,14 @@ var cachedDiskInfoeErr *error
 
 // cachedDiskInfo will return a cached disk usage info
 func cachedDiskInfo() (disk.UsageStat, error) {
-	if cachedDisk == nil || cachedDiskInfoeErr != nil {
+	if cachedDisk == nil {
 		d, err := disk.Usage("/")
 		cachedDisk = d
 		cachedDiskInfoeErr = &err
 	}
+	if cachedDiskInfoeErr == nil {
+		return *cachedDisk, nil
+	}
 	return *cachedDisk, *cachedDiskInfoeErr
 }
 
@@ -137,13 +143,13 @@ var cachedCPUErr *error
 
 //  cachedCPUInfo will return a cached cpu info
 func cachedCPUInfo() ([]cpu.InfoStat, error) {
-	if cachedCPU == nil || cachedCPUErr != nil {
+	if cachedCPU == nil {
 		i, err := cpu.Info()
 		cachedCPU = &i
 		cachedCPUErr = &err
-		if err != nil {
-			return nil, *cachedCPUErr
-		}
+	}
+	if cachedCPUErr == nil {
+		return *cachedCPU, nil
 	}
 	return *cachedCPU, *cachedCPUErr
 }