summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJoey Grover <joeygrover@gmail.com>2016-11-18 10:29:59 -0500
committerJoey Grover <joeygrover@gmail.com>2016-11-18 10:29:59 -0500
commita0e086594ccb5d4440696e4b492223593f4989c9 (patch)
tree20b7941774c9421a4c75c8f7b8c9a555798dc477
parentf5e92d8b1f6d05776f5b6d54f4e569c81e97330e (diff)
downloadsdl_android-a0e086594ccb5d4440696e4b492223593f4989c9.tar.gz
Fix potential NPE in SdlRouterService WeakRefernce usage
-rw-r--r--sdl_android_lib/src/com/smartdevicelink/transport/SdlRouterService.java11
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: