never update ProviderOberservable with a null provider, use a default provider object instead

1 files changed, 3 insertions, 1 deletions

diff --git a/app/src/main/java/se/leap/bitmaskclient/base/models/ProviderObservable.java b/app/src/main/java/se/leap/bitmaskclient/base/models/ProviderObservable.java
index 19555504..3e1e1fcc 100644
--- a/app/src/main/java/se/leap/bitmaskclient/base/models/ProviderObservable.java
+++ b/app/src/main/java/se/leap/bitmaskclient/base/models/ProviderObservable.java
@@ -1,5 +1,7 @@
 package se.leap.bitmaskclient.base.models;
 
+import androidx.annotation.NonNull;
+
 import java.util.Observable;
 
 /**
@@ -17,7 +19,7 @@ public class ProviderObservable extends Observable {
         return instance;
     }
 
-    public synchronized void updateProvider(Provider provider) {
+    public synchronized void updateProvider(@NonNull Provider provider) {
         instance.currentProvider = provider;
         instance.providerForDns = null;
         instance.setChanged();