cvs commit: src/usr.bin/make arch.c

Harti Brandt hartmut.brandt at dlr.de
Tue Mar 4 16:15:49 UTC 2008


On Tue, 4 Mar 2008, Warner Losh wrote:

WL>imp         2008-03-04 15:56:18 UTC
WL>
WL>  FreeBSD src repository
WL>
WL>  Modified files:
WL>    usr.bin/make         arch.c 
WL>  Log:
WL>  <limits.h> is necessary for using INT_MIN, so included it here
WL>  explicitly rather than relying on name space pollution to pull it in
WL>  for us.
WL>  
WL>  NB: The usage of INT_MIN is somewhat bogus and suspect to my eye, but this
WL>  commit doesn't address that issue.

Yes. This looks like a bug. ArchStatMember returns INT64_MIN if it cannot 
stat the archive member. So the caller must check for INT64_MIN.

harti


More information about the cvs-all mailing list