On Sat, Nov 01, 2014 at 08:57:12PM +0000, Dimitris Papastamos wrote: > On Sat, Nov 01, 2014 at 08:36:37PM +0000, Michael Forney wrote: > > - snprintf(h->uname, sizeof h->uname, "%s", pw->pw_name); > > - snprintf(h->gname, sizeof h->gname, "%s", gr->gr_name); > > + snprintf(h->uname, sizeof h->uname, "%s", pw ? pw->pw_name : ""); > > + snprintf(h->gname, sizeof h->gname, "%s", gr ? gr->gr_name : ""); > > The patches look good, thanks! > > Just a small clarification on this one, do other tar implementations > do the same here?
It looks like GNU tar does¹, but BSD tar uses the string representation of the UID/GID. ¹ http://git.savannah.gnu.org/cgit/tar.git/tree/src/names.c#n66 -- Michael Forney <mfor...@mforney.org>
pgpemX9fk9wLs.pgp
Description: PGP signature