This is the mail archive of the cygwin@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Segmentation Fault of ssh3.7.1 with cygwin1.5.5-1 in W95


At 08:06 PM 11/3/2003 +0100, Corinna Vinschen wrote:
>On Mon, Nov 03, 2003 at 01:39:56PM -0500, Pierre A. Humblet wrote:
>> On Mon, Nov 03, 2003 at 05:31:15PM +0100, Corinna Vinschen wrote:
>> >
>> > I see.  I'll create a patch.
>>  
>> I started one already, must still compile and test.
>> Let me know if you proceed on your side.
>
>I have a patch ready and it seems to work fine(tm).  Would you mind
>to give it a try on 95?  Patch below.

Turns out your patch is identical to mine (there is no need to 
initialize protolen), and it worked on Win95, but a few things 
trouble me in the reused code:

1) The test "IsBadReadPtr (src->s_proto ..." will unduly fail on Win95
   if the 16 lsb of the first alias address are the 16 msb of a readable
   memory address. I don't see why that's impossible. If would safer 
   to have a wincap entry, or (horror !), testing wincap.osname ().
2) The test "&& !IsBadReadPtr (((pservent *) src) " should never fail
   when it is reached, but should it fail, s_proto would be assigned an
   invalid address and the program would crash. So if that test is kept
   it should be part of a if .. else if ..  (leaving s_proto NULL).
3) Strictly speaking, we should use IsBadStringPtr..

Pierre


--
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
Problem reports:       http://cygwin.com/problems.html
Documentation:         http://cygwin.com/docs.html
FAQ:                   http://cygwin.com/faq/


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]