[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#728057: adt-virt-schroot: set incorrect TMPDIR permissions



tag 728057 pending
thanks

Hello Arthur,

Arthur de Jong [2013-10-27 23:56 +0100]:
> Setting a more liberal umask before running atd-run fixes the issue. The
> attached patch also fixes the issue but I haven't checked whether other
> directories are created with the correct permissions.

I can reproduce the failure by setting umask 077 in the
NullRunner.test_tmpdir_for_other_users test.

> -        script += 'rm -rf -- "$@"; mkdir -- "$@"\n'
> +        script += 'rm -rf -- "$@"; mkdir -- "$@"; chmod 755 "$@"\n'

I used mkdir -m 755, which is more efficient and does the same.

Thanks! Fixed in
http://anonscm.debian.org/gitweb/?p=autopkgtest/autopkgtest.git;a=commitdiff;h=b79ba2d4da8

Martin

-- 
Martin Pitt                        | http://www.piware.de
Ubuntu Developer (www.ubuntu.com)  | Debian Developer  (www.debian.org)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.alioth.debian.org/pipermail/autopkgtest-devel/attachments/20131028/38c2b9ef/attachment-0001.sig>


Reply to: