diff options
| author | Kali Kaneko <kali@leap.se> | 2013-06-23 23:54:22 +0900 |
|---|---|---|
| committer | Kali Kaneko <kali@leap.se> | 2013-06-23 23:54:22 +0900 |
| commit | 4a0536e5831eeac539c1b746932e60d4dd5e9591 (patch) | |
| tree | 16c4590c7bfce754c31a1fa87326c0b29baf5a11 /src/leap | |
| parent | 53668de6f4ced844b10fc34fbc5ba77b00d54825 (diff) | |
| parent | b265380ebedb1603933251a6e8fd0e7c850eba5a (diff) | |
Merge remote-tracking branch 'chiiph/bug/properly_check_permissions' into develop
Diffstat (limited to 'src/leap')
| -rw-r--r-- | src/leap/platform_init/initializers.py | 2 | ||||
| -rw-r--r-- | src/leap/services/eip/vpnlaunchers.py | 3 |
2 files changed, 3 insertions, 2 deletions
diff --git a/src/leap/platform_init/initializers.py b/src/leap/platform_init/initializers.py index 3374e32e..5345f11a 100644 --- a/src/leap/platform_init/initializers.py +++ b/src/leap/platform_init/initializers.py @@ -195,7 +195,7 @@ def WindowsInitializer(): dev_installer = os.path.join(driver_path, "devcon.exe") if os.path.isfile(dev_installer) and \ - os.access(dev_installer, os.X_OK): + stat.S_IXUSR & os.stat(dev_installer)[stat.ST_MODE] != 0: inf_path = os.path.join(driver_path, "OemWin2k.inf") cmd = [dev_installer, "install", inf_path, "tap0901"] diff --git a/src/leap/services/eip/vpnlaunchers.py b/src/leap/services/eip/vpnlaunchers.py index 762b536d..c5b21eac 100644 --- a/src/leap/services/eip/vpnlaunchers.py +++ b/src/leap/services/eip/vpnlaunchers.py @@ -24,6 +24,7 @@ import getpass import os import platform import subprocess +import stat try: import grp except ImportError: @@ -167,7 +168,7 @@ def _has_updown_scripts(path, warn=True): "Might produce DNS leaks." % (path,)) # XXX check if applies in win - is_exe = os.access(path, os.X_OK) + is_exe = (stat.S_IXUSR & os.stat(path)[stat.ST_MODE] != 0) if warn and not is_exe: logger.error("Up/down script %s is not executable. " "Might produce DNS leaks." % (path,)) |