On 06/09/2011 11:49 AM, Matt Wagner wrote:
---
src/app/util/condormatic.rb | 3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/src/app/util/condormatic.rb b/src/app/util/condormatic.rb
index 4278d06..23379a4 100644
--- a/src/app/util/condormatic.rb
+++ b/src/app/util/condormatic.rb
@@ -91,6 +91,9 @@ def write_pw_file(job_name, pw)
FileUtils.mkdir_p(pwdir, options={:mode => 0700})
FileUtils.chown('aeolus', 'aeolus', pwdir)
+ # Restrict job names to relatively sane characters only
+ job_name.gsub!(/[^a-zA-Z0-9\.\-]/, '_')
+
pwfilename = File.join(pwdir, job_name)
tmpfile = Tempfile.new(job_name, pwdir)
ACK
Works for me
Scott