Merge pull request #243 from MalloZup/15-files

Remove unnecessary chmod

hawk/app/lib/crm_script.rb

@@ -45,7 +45,6 @@ def run(jsondata, rootpw)
     tmpf = Tempfile.new 'crmscript'
     tmpf.write("script json \"#{cmd}\"")
     tmpf.close
-    File.chmod(0666, tmpf.path)
 
     if rootpw.nil?
       cmdline = ['/usr/sbin/hawk_invoke', user, 'crm', '-f', tmpf.path]

hawk/app/lib/invoker.rb

@@ -62,11 +62,6 @@ def crm_configure_load_update(cmd)
     begin
       f << cmd
       f.close
-      # Evil to allow unprivileged user running crm shell to read the file
-      # TODO(should): can we just allow group (probably ok live, but no
-      # good for testing when running as root), or some other alternative
-      # with piping data to crm?
-      File.chmod(0666, f.path)
       CrmEvents.instance.push "crm configure\n#{cmd}\n" unless @no_log
       result = crm '-F', 'configure', 'load', 'update', f.path
     ensure

hawk/app/models/report.rb

@@ -144,7 +144,6 @@ def graph(hb_report, path, format = :svg)
     require "tempfile"
     tmpfile = Tempfile.new("hawk_dot")
     tmpfile.close
-    File.chmod(0666, tmpfile.path)
     _out, err, status = Util.capture3('crm_simulate', '-x', tpath.to_s, format == :xml ? "-G" : "-D", tmpfile.path.to_s)
     rc = status.exitstatus