This is the mail archive of the
cygwin
mailing list for the Cygwin project.
Re: Core dump on 32-bit Cygwin if program calls dlopen
- From: Kai Tietz <ktietz70 at googlemail dot com>
- To: cygwin at cygwin dot com, DJ Delorie <dj at redhat dot com>, Kai Tietz <ktietz at redhat dot com>
- Date: Thu, 24 Jul 2014 15:08:33 +0200
- Subject: Re: Core dump on 32-bit Cygwin if program calls dlopen
- Authentication-results: sourceware.org; auth=none
- References: <20140715130829 dot GK10401 at calimero dot vinschen dot de> <53C532D9 dot 7050506 at gmail dot com> <20140715143915 dot GA8330 at calimero dot vinschen dot de> <20140716070231 dot GA25020 at calimero dot vinschen dot de> <53C6F68A dot 9000801 at gmail dot com> <20140717073734 dot GA15332 at calimero dot vinschen dot de> <53C7EC48 dot 5060903 at dronecode dot org dot uk> <20140717182429 dot GS15332 at calimero dot vinschen dot de> <20140722082711 dot GA22860 at calimero dot vinschen dot de> <53CE604D dot 1070309 at dronecode dot org dot uk> <20140722132001 dot GD27005 at calimero dot vinschen dot de>
2014-07-22 15:20 GMT+02:00 Corinna Vinschen wrote:
> On Jul 22 13:59, Jon TURNEY wrote:
>> On 22/07/2014 09:27, Corinna Vinschen wrote:
>> >On Jul 17 20:24, Corinna Vinschen wrote:
>> >>On Jul 17 16:31, Jon TURNEY wrote:
>> >>>On 17/07/2014 08:37, Corinna Vinschen wrote:
>> >>>>It's the libgcc DLL which gives us grief, so a new libgcc package is
>> >>>>sufficent, afaics. We should check if this DLL fixes the problem and
>> >>>>then make it "curr" soon, I think.
>> >>>
>> >>>I briefly tested this other patch with my test case from the mail above, and
>> >>>it doesn't seem to help.
>> >>>[...]
>> >>I asked DJ to take another look, but I guess ultimately we need the
>> >>attention of one of the GCC Windows maintainers. Kai Tietz seems to be
>> >>unavailable right now, unfortunately.
>> >
>> >Looks like I totally misunderstood DJ's patch. The patch does *not*
>> >change libgcc, it changes cygmin-crtbegin.c, thus the crtbegin.o file
>> >which is statically linked into the executable.
>> >
>> >That means, to fix the issue, you don't have to replace libgcc, you
>> >have to recompile the executable against the new crtbegin.o.
>> >
>> >DJ still claims his patch is the correct one. The simple testcase
>> >dlopen'ing cyggs-9.dll works fine with the new crtbegin.o, according to
>> >him.
>>
>> Sorry, I hadn't tested it correctly.
>
> No worries, this was my fault. Talking about building a new libgcc
> rather than building a new crtbegin.o was bound to lead everyone
> off track :|
>
>> Building my test with an updated crtbegin.o as well, my test case is fixed.
>
> Nice!
>
>> I agree this patch seems better than my suggested one, as it makes crtbegin
>> do the right thing in the face of unbalanced libgcc load/unload, rather than
>> attempting to balance the libgcc load/unload as mine does.
>
> Ok. I CC'ed DJ so he knows all is good, and then we probably need a
> windows GCC maintainer to approve the patch I guess. Kai? Ping?
>
Hi Corinna
As I we spoke yesterday on irc, I took already a look to DJ's patch
and it is sensible. If DJ sends patch upstream to gcc's ML, and put
me CC, I will review it asap.
Thanks,
Kai
--
Problem reports: http://cygwin.com/problems.html
FAQ: http://cygwin.com/faq/
Documentation: http://cygwin.com/docs.html
Unsubscribe info: http://cygwin.com/ml/#unsubscribe-simple