DomainKey-Signature: a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:date:from:to:cc:subject:message-id:reply-to
	:references:mime-version:content-type:in-reply-to; q=dns; s=
	default; b=KAvgNBpz8N8JbW3Z2Pr+vVrNbkz5yFHkuJicQwfCKCiojgv+25gmd
	CAE6f81+e2dwoGUx8XWlc8FEbuh61E/aG/JV6XbfW8aTzNN31/f9dAHsj6i+1FN7
	sndSJimw7GQlg68sIGUZ3KeNi4rbO3HSYL/YtiJ4XMqd8Oz48Ph5MM=
Mailing-List: contact cygwin-help@cygwin.com; run by ezmlm
Sender: cygwin-owner@cygwin.com
Mail-Followup-To: cygwin@cygwin.com
Date: Tue, 22 Jul 2014 15:20:01 +0200
From: Corinna Vinschen <corinna-cygwin@cygwin.com>
To: cygwin@cygwin.com
Cc: DJ Delorie <dj@redhat.com>, Kai Tietz <ktietz@redhat.com>
Subject: Re: Core dump on 32-bit Cygwin if program calls dlopen
Message-ID: <20140722132001.GD27005@calimero.vinschen.de>
Reply-To: cygwin@cygwin.com
Mail-Followup-To: cygwin@cygwin.com, DJ Delorie <dj@redhat.com>,	Kai Tietz <ktietz@redhat.com>
References: <20140715130829.GK10401@calimero.vinschen.de> <53C532D9.7050506@gmail.com> <20140715143915.GA8330@calimero.vinschen.de> <20140716070231.GA25020@calimero.vinschen.de> <53C6F68A.9000801@gmail.com> <20140717073734.GA15332@calimero.vinschen.de> <53C7EC48.5060903@dronecode.org.uk> <20140717182429.GS15332@calimero.vinschen.de> <20140722082711.GA22860@calimero.vinschen.de> <53CE604D.1070309@dronecode.org.uk>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha1;	protocol="application/pgp-signature"; boundary="p2kqVDKq5asng8Dg"
Content-Disposition: inline
In-Reply-To: <53CE604D.1070309@dronecode.org.uk>
User-Agent: Mutt/1.5.23 (2014-03-12)

--p2kqVDKq5asng8Dg
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

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 abov=
e, 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.
>=20
> 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 fixe=
d.

Nice!

> I agree this patch seems better than my suggested one, as it makes crtbeg=
in
> do the right thing in the face of unbalanced libgcc load/unload, rather t=
han
> 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?


Corinna

--=20
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Maintainer                 cygwin AT cygwin DOT com
Red Hat

--p2kqVDKq5asng8Dg
Content-Type: application/pgp-signature

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1

iQIcBAEBAgAGBQJTzmUBAAoJEPU2Bp2uRE+g0ZgP/ict/Q8awLpbh6mCvzfNK629
KuoPZlOJLYnhmnt/1C1J2KB+/IymvORRyD5KHtz9WMuz8aZi0takM2G5b9w1Lxe1
vUBxOrqH7E308lh822abYIq6O+61FUqoKQipCc6HIxsQQzQa/hp3JbntPNP1qTzL
Z/k2x5wv+9be77tG4XjDryJ1aSvGFoCXtT7AO9KqFlCeNFWeTl1kNrYm8FfaqkGC
x3FfhlH2heG45FkCBWckPNnPsrXXBLkkLMRc76mMRLZA3ZO/foqLeOa1L2WOZKCS
xKHRD8mHWlHusSXLETlYhOsScSVq1LJKfXtPJM42RuDhKAE2NsQXDf+yQsT1vyx+
aJ797L1NXMxQDrT+wiXeTH29HIMR04JQR/ZRkmwkXiccfS0j3aMoy21UUReoEZO7
GG3x3l73NHii8HbjUJF6JcWodiqtiovJs1D3x9FQJQIW41RzFErwnUKJkTnl0Zh8
t5qZ2cYNVbVBDkRnmdt88OD4YCe689hzwoeICxgmrBURqruhgqurHvE4rs/jmPf6
ZchDO8WnOpIisIuHtJszyRgH8VxBv6W21Ur8MwuV9RhgPFU5dTtjrziqo4w3U1pq
nVbYSFxaPoaumlsZHKgwrQ8EF0mXi9fP3GkKOlts0IeN/8Ge+cp7wq8oYe/mdNHb
59v8eFSNoya8V55D433j
=7AUh
-----END PGP SIGNATURE-----

--p2kqVDKq5asng8Dg--