diff --git a/files/copy b/files/copy index 862a1ac77b4..47012155f25 100644 --- a/files/copy +++ b/files/copy @@ -204,6 +204,8 @@ def main(): os.unlink(dest) open(dest, 'w').close() if validate: + if "%s" not in validate: + module.fail_json(msg="validate must contain %%s: %s" % (validate)) (rc,out,err) = module.run_command(validate % src) if rc != 0: module.fail_json(msg="failed to validate: rc:%s error:%s" % (rc,err)) diff --git a/files/lineinfile b/files/lineinfile index 780764ad7b8..ef629382f27 100644 --- a/files/lineinfile +++ b/files/lineinfile @@ -154,6 +154,8 @@ def write_changes(module,lines,dest): validate = module.params.get('validate', None) valid = not validate if validate: + if "%s" not in validate: + module.fail_json(msg="validate must contain %%s: %s" % (validate)) (rc, out, err) = module.run_command(validate % tmpfile) valid = rc == 0 if rc != 0: diff --git a/files/replace b/files/replace index 536ac8cd8fc..57b522dd773 100644 --- a/files/replace +++ b/files/replace @@ -90,6 +90,8 @@ def write_changes(module,contents,dest): validate = module.params.get('validate', None) valid = not validate if validate: + if "%s" not in validate: + module.fail_json(msg="validate must contain %%s: %s" % (validate)) (rc, out, err) = module.run_command(validate % tmpfile) valid = rc == 0 if rc != 0: