The os.Process struct has unexported fields, so you can't expect to create a valid one using a struct literal.

Instead, look up the desired PID using os.FindProcess.

We expected this to work because it was working in 1.16.14, so 1.17 breaks backwards compatibility. This seems to be the change that broke us 105c5b5?diff=split . When I run with the function

func TerminateProcess(process os.Process, exitCode int) error {
	h, e := syscall.OpenProcess(syscall.PROCESS_TERMINATE, false, uint32(process.Pid))
	if e != nil {
		return os.NewSyscallError("OpenProcess", e)
	}
	defer syscall.CloseHandle(h)
	e = syscall.TerminateProcess(h, uint32(exitCode))
	if e != nil {
		return os.NewSyscallError("TerminateProcess", e)
	}

	runtime.KeepAlive(process)
	return nil
}

execution succeeds.

Source Code: https://go.dev/play/p/mELlwQqiBZa

(Note: When we run os.FindProcess on 1.16 and 1.17 it returns "OpenProcess: Access is denied." for our processes)

Writing

	process := os.Process{
		Pid: pid,
	}

was never supposed to work. I'm sorry that it did.

In this case, it should have been documented that it cannot work. While there are unexported members, it isn't immediately clear from the docs that Kill will not work.

To me it seems clear from the documentation which explicitly says that Process stores information about a process created by StartProcess. But if someone wants to send a brief and to the point update to the docs we can certainly take a look. Thanks.