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

Commit 0e8f83f6 authored by Matthew Sedam's avatar Matthew Sedam
Browse files

ContextHubClientManager: Fix logging bug 

The function should not use Log.wtf as that could
crash. It should use Log.e and return an error like
other failures in this function.

Bug: 331795143
Change-Id: I52e4014b32a0f76f11d06a1c87854c58ea226364
Flag: EXEMPT bug fix
Test: Presubmits
parent 7d7d8e38

services/core/java/com/android/server/location/contexthub/ContextHubClientManager.java

+2 −1
Original line number Diff line number Diff line
@@ -244,8 +244,9 @@ import java.util.function.Consumer; 
            // Broadcast messages shouldn't be sent with any permissions tagged per CHRE API

            // requirements.

            if (!messagePermissions.isEmpty()) {

                Log.wtf(TAG, "Received broadcast message with permissions from "

                Log.e(TAG, "Received broadcast message with permissions from "

                        + message.getNanoAppId());

                return ErrorCode.PERMANENT_ERROR;

            }



            ContextHubEventLogger.getInstance().logMessageFromNanoapp(contextHubId, message,