Issue #5113: Fix a test_posix failure on HP-UX, where non-root users can
chown() to root under certain circumstances.
This commit is contained in:
commit
fba807ac44
|
@ -10,6 +10,7 @@ import sys
|
||||||
import time
|
import time
|
||||||
import os
|
import os
|
||||||
import fcntl
|
import fcntl
|
||||||
|
import platform
|
||||||
import pwd
|
import pwd
|
||||||
import shutil
|
import shutil
|
||||||
import stat
|
import stat
|
||||||
|
@ -390,6 +391,9 @@ class PosixTester(unittest.TestCase):
|
||||||
|
|
||||||
def _test_all_chown_common(self, chown_func, first_param):
|
def _test_all_chown_common(self, chown_func, first_param):
|
||||||
"""Common code for chown, fchown and lchown tests."""
|
"""Common code for chown, fchown and lchown tests."""
|
||||||
|
# test a successful chown call
|
||||||
|
chown_func(first_param, os.getuid(), os.getgid())
|
||||||
|
|
||||||
if os.getuid() == 0:
|
if os.getuid() == 0:
|
||||||
try:
|
try:
|
||||||
# Many linux distros have a nfsnobody user as MAX_UID-2
|
# Many linux distros have a nfsnobody user as MAX_UID-2
|
||||||
|
@ -401,12 +405,15 @@ class PosixTester(unittest.TestCase):
|
||||||
chown_func(first_param, ent.pw_uid, ent.pw_gid)
|
chown_func(first_param, ent.pw_uid, ent.pw_gid)
|
||||||
except KeyError:
|
except KeyError:
|
||||||
pass
|
pass
|
||||||
|
elif platform.system() in ('HP-UX', 'SunOS'):
|
||||||
|
# HP-UX and Solaris can allow a non-root user to chown() to root
|
||||||
|
# (issue #5113)
|
||||||
|
raise unittest.SkipTest("Skipping because of non-standard chown() "
|
||||||
|
"behavior")
|
||||||
else:
|
else:
|
||||||
# non-root cannot chown to root, raises OSError
|
# non-root cannot chown to root, raises OSError
|
||||||
self.assertRaises(OSError, chown_func,
|
self.assertRaises(OSError, chown_func,
|
||||||
first_param, 0, 0)
|
first_param, 0, 0)
|
||||||
# test a successful chown call
|
|
||||||
chown_func(first_param, os.getuid(), os.getgid())
|
|
||||||
|
|
||||||
@unittest.skipUnless(hasattr(posix, 'chown'), "test needs os.chown()")
|
@unittest.skipUnless(hasattr(posix, 'chown'), "test needs os.chown()")
|
||||||
def test_chown(self):
|
def test_chown(self):
|
||||||
|
|
Loading…
Reference in New Issue