diff options
| -rw-r--r-- | src/leap/bitmask/vpn/fw/firewall.py | 20 |
1 files changed, 13 insertions, 7 deletions
diff --git a/src/leap/bitmask/vpn/fw/firewall.py b/src/leap/bitmask/vpn/fw/firewall.py index 7f3f404..95130b2 100644 --- a/src/leap/bitmask/vpn/fw/firewall.py +++ b/src/leap/bitmask/vpn/fw/firewall.py @@ -23,9 +23,13 @@ import commands import os import subprocess +from twisted.logger import getLogger + from leap.bitmask.vpn.constants import IS_MAC from leap.common.events import catalog, emit_async +log = getLogger() + # TODO -- subclass it for osx/windows, not only for linux. @@ -82,14 +86,16 @@ class FirewallManager(object): if restart: cmd.append("restart") - # FIXME -- use a processprotocol - exitCode = subprocess.call(cmd + gateways) - emit_async(catalog.VPN_STATUS_CHANGED) - - if exitCode == 0: - return True + try: + result = subprocess.check_output( + cmd + gateways, + stderr=subprocess.STDOUT) + except Exception: + log.failure('Error launching the firewall') else: - return False + log.debug(result) + emit_async(catalog.VPN_STATUS_CHANGED) + return True def stop(self): """ |