summaryrefslogtreecommitdiff
path: root/src/leap/bitmask/vpn/helpers
diff options
context:
space:
mode:
authorRuben Pollan <meskio@sindominio.net>2018-01-26 12:46:29 +0100
committerKali Kaneko <kali@leap.se>2018-01-26 22:55:16 +0100
commite6d91023e7ad044d09e59cc076aef9c9f583592c (patch)
tree863ba3ef9e833374e43b3c7ab3bd0ece24413fe8 /src/leap/bitmask/vpn/helpers
parent9b73b8baf48a1b9ef388496f05e472963d5ad459 (diff)
[refactor] yet another readability pass
Diffstat (limited to 'src/leap/bitmask/vpn/helpers')
-rw-r--r--src/leap/bitmask/vpn/helpers/__init__.py20
1 files changed, 14 insertions, 6 deletions
diff --git a/src/leap/bitmask/vpn/helpers/__init__.py b/src/leap/bitmask/vpn/helpers/__init__.py
index 631ad785..4052a8d1 100644
--- a/src/leap/bitmask/vpn/helpers/__init__.py
+++ b/src/leap/bitmask/vpn/helpers/__init__.py
@@ -56,19 +56,19 @@ if IS_LINUX:
return (
is_pkexec_in_system() and
_check_helper() and
- _check_polkit() and
+ _check_polkit_file_exist() and
_check_openvpn())
def _check_helper():
helper_path = _config.get_bitmask_helper_path()
- if not access(helper_path, R_OK):
+ if not _exists_and_can_read(helper_path):
return True
helper_path_digest = digest(helper_path)
- if (access(BITMASK_ROOT_SYSTEM, R_OK) and
+ if (_exists_and_can_read(BITMASK_ROOT_SYSTEM) and
helper_path_digest == digest(BITMASK_ROOT_SYSTEM)):
return True
- if (access(BITMASK_ROOT_LOCAL, R_OK) and
+ if (_exists_and_can_read(BITMASK_ROOT_LOCAL) and
helper_path_digest == digest(BITMASK_ROOT_LOCAL)):
return True
@@ -80,16 +80,21 @@ if IS_LINUX:
openvpn_path = _config.get_bitmask_openvpn_path()
if openvpn_path is None:
+ # If there bitmask doesn't provide any openvpn binary (we are not
+ # in a bundle), reporting an error on check will trigger an attempt
+ # to install helpers that can not succeed.
+ # XXX: we need a better way to flag errors that can not be solved
+ # by installing helpers
return True
openvpn_path_digest = digest(openvpn_path)
- if (access(OPENVPN_LOCAL, R_OK) and
+ if (_exists_and_can_read(OPENVPN_LOCAL) and
openvpn_path_digest == digest(OPENVPN_LOCAL)):
return True
return False
- def _check_polkit():
+ def _check_polkit_file_exist():
# XXX: we are just checking if there is any policy file installed not
# if it's valid or if it's the correct one that will be used.
# (if LOCAL is used if /usr/local/sbin/bitmask-root is used and SYSTEM
@@ -97,6 +102,9 @@ if IS_LINUX:
return (os.path.exists(POLKIT_LOCAL) or
os.path.exists(POLKIT_SYSTEM))
+ def _exists_and_can_read(file_path):
+ return access(file_path, R_OK)
+
elif IS_MAC: