From 61f54d3fcd3d12619b254223eb233bda6f8b6bf8 Mon Sep 17 00:00:00 2001 From: Hugo Benichi Date: Thu, 22 Jun 2017 15:19:35 +0900 Subject: IpManager: always log immediate provisioning errors And also migrate remaining Log.e usage to logError. Bug: 62716267 Test: built and flashed Change-Id: I61a6cec081d2bc0fe3d73e46fd52f1a42684e58c --- services/net/java/android/net/ip/IpManager.java | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/services/net/java/android/net/ip/IpManager.java b/services/net/java/android/net/ip/IpManager.java index 7b57bbd397d7..9e2d6961a7b3 100644 --- a/services/net/java/android/net/ip/IpManager.java +++ b/services/net/java/android/net/ip/IpManager.java @@ -714,7 +714,6 @@ public class IpManager extends StateMachine { return shouldLog; } - // TODO: Migrate all Log.e(...) to logError(...). private void logError(String fmt, Object... args) { final String msg = "ERROR " + String.format(fmt, args); Log.e(mTag, msg); @@ -1035,7 +1034,7 @@ public class IpManager extends StateMachine { } private void doImmediateProvisioningFailure(int failureType) { - if (DBG) { Log.e(mTag, "onProvisioningFailure(): " + failureType); } + logError("onProvisioningFailure(): %s", failureType); recordMetric(failureType); mCallback.onProvisioningFailure(new LinkProperties(mLinkProperties)); } @@ -1170,7 +1169,7 @@ public class IpManager extends StateMachine { case DhcpClient.CMD_ON_QUIT: // Everything is already stopped. - Log.e(mTag, "Unexpected CMD_ON_QUIT (already stopped)."); + logError("Unexpected CMD_ON_QUIT (already stopped)."); break; default: @@ -1376,7 +1375,7 @@ public class IpManager extends StateMachine { break; case CMD_START: - Log.e(mTag, "ALERT: START received in StartedState. Please fix caller."); + logError("ALERT: START received in StartedState. Please fix caller."); break; case CMD_CONFIRM: @@ -1475,13 +1474,13 @@ public class IpManager extends StateMachine { handleIPv4Failure(); break; default: - Log.e(mTag, "Unknown CMD_POST_DHCP_ACTION status:" + msg.arg1); + logError("Unknown CMD_POST_DHCP_ACTION status: %s", msg.arg1); } break; case DhcpClient.CMD_ON_QUIT: // DHCPv4 quit early for some reason. - Log.e(mTag, "Unexpected CMD_ON_QUIT."); + logError("Unexpected CMD_ON_QUIT."); mDhcpClient = null; break; -- cgit v1.2.3-59-g8ed1b