diff options
| author | Lorenzo Colitti <lorenzo@google.com> | 2013-11-26 08:38:45 -0800 |
|---|---|---|
| committer | Android Git Automerger <android-git-automerger@android.com> | 2013-11-26 08:38:45 -0800 |
| commit | 6747ea1b1cdf796710c01317da204f50d5721755 (patch) | |
| tree | 3af4f6f650e2677694417dc5bc7a1bcf7421462c /services/java/com | |
| parent | aeefa69a882eb1068da3a45cec82e50971c45df2 (diff) | |
| parent | eede7e89c94b3bb56c26aec137c53d592d3bc7c3 (diff) | |
| download | frameworks_base-6747ea1b1cdf796710c01317da204f50d5721755.zip frameworks_base-6747ea1b1cdf796710c01317da204f50d5721755.tar.gz frameworks_base-6747ea1b1cdf796710c01317da204f50d5721755.tar.bz2 | |
am eede7e89: Merge "Minor cleanups to NetdCallbackReceiver.onEvent."
* commit 'eede7e89c94b3bb56c26aec137c53d592d3bc7c3':
Minor cleanups to NetdCallbackReceiver.onEvent.
Diffstat (limited to 'services/java/com')
| -rw-r--r-- | services/java/com/android/server/NetworkManagementService.java | 23 |
1 files changed, 9 insertions, 14 deletions
diff --git a/services/java/com/android/server/NetworkManagementService.java b/services/java/com/android/server/NetworkManagementService.java index 92f99c2..9a0d648 100644 --- a/services/java/com/android/server/NetworkManagementService.java +++ b/services/java/com/android/server/NetworkManagementService.java @@ -455,6 +455,7 @@ public class NetworkManagementService extends INetworkManagementService.Stub @Override public boolean onEvent(int code, String raw, String[] cooked) { + String errorMessage = String.format("Invalid event from daemon (%s)", raw); switch (code) { case NetdResponseCode.InterfaceChange: /* @@ -465,8 +466,7 @@ public class NetworkManagementService extends INetworkManagementService.Stub * "NNN Iface linkstatus <name> <up/down>" */ if (cooked.length < 4 || !cooked[1].equals("Iface")) { - throw new IllegalStateException( - String.format("Invalid event from daemon (%s)", raw)); + throw new IllegalStateException(errorMessage); } if (cooked[2].equals("added")) { notifyInterfaceAdded(cooked[3]); @@ -481,8 +481,7 @@ public class NetworkManagementService extends INetworkManagementService.Stub notifyInterfaceLinkStateChanged(cooked[3], cooked[4].equals("up")); return true; } - throw new IllegalStateException( - String.format("Invalid event from daemon (%s)", raw)); + throw new IllegalStateException(errorMessage); // break; case NetdResponseCode.BandwidthControl: /* @@ -490,15 +489,13 @@ public class NetworkManagementService extends INetworkManagementService.Stub * Format: "NNN limit alert <alertName> <ifaceName>" */ if (cooked.length < 5 || !cooked[1].equals("limit")) { - throw new IllegalStateException( - String.format("Invalid event from daemon (%s)", raw)); + throw new IllegalStateException(errorMessage); } if (cooked[2].equals("alert")) { notifyLimitReached(cooked[3], cooked[4]); return true; } - throw new IllegalStateException( - String.format("Invalid event from daemon (%s)", raw)); + throw new IllegalStateException(errorMessage); // break; case NetdResponseCode.InterfaceClassActivity: /* @@ -506,8 +503,7 @@ public class NetworkManagementService extends INetworkManagementService.Stub * Format: "NNN IfaceClass <active/idle> <label>" */ if (cooked.length < 4 || !cooked[1].equals("IfaceClass")) { - throw new IllegalStateException( - String.format("Invalid event from daemon (%s)", raw)); + throw new IllegalStateException(errorMessage); } boolean isActive = cooked[2].equals("active"); notifyInterfaceClassActivity(cooked[3], isActive); @@ -519,9 +515,8 @@ public class NetworkManagementService extends INetworkManagementService.Stub * Format: "NNN Address updated <addr> <iface> <flags> <scope>" * "NNN Address removed <addr> <iface> <flags> <scope>" */ - String msg = String.format("Invalid event from daemon (%s)", raw); - if (cooked.length < 6 || !cooked[1].equals("Address")) { - throw new IllegalStateException(msg); + if (cooked.length < 7 || !cooked[1].equals("Address")) { + throw new IllegalStateException(errorMessage); } int flags; @@ -530,7 +525,7 @@ public class NetworkManagementService extends INetworkManagementService.Stub flags = Integer.parseInt(cooked[5]); scope = Integer.parseInt(cooked[6]); } catch(NumberFormatException e) { - throw new IllegalStateException(msg); + throw new IllegalStateException(errorMessage); } if (cooked[2].equals("updated")) { |