diff options
| author | Daniel Lublin <daniel@lublin.se> | 2020-06-21 08:53:57 +0300 |
|---|---|---|
| committer | Daniel Lublin <daniel@lublin.se> | 2020-06-21 11:44:26 +0300 |
| commit | 3ad85f4685f02b67dbfaf90360393a021f7406cc (patch) | |
| tree | a8d07f86cbf61df949f3cc5a9cf7146c592a0e0b | |
| parent | f09eb2238fe0f0f96ad2c85b8d3ed735e9df969a (diff) | |
Avoid NullPointerException here; must be a racy crash?
I'm somewhat surpsied, but had a stacktrace indicating a crash on this
line. That's very racy then...
| -rw-r--r-- | src/main/java/se/lublin/humla/HumlaService.java | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/main/java/se/lublin/humla/HumlaService.java b/src/main/java/se/lublin/humla/HumlaService.java index e182ba9..975a765 100644 --- a/src/main/java/se/lublin/humla/HumlaService.java +++ b/src/main/java/se/lublin/humla/HumlaService.java @@ -165,7 +165,8 @@ public class HumlaService extends Service implements IHumlaService, IHumlaSessio } ConnectivityManager cm = (ConnectivityManager) getSystemService(CONNECTIVITY_SERVICE); - if (cm.getActiveNetworkInfo() != null && cm.getActiveNetworkInfo().isConnected()) { + NetworkInfo info = cm.getActiveNetworkInfo(); + if (info != null && info.isConnected()) { Log.v(Constants.TAG, "Connectivity restored, attempting reconnect."); connect(); } |