X-Recipient:	archive-cygwin AT delorie DOT com
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:subject:message-id:reply-to
	:references:mime-version:content-type:in-reply-to; q=dns; s=
	default; b=IO709uE5uQKcnV8Mwz23pR7DD1ZWSXOJzJTBDhrfRo0HXrSMT47HB
	sxonD2/Og73pAmRhyVLEEP3y2uGZQHT14cyU1G7PnWSP3NfrUxhKan+KRE96J+JD
	fzbC4IO1JTa3m5Ix3GjXdgYaa3s955OChXXSKM1aGNOC1rjGtMQAiw=
DKIM-Signature:	v=1; a=rsa-sha1; c=relaxed; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:date:from:to:subject:message-id:reply-to
	:references:mime-version:content-type:in-reply-to; s=default;
	bh=pN6ExlfkVOTlmqf8QT35Gr3egDg=; b=tWTDe6dlCw0s01RPSFkUZ+qFwDbg
	sLF0oxZwNZt4j5nXEGexJEd7LCMGCuInlKzjWuXzRS/lPLIjCirZfo6ddamEwuqC
	1D88eIEyvreQpoud/31uegGrVev0rKrgM2okQZZ+uW9C2Tk0aZ/Oe4nVdFKBRqeM
	XAQZ0E+URFRp2sE=
Mailing-List:	contact cygwin-help AT cygwin DOT com; run by ezmlm
List-Id:	<cygwin.cygwin.com>
List-Subscribe:	<mailto:cygwin-subscribe AT cygwin DOT com>
List-Archive:	<http://sourceware.org/ml/cygwin/>
List-Post:	<mailto:cygwin AT cygwin DOT com>
List-Help:	<mailto:cygwin-help AT cygwin DOT com>, <http://sourceware.org/ml/#faqs>
Sender:	cygwin-owner AT cygwin DOT com
Mail-Followup-To:	cygwin AT cygwin DOT com
Delivered-To:	mailing list cygwin AT cygwin DOT com
Authentication-Results:	sourceware.org; auth=none
X-Spam-SWARE-Status:	No, score=-100.4 required=5.0 tests=BAYES_00,GOOD_FROM_CORINNA_CYGWIN,KAM_LAZY_DOMAIN_SECURITY,KAM_NUMSUBJECT,RCVD_IN_DNSWL_NONE autolearn=ham version=3.3.2 spammy=retry, tomorrow
X-HELO:	mout.kundenserver.de
Date:	Sun, 17 Feb 2019 10:13:08 +0100
From:	Corinna Vinschen <corinna-cygwin AT cygwin DOT com>
To:	cygwin AT cygwin DOT com
Subject:	Re: fixup_mmaps_after_fork failure with cygwin-3.0.0
Message-ID:	<20190217091308.GK4256@calimero.vinschen.de>
Reply-To:	cygwin AT cygwin DOT com
Mail-Followup-To:	cygwin AT cygwin DOT com
References:	<1d506605-a0af-12c2-6faf-ac74ab842182 AT cornell DOT edu> <20190217090820 DOT GI4256 AT calimero DOT vinschen DOT de>
MIME-Version:	1.0
In-Reply-To:	<20190217090820.GI4256@calimero.vinschen.de>
User-Agent:	Mutt/1.10.1 (2018-07-13)

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

