diff options
author | Joey Grover <joeygrover@gmail.com> | 2016-11-18 10:29:59 -0500 |
---|---|---|
committer | Joey Grover <joeygrover@gmail.com> | 2016-11-18 10:29:59 -0500 |
commit | a0e086594ccb5d4440696e4b492223593f4989c9 (patch) | |
tree | 20b7941774c9421a4c75c8f7b8c9a555798dc477 | |
parent | f5e92d8b1f6d05776f5b6d54f4e569c81e97330e (diff) | |
download | sdl_android-a0e086594ccb5d4440696e4b492223593f4989c9.tar.gz |
Fix potential NPE in SdlRouterService WeakRefernce usage
-rw-r--r-- | sdl_android_lib/src/com/smartdevicelink/transport/SdlRouterService.java | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/sdl_android_lib/src/com/smartdevicelink/transport/SdlRouterService.java b/sdl_android_lib/src/com/smartdevicelink/transport/SdlRouterService.java index 254556014..d5a766b64 100644 --- a/sdl_android_lib/src/com/smartdevicelink/transport/SdlRouterService.java +++ b/sdl_android_lib/src/com/smartdevicelink/transport/SdlRouterService.java @@ -86,7 +86,7 @@ public class SdlRouterService extends Service{ /** * <b> NOTE: DO NOT MODIFY THIS UNLESS YOU KNOW WHAT YOU'RE DOING.</b> */ - protected static final int ROUTER_SERVICE_VERSION_NUMBER = 2; + protected static final int ROUTER_SERVICE_VERSION_NUMBER = 3; private static final String ROUTER_SERVICE_PROCESS = "com.smartdevicelink.router"; @@ -328,6 +328,9 @@ public class SdlRouterService extends Service{ @Override public void handleMessage(Message msg) { + if(this.provider == null || this.provider.get() == null){ + return; + } final Bundle receivedBundle = msg.getData(); Bundle returnBundle; final SdlRouterService service = this.provider.get(); @@ -552,6 +555,9 @@ public class SdlRouterService extends Service{ @Override public void handleMessage(Message msg) { + if(this.provider == null || this.provider.get() == null){ + return; + } SdlRouterService service = this.provider.get(); Bundle receivedBundle = msg.getData(); switch(msg.what){ @@ -634,6 +640,9 @@ public class SdlRouterService extends Service{ @Override public void handleMessage(Message msg) { + if(this.provider == null || this.provider.get() == null){ + return; + } SdlRouterService service = this.provider.get(); switch(msg.what){ case TransportConstants.ROUTER_STATUS_CONNECTED_STATE_REQUEST: |