pkgsrc-Bugs archive

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

Re: pkg/45044: archivers/libarchive minix support

On Fri, Jun 10, 2011 at 03:20:01PM +0000, wrote:
> Minix UIDs are 16-bits and Minix GIDs are 8-bits. The tar format
> supports much larger UIDs/GIDs. libarchive should handle the case where
> the UIDs/GIDs overflow the archive_entry uid/gid fields by setting
> the uid/gid to something valid (example: the uid of the nobody user and
> the gid of the nobody group) and it should produce a warning when doing so.

I don't think the complexity of this is justified with the gain.
Ultimately, you are either ending up ignoring the UID/GID or it won't
work at all. The behavior is already "set it to something valid", since
it truncates. Doing a user name look up in this piece of code is
completely inacceptable. In short, I feel this is a case of "use saner
types or just cope". You can, of course, bring this up on the libarchive
list or bug tracker, but I am going to repeat my answer in that case.


Home | Main Index | Thread Index | Old Index