Message134074
In TarFile.chown, if the lookup
u = pwd.getpwnam(tarinfo.uname)[2]
fails, this line is used:
u = pwd.getpwuid(tarinfo.uid)[2]
This will fail if the uid isn't in /etc/passwd. I think "u = tarinfo.uid" would make more sense. This fallback could also be used if the pwd module isn't present or tarinfo.uname isn't filled. Here's a code sample:
u = tarinfo.uid
if tarinfo.uname and pwd:
try: u = pwd.getpwnam(tarinfo.uname)[2]
except KeyError: pass
The same issue applies to group lookup. |
|
Date |
User |
Action |
Args |
2011-04-19 16:03:35 | mgold-qnx | set | recipients:
+ mgold-qnx |
2011-04-19 16:03:35 | mgold-qnx | set | messageid: <1303229015.7.0.433519463198.issue11879@psf.upfronthosting.co.za> |
2011-04-19 16:03:35 | mgold-qnx | link | issue11879 messages |
2011-04-19 16:03:34 | mgold-qnx | create | |
|