|
msg80835 - (view) |
Author: Andreas Kupries (andreask) |
Date: 2009-01-30 20:12 |
The 'test_posix' of Python's 2.5.4.3 testsuite fails on HPUX. This has
happened on parisc and ia64 systems. On the parisc system failure occurs
for both parisc1.1 and parisc2.0w builds. The HPUX versions are 11.00
(parisc) and 11.22 (ia64).
The attached file contains the output of executing
./apy/bin/python2.5 ./apy/lib/python2.5/test/regrtest.py -v test_posix
|
|
msg80913 - (view) |
Author: Antoine Pitrou (pitrou) *  |
Date: 2009-02-01 18:09 |
If you try "chown root foobar", with <foobar> being a valid file you
own, does it succeed?
|
|
msg80973 - (view) |
Author: Andreas Kupries (andreask) |
Date: 2009-02-02 18:19 |
Yes.
Session log:
andreask@bertha:~/tmp/x> ll
total 0
andreask@bertha:~/tmp/x> touch foobar
andreask@bertha:~/tmp/x> ll
total 0
-rw-rw-r-- 1 andreask DomainUsers 0 Feb 2 10:17 foobar
andreask@bertha:~/tmp/x> chown root foobar
andreask@bertha:~/tmp/x> ll
total 0
-rw-rw-r-- 1 root DomainUsers 0 Feb 2 10:17 foobar
|
|
msg80976 - (view) |
Author: Antoine Pitrou (pitrou) *  |
Date: 2009-02-02 18:25 |
That explains what the test fails then. The test expects that you can't
chown a file to root as a normal user, and it verifies that an exception
is raised in that case.
Do you know if this behaviour is standard under HP-UX? If so, what do
sys.platform and platform.system() return?
(this won't be fixed in 2.5 by the way: 2.5 only receives security
fixes. 2.6 is the current stable branch and 2.7 is the in-development
version)
|
|
msg81012 - (view) |
Author: Andreas Kupries (andreask) |
Date: 2009-02-02 22:42 |
I do not know if this is standard for hpux or not.
I can say that right now we have six different HPUX machines which
behave in this manner, i.e. do not throw an error for the 'chown root
foobar' command for an ordinary (non-root) user.
Regarding the platform codes, here are the results.
andreask@bertha:~/tmp/autotest> ./apy/bin/python2.5
ActivePython 2.5.4.3 (ActiveState Software Inc.) based on
Python 2.5.4 (r254:67916, Jan 20 2009, 14:48:58) [C] on hp-ux11
Type "help", "copyright", "credits" or "license" for more information.
>>> import sys, platform
>>> sys.platform
'hp-ux11'
>>> platform.system()
'HP-UX'
>>> quit()
The six machines I mentioned are
bertha - HP-UX 11.00 (PA-RISC)
bigsur03 - HP-UX 11.22 (Itanium)
hpa180 - HP-UX 11.00 (PA-RISC)
bigsur03 - HP-UX 11.22 (Itanium)
hpl3000 - HP-UX 11.23 (PA-RISC)
bigsur05 - HP-UX 11.32 (Itanium)
|
|
msg81014 - (view) |
Author: Antoine Pitrou (pitrou) *  |
Date: 2009-02-02 23:00 |
Ok, and when you try to chown back to yourself the root-owned foobar,
does it fail?
|
|
msg81015 - (view) |
Author: Andreas Kupries (andreask) |
Date: 2009-02-02 23:07 |
Yes, that does fail with error 'not owner'.
andreask@bertha:~/tmp> mkdir x
andreask@bertha:~/tmp> cd x
andreask@bertha:~/tmp/x> touch foobar
andreask@bertha:~/tmp/x> ll
total 0
-rw-rw-r-- 1 andreask DomainUsers 0 Feb 2 15:06 foobar
andreask@bertha:~/tmp/x> chown root foobar
andreask@bertha:~/tmp/x> ll
total 0
-rw-rw-r-- 1 root DomainUsers 0 Feb 2 15:06 foobar
andreask@bertha:~/tmp/x> chown andreask foobar
foobar: Not owner
andreask@bertha:~/tmp/x>
|
|
msg116648 - (view) |
Author: Charles-François Natali (neologix) *  |
Date: 2010-09-17 08:52 |
Actually, looking at the man page, this seems to be normal:
---
In order to change the owner or group, you must own the file and have the CHOWN privilege (see setprivgrp(1M)). [...] Note that a given user's or group's ability to use this command can be restricted by setprivgrp (see setprivgrp(1M)).
---
So depending on the setting of setprivgrp, chown'ing a file to root may be allowed or not.
It might be a good idea to skip this test on HP-UX. Solaris apparently behaves in the same way.
|
|
msg141267 - (view) |
Author: Charles-François Natali (neologix) *  |
Date: 2011-07-27 19:15 |
Patch attached.
|
|
msg142938 - (view) |
Author: STINNER Victor (vstinner) *  |
Date: 2011-08-24 22:49 |
It's surprising to raise a SkipTest *at the end* of the test. Here is another patch.
|
|
msg142940 - (view) |
Author: STINNER Victor (vstinner) *  |
Date: 2011-08-24 22:53 |
Using chown_hpux-4.diff, we don't skip anything on HP-UX.
|
|
msg142944 - (view) |
Author: Charles-François Natali (neologix) *  |
Date: 2011-08-24 22:59 |
> Using chown_hpux-4.diff, we don't skip anything on HP-UX.
Victor, I don't understand your patch: if we try chown()ing a file to root, then this will fail on HP-UX (because chown won't fail with EPERM). That's the whole point of the patch.
|
|
msg142945 - (view) |
Author: STINNER Victor (vstinner) *  |
Date: 2011-08-24 23:02 |
> if we try chown()ing a file to root, then this will fail on HP-UX
Oooh, I'm confused. I removed this stupid patch.
|
|
msg142946 - (view) |
Author: STINNER Victor (vstinner) *  |
Date: 2011-08-24 23:04 |
# HP-UX and Solaris allow a non-root user to chown to root
I didn't see failure on our Solaris / OpenIndiana buildbot. Should we really skip the test on Solaris?
|
|
msg142948 - (view) |
Author: Charles-François Natali (neologix) *  |
Date: 2011-08-24 23:15 |
Well, IIRC, I read in the Solaris man pages that it behaves like HP-UX
in this regard. But this it can be changed with `setprivgrp`, so it
might work on some machines, and fail on others.
According to http://unix.derkeiler.com/Newsgroups/comp.unix.solaris/2004-03/2579.html,
the test should work with the default setting (maybe the OP was
testing on a HP-UX machine with the non-default setting).
Anyway, since it depends on the system configuration, I wanted to stay
on the safe side, and just skip on both OS.
But I honestly don't have a strong opinion about this.
|
|
msg142985 - (view) |
Author: Charles-François Natali (neologix) *  |
Date: 2011-08-25 18:34 |
> Oooh, I'm confused. I removed this stupid patch.
chown_hpux-3.diff has a problem too: it skips the successful chown call (that's why I put the SkipTest at the end).
|
|
msg158152 - (view) |
Author: Charles-François Natali (neologix) *  |
Date: 2012-04-12 16:45 |
Adi, since you have access to an HP-UX box, could you test the attached patch (chown_hpux.diff)?
Also, if you're interested, you could search for other isues HP-UX-specific to see if you can help.
|
|
msg158155 - (view) |
Author: Adi Roiban (adiroiban) |
Date: 2012-04-12 17:43 |
Hi,
Not sure what codebase was used for the patch.
I have manually patched the test on 2.5.6 and the test_posix tests passed.
Thanks!
Adi
|
|
msg158179 - (view) |
Author: Adi Roiban (adiroiban) |
Date: 2012-04-12 22:04 |
I am not sure how the search works, but for example I was not able to reach http://bugs.python.org/issue5895 while searching for hpux or hp-ux using the internal search.
Also I was not able to find any aix or solaris bug.
In case you bump over an hpux, aix, solaris bug, please add me to the noisy list and I will take a look.
Thanks!
Adi
|
|
msg158181 - (view) |
Author: R. David Murray (r.david.murray) *  |
Date: 2012-04-12 22:31 |
Search is currently not returning all matching issues, unfortunately. You might get a few more hits by searching for hpux in the title field via advanced search.
|
|
msg158562 - (view) |
Author: Roundup Robot (python-dev)  |
Date: 2012-04-17 17:59 |
New changeset 8a4c9a168d09 by Charles-François Natali in branch '2.7':
Issue #5113: Fix a test_posix failure on HP-UX, where non-root users can
http://hg.python.org/cpython/rev/8a4c9a168d09
New changeset 428bece48029 by Charles-François Natali in branch '3.2':
Issue #5113: Fix a test_posix failure on HP-UX, where non-root users can
http://hg.python.org/cpython/rev/428bece48029
New changeset 3d4922cf0d65 by Charles-François Natali in branch 'default':
Issue #5113: Fix a test_posix failure on HP-UX, where non-root users can
http://hg.python.org/cpython/rev/3d4922cf0d65
|
|
msg158563 - (view) |
Author: Charles-François Natali (neologix) *  |
Date: 2012-04-17 18:03 |
Committed, thanks.
|
|
| Date |
User |
Action |
Args |
| 2022-04-11 14:56:45 | admin | set | github: 49363 |
| 2012-04-17 18:03:15 | neologix | set | status: open -> closed versions:
- Python 2.6 messages:
+ msg158563
resolution: fixed stage: patch review -> resolved |
| 2012-04-17 17:59:30 | python-dev | set | nosy:
+ python-dev messages:
+ msg158562
|
| 2012-04-12 22:31:46 | r.david.murray | set | nosy:
+ r.david.murray messages:
+ msg158181
|
| 2012-04-12 22:04:48 | adiroiban | set | messages:
+ msg158179 |
| 2012-04-12 17:43:45 | adiroiban | set | messages:
+ msg158155 |
| 2012-04-12 16:45:28 | neologix | set | messages:
+ msg158152 |
| 2012-04-12 02:08:15 | adiroiban | set | nosy:
+ adiroiban
|
| 2011-08-25 18:34:45 | neologix | set | messages:
+ msg142985 |
| 2011-08-24 23:15:27 | neologix | set | messages:
+ msg142948 |
| 2011-08-24 23:04:10 | vstinner | set | messages:
+ msg142946 |
| 2011-08-24 23:02:25 | vstinner | set | messages:
+ msg142945 |
| 2011-08-24 23:01:13 | vstinner | set | files:
- chown_hpux-4.diff |
| 2011-08-24 22:59:39 | neologix | set | messages:
+ msg142944 |
| 2011-08-24 22:53:19 | vstinner | set | messages:
+ msg142940 |
| 2011-08-24 22:52:18 | vstinner | set | files:
+ chown_hpux-4.diff |
| 2011-08-24 22:50:31 | vstinner | set | files:
- chown_hpux-2.diff |
| 2011-08-24 22:50:19 | vstinner | set | files:
+ chown_hpux-3.diff |
| 2011-08-24 22:49:01 | vstinner | set | files:
+ chown_hpux-2.diff
messages:
+ msg142938 |
| 2011-08-24 19:47:24 | neologix | set | nosy:
+ vstinner
|
| 2011-07-27 19:15:20 | neologix | set | files:
+ chown_hpux.diff versions:
+ Python 3.2, Python 3.3 messages:
+ msg141267
keywords:
+ patch, needs review stage: patch review |
| 2010-09-17 08:52:21 | neologix | set | nosy:
+ neologix messages:
+ msg116648
|
| 2009-02-02 23:07:46 | andreask | set | messages:
+ msg81015 |
| 2009-02-02 23:00:51 | pitrou | set | messages:
+ msg81014 |
| 2009-02-02 22:42:01 | andreask | set | messages:
+ msg81012 |
| 2009-02-02 18:25:26 | pitrou | set | messages:
+ msg80976 versions:
+ Python 2.6, Python 2.7, - Python 2.5 |
| 2009-02-02 18:19:47 | andreask | set | messages:
+ msg80973 |
| 2009-02-01 18:09:28 | pitrou | set | nosy:
+ pitrou messages:
+ msg80913 |
| 2009-01-30 20:12:12 | andreask | create | |