This is the mail archive of the 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: Patch for Setup.exe problem and for

> -----Original Message-----
> From: Ton van Overbeek [] 
> Sent: Thursday, March 28, 2002 1:03 AM
> To:;
> Cc:
> Subject: Patch for Setup.exe problem and for
> Found the problem causing the segment violation and probably 
> causing Jonas Eriksson's problem. It is a typical case of 
> 'off by 1'. In PickView::set_headers the loop filling the 
> window header does one iteration too much, resulting in a 
> call to DoInsertItem with a NULL string pointer and hence a 
> crash following. While debugging this I could not compile the 
> new ( the
> c++ version of the original mklink2.c). It seems three & (address of 
> c++ operator)
> have disappeared in the transition. Putting them back made 
> the compiler happy. Is this OK Robert ?

I'll check the off-by-one fix in tomorrow, as I'm off to bed now. 

As for the &'s, I wonder if it's a w32api reference issue? The compiler
complains if they are present for me.

I have the latest-and-greatest w32api headers on my system - what do you

Thanks for finding the off-by-one... blush.


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