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

Commit 9bdddd71 authored by Tom Cherry's avatar Tom Cherry
Browse files

init: Remove superfluous error check 

new doesn't return nullptr in C++, so there is no need to check for
it.

Test: Boot bullhead
Change-Id: I666afb1ba7082dd8d5b9911605b7e20a5561b49e
parent 2af784bf

init/service.cpp

+3 −8
Original line number Diff line number Diff line
@@ -889,15 +889,10 @@ Service* ServiceManager::MakeExecOneshotService(const std::vector<std::string>& 
        }

    }



    std::unique_ptr<Service> svc_p(new Service(name, "default", flags, uid, gid, supp_gids,

                                               no_capabilities, namespace_flags, seclabel,

                                               str_args));

    if (!svc_p) {

        LOG(ERROR) << "Couldn't allocate service for exec of '" << str_args[0] << "'";

        return nullptr;

    }

    auto svc_p = std::make_unique<Service>(name, "default", flags, uid, gid, supp_gids,

                                           no_capabilities, namespace_flags, seclabel, str_args);

    Service* svc = svc_p.get();

    services_.push_back(std::move(svc_p));

    services_.emplace_back(std::move(svc_p));



    return svc;

}