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

Commit 4092a777 authored by Rajesh Nyamagoud's avatar Rajesh Nyamagoud
Browse files

Replaced `-1` with `EXIT_FAILURE` and validated confirmationui service 

status after registering it with `CHECK_EQ` macro.

Bug: b/205760172
Test: atest VtsHalConfirmationUITargetTest
Change-Id: I00f5a09ca525e3abb63a5d1f404fb6f3daed9442
parent 882c1681

trusty/confirmationui/service.cpp

+2 −6
Original line number Diff line number Diff line
@@ -31,12 +31,8 @@ int main() { 
    const auto instance = std::string(IConfirmationUI::descriptor) + "/default";

    binder_status_t status =

        AServiceManager_addService(confirmationui->asBinder().get(), instance.c_str());



    if (status != STATUS_OK) {

        LOG(FATAL) << "Could not register service for " << instance.c_str() << "(" << status << ")";

        return -1;

    }

    CHECK_EQ(status, STATUS_OK) << "Could not register " << instance;



    ABinderProcess_joinThreadPool();

    return -1;

    return EXIT_FAILURE;

}