Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Set idmap2 binary uid and gid after forking" into qt-dev am: ebad9125c8 am: e9acb9c566

Browse Source 
am: 18e2a06453

Change-Id: I0bf570d5c4198f3ff9e5c4c62c8b88bc48db7ec6
This commit is contained in:
Ryan Mitchell
2019-07-02 12:28:17 -07:00
committed by android-build-merger
parent cf2fe2c291 18e2a06453
commit 913c8074a7
1 changed files with 13 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
libs/androidfw/PosixUtils.cpp
View File

@@ -64,6 +64,9 @@ std::unique_ptr<ProcResult> ExecuteBinary(const std::vector<std::string>& argv)
return nullptr;
}
auto gid = getgid();
auto uid = getuid();
char const** argv0 = (char const**)malloc(sizeof(char*) * (argv.size() + 1));
for (size_t i = 0; i < argv.size(); i++) {
argv0[i] = argv[i].c_str();
@@ -75,6 +78,16 @@ std::unique_ptr<ProcResult> ExecuteBinary(const std::vector<std::string>& argv)
PLOG(ERROR) << "fork";
return nullptr;
case 0: // child
if (setgid(gid) != 0) {
PLOG(ERROR) << "setgid";
exit(1);
}
if (setuid(uid) != 0) {
PLOG(ERROR) << "setuid";
exit(1);
}
close(stdout[0]);
if (dup2(stdout[1], STDOUT_FILENO) == -1) {
abort();