Remove the chmod+chown done by Host.send_file at the end of a copy.
It's generally unnecessary, and often wrong because it often tries to
chmod a lot more than the copied files.
Risk: Medium
Visibility: The dest given to Host.send_file will no longer be
automatically made world-readable-executable.
Signed-off-by: John Admanski <[email protected]>
--- autotest/server/hosts/abstract_ssh.py 2009-12-18 13:54:05.000000000
-0800
+++ autotest/server/hosts/abstract_ssh.py 2009-12-18 13:54:05.000000000
-0800
@@ -284,11 +284,6 @@
except error.CmdError, e:
raise error.AutoservRunError(e.args[0], e.args[1])
- self.run('find "%s" -type d -print0 | xargs -0r chmod o+rx' % dest)
- self.run('find "%s" -type f -print0 | xargs -0r chmod o+r' % dest)
- if self.target_file_owner:
- self.run('chown -R %s %s' % (self.target_file_owner, dest))
-
def ssh_ping(self, timeout=60):
try:
_______________________________________________
Autotest mailing list
[email protected]
http://test.kernel.org/cgi-bin/mailman/listinfo/autotest
