Merge pull request #1722 from uubulb/fix_windows_mem

shirou · web-flow · commit 44c71b62c0f3 · 2024-10-11T15:45:29.000+09:00
fix(mem): possible memory leak on Windows
diff --git a/mem/mem_windows.go b/mem/mem_windows.go
@@ -82,6 +82,8 @@ func SwapMemoryWithContext(ctx context.Context) (*SwapMemoryStat, error) {
 	if err != nil {
 		return nil, err
 	}
+	defer common.PdhCloseQuery.Call(uintptr(counter.Query))
+
 	usedPercent, err := counter.GetValue()
 	if err != nil {
 		return nil, err

Original file line number	Diff line number	Diff line change
`@@ -82,6 +82,8 @@ func SwapMemoryWithContext(ctx context.Context) (*SwapMemoryStat, error) {`
`82`	`82`	`if err != nil {`
`83`	`83`	`return nil, err`
`84`	`84`	`}`
	`85`	`+ defer common.PdhCloseQuery.Call(uintptr(counter.Query))`
	`86`	`+`
`85`	`87`	`usedPercent, err := counter.GetValue()`
`86`	`88`	`if err != nil {`
`87`	`89`	`return nil, err`