Reconcile car watchdog daemon to CarWatchdogService

- When there is a duplicated client registration, CarWatchdogService and
CarWatchdogManager don't generate errors.
- Car watchdog daemon will follow the same behavior.

Bug: 149875865
Test: From the same client, register the client two times and make sure
that registerClient doesn't return error.

Change-Id: I7a817647ad83eb999fc94dc7a7966f16144b9b51

1 files changed, 2 insertions, 4 deletions

diff --git a/watchdog/server/src/WatchdogProcessService.cpp b/watchdog/server/src/WatchdogProcessService.cpp
index 51d1f07bb..b4392cbb4 100644
--- a/watchdog/server/src/WatchdogProcessService.cpp
+++ b/watchdog/server/src/WatchdogProcessService.cpp
@@ -346,10 +346,8 @@ Status WatchdogProcessService::registerClientLocked(const sp<ICarWatchdogClient>
                                                     TimeoutLength timeout, ClientType clientType) {
     const char* clientName = clientType == ClientType::Regular ? "client" : "mediator";
     if (isRegisteredLocked(client)) {
-        std::string errorStr = StringPrintf("The %s is already registered.", clientName);
-        const char* errorCause = errorStr.c_str();
-        ALOGW("Cannot register the %s. %s", clientName, errorCause);
-        return Status::fromExceptionCode(Status::EX_ILLEGAL_ARGUMENT, errorCause);
+        ALOGW("Cannot register the %s: the %s is already registered.", clientName, clientName);
+        return Status::ok();
     }
     sp<IBinder> binder = BnCarWatchdog::asBinder(client);
     status_t status = binder->linkToDeath(this);