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

Commit 328e7d71 authored by Christopher Tate's avatar Christopher Tate Committed by Chris Tate
Browse files

It's legit for AMS to throw IllegalArgumentException 

So let's not log a WTF about it, hmm?

Change-Id: Ibaf72d883a85c6e526d5f707e90742b8f58359d3
Fixes: 68993753
Test: manual
parent 34116c78

services/core/java/com/android/server/am/ActivityManagerService.java

+4 −2
Original line number Diff line number Diff line
@@ -2942,9 +2942,11 @@ public class ActivityManagerService extends IActivityManager.Stub 
        try {

            return super.onTransact(code, data, reply, flags);

        } catch (RuntimeException e) {

            // The activity manager only throws security exceptions, so let's

            // The activity manager only throws certain exceptions intentionally, so let's

            // log all others.

            if (!(e instanceof SecurityException)) {

            if (!(e instanceof SecurityException

                    || e instanceof IllegalArgumentException

                    || e instanceof IllegalStateException)) {

                Slog.wtf(TAG, "Activity Manager Crash."

                        + " UID:" + Binder.getCallingUid()

                        + " PID:" + Binder.getCallingPid()