From ce9a59e719019d636af790de1b5185b702250fc1 Mon Sep 17 00:00:00 2001
From: Ivan Alejandro <ivanalejandro0@yahoo.com.ar>
Date: Mon, 1 Jul 2013 16:19:03 -0300
Subject: Check if there is no gateway to use.

Fix typo, improve docstrings.
---
 src/leap/gui/mainwindow.py            | 2 +-
 src/leap/services/eip/vpnlaunchers.py | 8 +++++++-
 src/leap/services/eip/vpnprocess.py   | 2 ++
 3 files changed, 10 insertions(+), 2 deletions(-)

(limited to 'src/leap')

diff --git a/src/leap/gui/mainwindow.py b/src/leap/gui/mainwindow.py
index 94343292..85c66bc0 100644
--- a/src/leap/gui/mainwindow.py
+++ b/src/leap/gui/mainwindow.py
@@ -1006,7 +1006,7 @@ class MainWindow(QtGui.QMainWindow):
                 error=True)
             self._set_eipstatus_off()
         except VPNLauncherException as e:
-            self._status_panel.set_gloal_status("%s" % (e,), error=True)
+            self._status_panel.set_global_status("%s" % (e,), error=True)
             self._set_eipstatus_off()
         else:
             self._already_started_eip = True
diff --git a/src/leap/services/eip/vpnlaunchers.py b/src/leap/services/eip/vpnlaunchers.py
index 570a7893..3cee9bbb 100644
--- a/src/leap/services/eip/vpnlaunchers.py
+++ b/src/leap/services/eip/vpnlaunchers.py
@@ -324,7 +324,9 @@ class LinuxVPNLauncher(VPNLauncher):
         look for openvpn in the regular paths and algo in
         path_prefix/apps/eip/ (in case standalone is set)
 
-        Might raise VPNException.
+        Might raise:
+            VPNLauncherException,
+            OpenVPNNotFoundException.
 
         :param eipconfig: eip configuration object
         :type eipconfig: EIPConfig
@@ -373,6 +375,10 @@ class LinuxVPNLauncher(VPNLauncher):
         gateway_selector = VPNGatewaySelector(eipconfig)
         gateways = gateway_selector.get_gateways()
 
+        if not gateways:
+            logger.error('No gateway was found!')
+            raise VPNLauncherException(self.tr('No gateway was found!'))
+
         logger.debug("Using gateways ips: {}".format(', '.join(gateways)))
 
         for gw in gateways:
diff --git a/src/leap/services/eip/vpnprocess.py b/src/leap/services/eip/vpnprocess.py
index 0ec56ae7..cbf554da 100644
--- a/src/leap/services/eip/vpnprocess.py
+++ b/src/leap/services/eip/vpnprocess.py
@@ -698,6 +698,8 @@ class VPNProcess(protocol.ProcessProtocol, VPNManager):
     def getCommand(self):
         """
         Gets the vpn command from the aproppriate launcher.
+
+        Might throw: VPNLauncherException, OpenVPNNotFoundException.
         """
         cmd = self._launcher.get_vpn_command(
             eipconfig=self._eipconfig,
-- 
cgit v1.2.3