Subject: Re: PR/36464 CVS commit: src/lib/libc/gen
To: None <lib-bug-people@netbsd.org, gnats-admin@netbsd.org,>
From: Ed Ravin <eravin@panix.com>
List: netbsd-bugs
Date: 06/10/2007 01:05:09
The following reply was made to PR lib/36464; it has been noted by GNATS.

From: Ed Ravin <eravin@panix.com>
To: gnats-bugs@NetBSD.org
Cc: lib-bug-people@netbsd.org, gnats-admin@netbsd.org,
	netbsd-bugs@netbsd.org, eravin@panix.com
Subject: Re: PR/36464 CVS commit: src/lib/libc/gen
Date: Sat, 9 Jun 2007 21:00:20 -0400

 Christos - thanks for the quick fix, I will test it out later this
 week.
 
 The man page for stat(2) also needs updating, in particular this part:
 
 >     The size-related fields of the struct stat are as follows:
 >
 >     st_size        The size of the file in bytes.  A directory will be a mul-
 >                    tiple of the size of the dirent(5) structure.
 
 Should be changed to say that the meaning of st_size for a directory is
 filesystem-dependent (the ZFS behavior of "number of entries in the
 directory" is intentional).
 
 
 On Sun, Jun 10, 2007 at 12:00:06AM +0000, Christos Zoulas wrote:
 ...
 > 
 >  Modified Files:
 >  	src/lib/libc/gen: scandir.c
 >  
 >  Log Message:
 >  PR/36464: scandir(3) corrupts heap when run on ZFS directories because ZFS
 >  returns the number of entries as the directory size.
 >  Use a new, more conservative entries estimator.