OnReleaseHardware: Close EC driver handle

JohnAZoidberg · JohnAZoidberg · commit 9d428b577787 · 2025-03-26T16:29:03.000+08:00
Previously the following HLK tests would fail and also uninstalling the
device from device manager would trigger a debugger break.

Already succesfully tested
DF - PNP DIF Remove Device Test (Reliability)
DF - PNP Rebalance Fail Restart Device Test (Reliability)

Still retesting
DF - Reinstall with IO Before and After (Reliability)
DF - Sleep and PNP (disable and enable) with IO Before and After (Reliability)

Signed-off-by: Daniel Schaefer &lt;dhs@frame.work&gt;
diff --git a/FrameworkSensors/Device.cpp b/FrameworkSensors/Device.cpp
@@ -403,6 +403,12 @@ OnReleaseHardware(
             pDevice->m_Lock = NULL;
         }
 
+        // Close handle to EC driver
+        if (pDevice->m_CrosEcHandle && pDevice->m_CrosEcHandle != INVALID_HANDLE_VALUE) {
+            CloseHandle(pDevice->m_CrosEcHandle);
+            pDevice->m_CrosEcHandle = INVALID_HANDLE_VALUE;
+        }
+
         // Delete sensor instance
         if (NULL != pDevice->m_SensorInstance)
         {

Original file line number	Diff line number	Diff line change
`@@ -403,6 +403,12 @@ OnReleaseHardware(`
`403`	`403`	`pDevice->m_Lock = NULL;`
`404`	`404`	`}`
`405`	`405`
	`406`	`+ // Close handle to EC driver`
	`407`	`+ if (pDevice->m_CrosEcHandle && pDevice->m_CrosEcHandle != INVALID_HANDLE_VALUE) {`
	`408`	`+ CloseHandle(pDevice->m_CrosEcHandle);`
	`409`	`+ pDevice->m_CrosEcHandle = INVALID_HANDLE_VALUE;`
	`410`	`+ }`
	`411`	`+`
`406`	`412`	`// Delete sensor instance`
`407`	`413`	`if (NULL != pDevice->m_SensorInstance)`
`408`	`414`	`{`