Mailing-List: contact cygwin-help@sourceware.cygnus.com; run by ezmlm
List-Subscribe: <mailto:cygwin-subscribe@sourceware.cygnus.com>
List-Archive: <http://sourceware.cygnus.com/ml/cygwin/>
List-Post: <mailto:cygwin@sourceware.cygnus.com>
List-Help: <mailto:cygwin-help@sourceware.cygnus.com>, <http://sourceware.cygnus.com/ml/#faqs>
Sender: cygwin-owner@sourceware.cygnus.com
Delivered-To: mailing list cygwin@sourceware.cygnus.com
From: Chris Faylor <cgf@cygnus.com>
Date: Wed, 15 Mar 2000 08:52:03 -0500
To: cygwin@sourceware.cygnus.com
Subject: Re: Permission denied with makeinfo from texinfo-4.0
Message-ID: <20000315085203.D19524@cygnus.com>
Reply-To: cygwin@sourceware.cygnus.com
Mail-Followup-To: cgf@cygnus.com, cygwin@sourceware.cygnus.com
References: <200003150556.AAA28729@mail.bcpl.net>; <20000315005835.A5528@cygnus.com> <200003150613.BAA01221@mail.bcpl.net>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
User-Agent: Mutt/1.1.8i
In-Reply-To: <200003150613.BAA01221@mail.bcpl.net>; from dbryan@bcpl.net on Wed, Mar 15, 2000 at 01:13:03AM -0500

On Wed, Mar 15, 2000 at 01:13:03AM -0500, J. David Bryan wrote:
>On 15 Mar 2000, at 0:58, Chris Faylor wrote:
>
>> Ok.  I've got it.  Sorry.  I obviously lost the thread pretty badly.
>> 
>> (slinks away)
>
>Must be a late night!  ;-)

They all are...

>As I noted, the proper solution is to fix the invalid buffer call, but one 
>can work around the problem either by defining WIN32 and accepting the 
>tradeoffs (slow reads, no $HOME support) or by undefining O_BINARY and 
>accepting the tradeoff (no \r\n support).

This sounds correct to me.  Passing what is essentially a bogus argument
for the buffer length is just wrong.  At the very least, it's subject to
problems if the file size changes while you're reading it.  Admittedly
this is an unusual occurrence but why not program for something like this
and fail gracefully rather than core dumping?

cgf

--
Want to unsubscribe from this list?
Send a message to cygwin-unsubscribe@sourceware.cygnus.com

