diff options
| author | Pavel Labath <labath@google.com> | 2018-05-23 10:10:36 +0000 |
|---|---|---|
| committer | Pavel Labath <labath@google.com> | 2018-05-23 10:10:36 +0000 |
| commit | 2eb720f51e432520e7bb0dfa95ad63658a072f4e (patch) | |
| tree | e8b2a5efaabff8644ab15ff7a8bffdb441973a78 /llvm/lib/Target/TargetMachineC.cpp | |
| parent | d99f3bacb4b92847e8fae3be4cab4acbe107df52 (diff) | |
| download | bcm5719-llvm-2eb720f51e432520e7bb0dfa95ad63658a072f4e.tar.gz bcm5719-llvm-2eb720f51e432520e7bb0dfa95ad63658a072f4e.zip | |
ProcessLauncherPosixFork: move setgid call into the if(debug) branch
This call was originally being only made when launching for debug (as an
attempt to make sure we don't impart extra privileges on the launched
process), but after the debug and non-debug paths were merged, it made
it's way into generic code. This was causing problems in locked down
android environments which disallowed calling setgid even if it would be
a no-op. This prevented launching llgs from lldb-server platform.
Overall I'm not sure we should be calling setgid in the first place
(it seems random -- e.g. why don't we call setuid then as well).
However, all our other copies of launch code have it, so I choose to
keep it for now.
llvm-svn: 333073
Diffstat (limited to 'llvm/lib/Target/TargetMachineC.cpp')
0 files changed, 0 insertions, 0 deletions
