← Back to branch summary

~azzar1/unity/add-show-desktop-key

« back to all changes in this revision

Viewing changes to ivle/makeuser.py

  • Committer: Matt Giuca
  • Date: 2009-04-22 04:56:58 UTC
  • Revision ID: matt.giuca@gmail.com-20090422045658-nnfepg0902n3mwtq
ivle.makeuser: Fixed odd code which would create the home directory, then
    immediately clobber it by restoring the backup.
    This broke in Python 2.6 because the behaviour of shutil.move changed.
    (Commented).

Show diffs side-by-side

added added

removed removed

Lines of Context:
ivle/makeuser.py
213
213
        warnings.simplefilter('ignore')
214
214
        homebackup = os.tempnam(tempdir)
215
215
        warnings.resetwarnings()
216
 
        # Note: shutil.move does not behave like "mv" - it does not put a file
217
 
        # into a directory if it already exists, just fails. Therefore it is
218
 
        # not susceptible to tmpnam symlink attack.
 
216
        # Back up the /home directory, delete the entire jail, recreate the
 
217
        # jail directory tree, then copy the /home back
 
218
        # NOTE that shutil.move changed in Python 2.6, it now moves a
 
219
        # directory INTO the target (like `mv`), which it didn't use to do.
 
220
        # This code works regardless.
219
221
        shutil.move(homedir, homebackup)
220
222
        shutil.rmtree(userdir)
221
 
        os.makedirs(homedir)
 
223
        os.makedirs(userdir)
222
224
        shutil.move(homebackup, homedir)
223
225
        # Change the ownership of all the files to the right unixid
224
226
        logging.debug("chown %s's home directory files to uid %d"