diff options
| author | Wander Lairson Costa <wcosta@mozilla.com> | 2017-02-10 04:10:48 -0200 |
|---|---|---|
| committer | Ian Lance Taylor <iant@golang.org> | 2017-02-17 14:36:27 +0000 |
| commit | 79f6a5c7bd684f2e6007ee505b522440beb86bf0 (patch) | |
| tree | dd5f9c8bd97279e54569e2c0029c562226d1e2df /src/sync | |
| parent | 708ba22a0c7b6c2e8f46fccb35998c21c60629b9 (diff) | |
| download | go-79f6a5c7bd684f2e6007ee505b522440beb86bf0.tar.xz | |
syscall: only call setgroups if we need to
If the caller set ups a Credential in os/exec.Command,
os/exec.Command.Start will end up calling setgroups(2), even if no
supplementary groups were given.
Only root can call setgroups(2) on BSD kernels, which causes Start to
fail for non-root users when they try to set uid and gid for the new
process.
We fix by introducing a new field to syscall.Credential named
NoSetGroups, and setgroups(2) is only called if it is false.
We make this field with inverted logic to preserve backward
compatibility.
RELNOTES=yes
Change-Id: I3cff1f21c117a1430834f640ef21fd4e87e06804
Reviewed-on: https://go-review.googlesource.com/36697
Reviewed-by: Ian Lance Taylor <iant@golang.org>
Diffstat (limited to 'src/sync')
0 files changed, 0 insertions, 0 deletions