Use os module for checking absolute/joining paths

9 years ago · 030fbaef64
parent d4275e576e
commit 030fbaef64
1 changed files with 4 additions and 3 deletions
--- a/lib/ansible/modules/extras/system/cronvar.py
+++ b/lib/ansible/modules/extras/system/cronvar.py
@ -129,9 +129,10 @@ class CronVar(object):

        if cron_file:
            self.cron_file = ""
-            if cron_file[0] != '/':
-                self.cron_file = '/etc/cron.d/'
-            self.cron_file = self.cron_file + cron_file
+            if os.path.isabs(cron_file):
+                self.cron_file = cron_file
+            else:
+                self.cron_file = os.path.join('/etc/cron.d', cron_file)
        else:
            self.cron_file = None