pkgsrc-Bugs archive

[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>
To: gnats-bugs%NetBSD.org@localhost
Cc: pkg-manager%netbsd.org@localhost, gnats-admin%netbsd.org@localhost, 
pkgsrc-bugs%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.
 
 Joerg
 


Home | Main Index | Thread Index | Old Index