1
0
mirror of https://github.com/golang/go synced 2024-09-30 08:08:32 -06:00
go/playground
Olivier Wulveryck 2364a5e8fd playground/socket: remove the os cleanup from start method of process
The cleaning process happens in the end method of process, after cmd.Wait.
Deleting it early while the cmd is still running is causing issues for
security tools.

Move the assignment of p.path along with its comment to happen earlier,
after the temporary directory is created. It clarifies that the lack of
call to os.RemoveAll is not a mistake.

Fixes golang/go#40902

Change-Id: I0e555c58893c64a0de5880a5e12e6ff2e01b1eec
Reviewed-on: https://go-review.googlesource.com/c/tools/+/249777
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Alexander Rakoczy <alex@golang.org>
2020-09-03 00:54:29 +00:00
..
socket playground/socket: remove the os cleanup from start method of process 2020-09-03 00:54:29 +00:00
playground.go playground: use stdlib instead of appengine packages 2019-03-07 16:39:23 +00:00