Prevent crash when sharing invalid Wi-Fi config

2019-06-01 07:42:45 +08:00
parent d168dea613
commit 8795941bab
1 changed files with 7 additions and 2 deletions
--- a/mobile/src/main/java/be/mygod/vpnhotspot/net/wifi/configuration/WifiApDialogFragment.kt
+++ b/mobile/src/main/java/be/mygod/vpnhotspot/net/wifi/configuration/WifiApDialogFragment.kt
@@ -187,8 +187,13 @@ class WifiApDialogFragment : AlertDialogFragment<WifiApDialogFragment.Arg, WifiA
                false
            }
            R.id.share_qr -> {
-                QRCodeDialog().withArg(ret.configuration.toQRString())
-                        .show(fragmentManager ?: return false, "QRCodeDialog")
+                val qrString = try {
+                    ret.configuration.toQRString()
+                } catch (e: IllegalArgumentException) {
+                    SmartSnackbar.make(e).show()
+                    return false
+                }
+                QRCodeDialog().withArg(qrString).show(fragmentManager ?: return false, "QRCodeDialog")
                true
            }
            else -> false