From 4cd743e27eef7495a71cee08a9e8ca971ffcc0a5 Mon Sep 17 00:00:00 2001 From: Kir Kolyshkin Date: Thu, 23 Nov 2023 11:43:56 -0800 Subject: os: use atomic.Uintptr for Process.handle Suggested-by: Michael Knyszek Change-Id: I116731b6c3738aae8ff1d3be227f8f51fa3320c7 Reviewed-on: https://go-review.googlesource.com/c/go/+/544795 LUCI-TryBot-Result: Go LUCI TryBot-Result: Gopher Robot Run-TryBot: Kirill Kolyshkin Reviewed-by: qiulaidongfeng <2645477756@qq.com> Reviewed-by: Dmitri Shuralyov Reviewed-by: Ian Lance Taylor --- src/os/exec.go | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) (limited to 'src/os/exec.go') diff --git a/src/os/exec.go b/src/os/exec.go index ed5a75c4d1..42e8a399a9 100644 --- a/src/os/exec.go +++ b/src/os/exec.go @@ -20,13 +20,14 @@ var ErrProcessDone = errors.New("os: process already finished") // Process stores the information about a process created by StartProcess. type Process struct { Pid int - handle uintptr // handle is accessed atomically on Windows + handle atomic.Uintptr isdone atomic.Bool // process has been successfully waited on sigMu sync.RWMutex // avoid race between wait and signal } func newProcess(pid int, handle uintptr) *Process { - p := &Process{Pid: pid, handle: handle} + p := &Process{Pid: pid} + p.handle.Store(handle) runtime.SetFinalizer(p, (*Process).Release) return p } -- cgit v1.3-5-g9baa