Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit f6b82314 authored by Treehugger Robot's avatar Treehugger Robot Committed by Gerrit Code Review
Browse files

Merge "init: Cleanly shut down subcontext processes"

parents d81c4f07 92c49bcb

init/reboot.cpp

+1 −0
Original line number Diff line number Diff line
@@ -443,6 +443,7 @@ void DoReboot(unsigned int cmd, const std::string& reason, const std::string& re 
    for (const auto& s : ServiceList::GetInstance().services_in_shutdown_order()) {

        if (!s->IsShutdownCritical()) s->Stop();

    }

    SubcontextTerminate();

    ReapAnyOutstandingChildren();



    // 3. send volume shutdown to vold

init/subcontext.cpp

+11 −1
Original line number Diff line number Diff line
@@ -352,6 +352,7 @@ Result<std::vector<std::string>> Subcontext::ExpandArgs(const std::vector<std::s 
}



static std::vector<Subcontext> subcontexts;

static bool shutting_down;



std::vector<Subcontext>* InitializeSubcontexts() {

    if (SelinuxHasVendorInit()) {
@@ -365,12 +366,21 @@ std::vector<Subcontext>* InitializeSubcontexts() { 
bool SubcontextChildReap(pid_t pid) {

    for (auto& subcontext : subcontexts) {

        if (subcontext.pid() == pid) {

            if (!shutting_down) {

                subcontext.Restart();

            }

            return true;

        }

    }

    return false;

}



void SubcontextTerminate() {

    shutting_down = true;

    for (auto& subcontext : subcontexts) {

        kill(subcontext.pid(), SIGTERM);

    }

}



}  // namespace init

}  // namespace android

init/subcontext.h

+1 −0
Original line number Diff line number Diff line
@@ -63,6 +63,7 @@ class Subcontext { 
int SubcontextMain(int argc, char** argv, const KeywordFunctionMap* function_map);

std::vector<Subcontext>* InitializeSubcontexts();

bool SubcontextChildReap(pid_t pid);

void SubcontextTerminate();



}  // namespace init

}  // namespace android