Re: FYI: getdelim.c needs SIZE_MAX

2005-10-06 Thread Simon Josefsson
Derek Price <[EMAIL PROTECTED]> writes: > Simon Josefsson wrote: > >>Derek Price <[EMAIL PROTECTED]> writes: >> >> >>>may not be defined. I've installed the attached patch, ripped verbatim >>>from serveral other modules. >>> >>> >> >>Shouldn't we use the size_max module instead? >> >> > >

Re: FYI: getdelim.c needs SIZE_MAX

2005-10-05 Thread Paul Eggert
Derek Price <[EMAIL PROTECTED]> writes: > I think the size_max module is only important if you want to use > SIZE_MAX as part of a preprocessor conditional Yes, that's my recollection as well. ___ bug-gnulib mailing list bug-gnulib@gnu.org http://list

Re: FYI: getdelim.c needs SIZE_MAX

2005-10-05 Thread Derek Price
Simon Josefsson wrote: >Derek Price <[EMAIL PROTECTED]> writes: > > >>may not be defined. I've installed the attached patch, ripped verbatim >>from serveral other modules. >> >> > >Shouldn't we use the size_max module instead? > > I'm not sure. Some 18 modules currently don't, defining

Re: FYI: getdelim.c needs SIZE_MAX

2005-10-05 Thread Simon Josefsson
Derek Price <[EMAIL PROTECTED]> writes: > Paul's recent change to getdelim.c breaks the build on at least NetBSD > 1.6.1: > > 2005-10-03 Paul Eggert <[EMAIL PROTECTED]> > > * getdelim.c: Include getdelim.h first. Include . > (SSIZE_MAX): New macro, if not already defined. >

FYI: getdelim.c needs SIZE_MAX

2005-10-05 Thread Derek Price
Paul's recent change to getdelim.c breaks the build on at least NetBSD 1.6.1: 2005-10-03 Paul Eggert <[EMAIL PROTECTED]> * getdelim.c: Include getdelim.h first. Include . (SSIZE_MAX): New macro, if not already defined. (getdelim): Fix buffer overrun on 64-bit hosts with