[buildd-tools-devel] Bug#586195: Bug#586195: 20nssdatabases checks for file equivalence

[Bastian Blank]

On Sat, Jun 26, 2010 at 11:36:12PM +0100, Roger Leigh wrote:
> On Thu, Jun 17, 2010 at 11:49:47AM +0200, Bastian Blank wrote:
> > 20nssdatabases checks for file equivalence and don't does anything in
> > thie case. However nss may include more modules then just "files" and
> > will fail to produce a usefull result in this case.
> However, we are checking the file device number and inode number, not
> the file contents.  These should never be the same both inside and
> outside the chroot.  If they are, something is very badly wrong:

The problem is a completely different one: the result of getent passwd
and the contents of /etc/passwd are not equivalent. So in case of a
hardlinked file the result is a completely different (just it) then if
the script creates a new one (the contents all nss databases).

Okay, to be exact: getent passwd may not provide a complete view anyway
(because of query limits or so in case of remote databases, like ldap).

> For example, 20nssdatabases does the equivalent of
>   getent passwd > $chroot/etc/passwd

It have to replace the old file in this case anyway and not truncate it.

Bastian

-- 
Earth -- mother of the most beautiful women in the universe.
		-- Apollo, "Who Mourns for Adonais?" stardate 3468.1