On Feb 17 10:08, Corinna Vinschen wrote:
> On Feb 17 00:11, Ken Brown wrote:
> > I'm getting a lot of errors like this when building emacs from its git =
repo:
> >=20
> > EMACSLOADPATH=3D '../src/emacs.exe' -batch --no-site-file --no-site-lis=
p --eval=20
> > '(setq load-prefer-newer t)'  -f batch-byte-compile ../../master/lisp/f=
inder.el
> >        1 [main] emacs 26848 fixup_mmaps_after_fork: VirtualProtectEx fa=
iled for=20
> > MAP_PRIVATE address 0x6FFFFCD0000, Win32 error 5
> >      190 [main] emacs 26848 C:\Users\kbrown\src\emacs\x86_64\src\emacs.=
exe: ***=20
> > fatal error in forked process - recreate_mmaps_after_fork_failed
> >      619 [main] emacs 26848 cygwin_exception::open_stackdumpfile: Dumpi=
ng stack=20
> > trace to emacs.exe.stackdump
> >        1 [main] emacs 27052 fork: child -1 - forked process 26848 died=
=20
> > unexpectedly, retry 0, exit code 0x100, errno 11
>=20
> Weird.  I built at least gawk and openssh with 3.0 already and I
> didn't notice this problem.
>=20
> > A bisection points to the following as the first bad commit:
> >=20
> > commit 69cc7a068656b5c6ef07ca079a213f801e02e650
> > Author: Corinna Vinschen <corinna AT vinschen DOT de>
> > Date:   Sun Jan 27 13:15:15 2019 +0100
> >=20
> >      Cygwin: fork: restrict parent handle perms and drop handle after u=
se
> >=20
> >=20
> > I'll try to do some debugging and/or provide further information tomorr=
ow, but I=20
> > thought I'd start with this preliminary report.
>=20
> That would be nice.  It's not clear to me which permission is missing.
> The first part of the patch has been reverted in 3.0.0, so=20
> child_info_spawn::get_parent_handle opens the parent with
> PROCESS_VM_READ again.  The second patch has been reworked as well,m
> the current perms are
>=20
> PROCESS_QUERY_LIMITED_INFORMATION | PROCESS_VM_READ | SYNCHRONIZE
>=20
> If I may place a bet, please try to replace
> PROCESS_QUERY_LIMITED_INFORMATION with PROCESS_QUERY_INFORMATION

On second thought, an even better candidate may be PROCESS_VM_OPERATION.

> What OS is that?  W7?


Corinna

--=20
Corinna Vinschen
Cygwin Maintainer

--Rmm1Stw9KgbdL9/H
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQIzBAEBCAAdFiEEoVYPmneWZnwT6kwF9TYGna5ET6AFAlxpJaQACgkQ9TYGna5E
T6Cybw/8DhJbYrgD0Ps9Bes4bZvMJRBWAnvYmJ7LjZMbd6+1+QxvbqpyGyWObtX4
M2Pjylio51IFfJNLPzLc2H+rJ274zGezhu0Jy81QyY1dR+97os2pXe3u3xSwwsSe
anaWMATnzYBbv7kBBKn2VVnjdTxdA/o0FDjXDLBejuI2oeCwcXFeb9TjF28IFjcj
gk9nEnYJJV5CueKbgkqJxrJl9joGrfEtiOjC/9eNdwQTxjrS1faOzFU5wN0n6iB3
e492ylPz0Kzawpm+ndJqyMOjS1rM2jjNhQVF+T3VXG4o1ilbCps0LV4JnI7dVqWV
1x+tGkwhYhGULzDIfXxaHWjqqu/tanIdU8GQR20l7BWxp7abLDF8jUqJrBQ9iZVh
n+eklqWU6U5zOzUqIXdqTTFJFB2M5Uyt6mbeAeRPg1h+bhh48ToZ2/kk6eSP4uq+
VfUz/uyXWghosyztjaZuDnGGicQnyvAtbGLf2pwxg0w2o6OEqsnDdy8qqlBpHthx
4MMlrqw7SSKIO73THvMdJwUip4evfhlDHzOkbKzIm9PmwD0YEoEWWQSxdvUlKFvu
WyQzgqZH4wZ2nduqWnrXynx9FsF74HbWqLyPgYPN0tOufeoLJES/rz+5Bt1X9IpP
Hl/MXylwX8MjLi03vtprX9jiyPBiU/7bf6m+OlH4JWCI8CoTHCQ=
=LY7d
-----END PGP SIGNATURE-----

--Rmm1Stw9KgbdL9/H--

- Raw text -