From cde606c778a205a9f2ac96353a6353242e19beb0 Mon Sep 17 00:00:00 2001 From: Robin Stocker Date: Tue, 19 Aug 2014 21:52:39 +1000 Subject: [PATCH] Fix IAE in Gerrit configuration page on null scheme URIish#getScheme can return null, handle it. Change-Id: If2cc15049b449f490f67332b914a271075134847 Signed-off-by: Robin Stocker Signed-off-by: Matthias Sohn --- .../org/eclipse/egit/ui/internal/gerrit/GerritConfigurationPage.java | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/org.eclipse.egit.ui/src/org/eclipse/egit/ui/internal/gerrit/GerritConfigurationPage.java b/org.eclipse.egit.ui/src/org/eclipse/egit/ui/internal/gerrit/GerritConfigurationPage.java index 6291137d3..91a4149ea 100644 --- a/org.eclipse.egit.ui/src/org/eclipse/egit/ui/internal/gerrit/GerritConfigurationPage.java +++ b/org.eclipse.egit.ui/src/org/eclipse/egit/ui/internal/gerrit/GerritConfigurationPage.java @@ -281,7 +281,9 @@ class GerritConfigurationPage extends WizardPage { private void checkPage() { try { pushURI = new URIish(uriText.getText()); - scheme.select(scheme.indexOf(pushURI.getScheme())); + String uriScheme = pushURI.getScheme(); + if (uriScheme != null) + scheme.select(scheme.indexOf(uriScheme)); } catch (URISyntaxException e) { setErrorMessage(e.getLocalizedMessage()); setPageComplete(false); -- 2.11.4.GIT