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

Commit 8730fded authored by Zach Riggle's avatar Zach Riggle Committed by android-build-merger
Browse files

Merge "Prevent otapreopt_chroot from crashing with no arguments" am: aaf2c87e am: 7cc6cd77

am: 27de5b1d

Change-Id: I217549967ab3c5e4bcea99b43eceac43c3ab7a1f
parents 0dc9685a 27de5b1d
Loading
Loading
Loading
Loading
+6 −0
Original line number Diff line number Diff line
@@ -59,6 +59,12 @@ static void CloseDescriptor(const char* descriptor_string) {
// The file descriptor denoted by status-fd will be closed. The rest of the parameters will
// be passed on to otapreopt in the chroot.
static int otapreopt_chroot(const int argc, char **arg) {
    // Validate arguments
    // We need the command, status channel and target slot, at a minimum.
    if(argc < 3) {
        PLOG(ERROR) << "Not enough arguments.";
        exit(208);
    }
    // Close all file descriptors. They are coming from the caller, we do not want to pass them
    // on across our fork/exec into a different domain.
    // 1) Default descriptors.