From 8dde6c3c0f0d33e3eef59e595b2a5fff816994fc Mon Sep 17 00:00:00 2001 From: Samuel Holland Date: Tue, 16 Jan 2018 23:12:49 -0600 Subject: [PATCH] fragments: Be extra paranoid about the binding getting destroyed Signed-off-by: Samuel Holland --- .../android/fragment/TunnelListFragment.java | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/app/src/main/java/com/wireguard/android/fragment/TunnelListFragment.java b/app/src/main/java/com/wireguard/android/fragment/TunnelListFragment.java index f03aeb9..352fbfb 100644 --- a/app/src/main/java/com/wireguard/android/fragment/TunnelListFragment.java +++ b/app/src/main/java/com/wireguard/android/fragment/TunnelListFragment.java @@ -130,7 +130,8 @@ public class TunnelListFragment extends BaseFragment { public void onRequestCreateConfig(@SuppressWarnings("unused") final View view) { startActivity(new Intent(getActivity(), TunnelCreatorActivity.class)); - binding.createMenu.collapse(); + if (binding != null) + binding.createMenu.collapse(); } public void onRequestImportConfig(@SuppressWarnings("unused") final View view) { @@ -138,7 +139,8 @@ public class TunnelListFragment extends BaseFragment { intent.addCategory(Intent.CATEGORY_OPENABLE); intent.setType("*/*"); startActivityForResult(intent, REQUEST_IMPORT); - binding.createMenu.collapse(); + if (binding != null) + binding.createMenu.collapse(); } @Override @@ -261,14 +263,16 @@ public class TunnelListFragment extends BaseFragment { final int position, final long id) { if (actionMode != null) return false; - binding.tunnelList.setItemChecked(position, true); + if (binding != null) + binding.tunnelList.setItemChecked(position, true); return true; } @Override @SuppressLint("ClickableViewAccessibility") public boolean onTouch(final View view, final MotionEvent motionEvent) { - binding.createMenu.collapse(); + if (binding != null) + binding.createMenu.collapse(); return false; } }