diff options
| author | cyberta <cyberta@riseup.net> | 2026-01-24 16:51:13 +0100 |
|---|---|---|
| committer | cyberta <cyberta@riseup.net> | 2026-01-26 14:34:00 +0100 |
| commit | 7fb17bc0c939a9614b5665368d1ff4506bc001f9 (patch) | |
| tree | c1423d3f7141c4665f9af5d64f115e20f32db316 /app | |
| parent | b482cf94ef52bb19e089800c7b773cd0ca2ab8f9 (diff) | |
fix potential NPE in ConfigureProviderFragment
Diffstat (limited to 'app')
| -rw-r--r-- | app/src/main/java/se/leap/bitmaskclient/providersetup/fragments/ConfigureProviderFragment.java | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/app/src/main/java/se/leap/bitmaskclient/providersetup/fragments/ConfigureProviderFragment.java b/app/src/main/java/se/leap/bitmaskclient/providersetup/fragments/ConfigureProviderFragment.java index dde3ed15..a8a90b1e 100644 --- a/app/src/main/java/se/leap/bitmaskclient/providersetup/fragments/ConfigureProviderFragment.java +++ b/app/src/main/java/se/leap/bitmaskclient/providersetup/fragments/ConfigureProviderFragment.java @@ -167,7 +167,7 @@ public class ConfigureProviderFragment extends BaseSetupFragment implements Prop super.onScrollStateChanged(recyclerView, newState); if (newState != SCROLL_STATE_IDLE) { torLogAdapter.postponeUpdate = true; - } else if (newState == SCROLL_STATE_IDLE && getFirstVisibleItemPosion() == 0) { + } else if (newState == SCROLL_STATE_IDLE && getFirstVisibleItemPosition() == 0) { torLogAdapter.postponeUpdate = false; } } @@ -177,8 +177,12 @@ public class ConfigureProviderFragment extends BaseSetupFragment implements Prop binding.torState.setText(getLastTorLog()); } - private int getFirstVisibleItemPosion() { - return ((LinearLayoutManager) binding.connectionDetailLogs.getLayoutManager()).findFirstVisibleItemPosition(); + private int getFirstVisibleItemPosition() { + try { + return ((LinearLayoutManager) binding.connectionDetailLogs.getLayoutManager()).findFirstVisibleItemPosition(); + } catch (NullPointerException npe) { + return 0; + } } @Override |
