[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
Re: pkg/45044: archivers/libarchive minix support
The following reply was made to PR pkg/45044; it has been noted by GNATS.
From: Joerg Sonnenberger <joerg%britannica.bec.de@localhost>
Cc: pkg-manager%netbsd.org@localhost, gnats-admin%netbsd.org@localhost,
Subject: Re: pkg/45044: archivers/libarchive minix support
Date: Sat, 11 Jun 2011 20:02:28 +0200
On Fri, Jun 10, 2011 at 03:20:01PM +0000, tcort%minix3.org@localhost 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.
Main Index |
Thread Index |