X-Spam-Check-By:	sourceware.org
MIME-Version:	1.0
Subject:	fork failure patch for cygrunsrv
Date:	Fri, 16 Jun 2006 16:49:48 -0400
Message-ID:	<93BD0000E4D72D458F0E8CDE6BA971A833D8BE@CINMLVEM11.e2k.ad.ge.com>
From:	"Hochstedler, Ben \(GE Healthcare\)" <ben DOT hochstedler AT med DOT ge DOT com>
To:	<cygwin AT cygwin DOT com>
X-IsSubscribed:	yes
Mailing-List:	contact cygwin-help AT cygwin DOT com; run by ezmlm
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
Note-from-DJ:	This may be spam

------_=_NextPart_001_01C69186.6842C1E4
Content-Type: text/plain;
	charset="US-ASCII"
Content-Transfer-Encoding: quoted-printable

fork() can fail when the system runs out of non-interactive heap space
(because there's not enough heap memory to allocate to the launched
process). See:
http://support.microsoft.com/default.aspx?scid=3Dkb;en-us;824422

The attached patch fixes two problems:
1. cygrunsrv doesn't exit when fork returns -1 (failed to
   spawn child process)
2. log message doesn't include the proper errno of the
   fork failure.

Ben Hochstedler




------_=_NextPart_001_01C69186.6842C1E4
Content-Type: application/octet-stream;
	name="cygrunsrv-forkerr.patch"
Content-Transfer-Encoding: base64
Content-Description: cygrunsrv-forkerr.patch
Content-Disposition: attachment;
	filename="cygrunsrv-forkerr.patch"

ZGlmZiAtdU5yIGN5Z3J1bnNydi0xLjE2LTEvY3lncnVuc3J2LmNjIGN5Z3J1
bnNydi0xLjE2LTEubmV3L2N5Z3J1bnNydi5jYwotLS0gY3lncnVuc3J2LTEu
MTYtMS9jeWdydW5zcnYuY2MJMjAwNi0wNi0xNiAxNDoxODozNS42NDkxMjM5
MDAgLTA1MDAKKysrIGN5Z3J1bnNydi0xLjE2LTEubmV3L2N5Z3J1bnNydi5j
YwkyMDA2LTA2LTE2IDE0OjE1OjM2LjQ2NTIzOTQwMCAtMDUwMApAQCAtMTQ4
OSw3ICsxNDg5LDExIEBACiAgIHN3aXRjaCAoY2hpbGRfcGlkKQogICAgIHsK
ICAgICBjYXNlIC0xOiAvKiBFcnJvciAqLwotICAgICAgc3lzbG9nX3N0YXJ0
ZXJyICgiZm9yayIsIDAsIGVycik7CisgICAgICBzeXNsb2dfc3RhcnRlcnIg
KCJmb3JrIiwgMCwgZXJybm8pOworICAgICAgLyogRG8gbm90IHJlcG9ydCB0
aGF0IHRoZSBzZXJ2aWNlIGlzIHN0b3BwZWQgc28gdGhhdCBpZgorCSByZWNv
dmVyeSBvcHRpb25zIGFyZSBzZXQsIFdpbmRvd3Mgd2lsbCBhdXRvbWF0aWNh
bGx5CisJIHJlc3RhcnQgdGhlIHNlcnZpY2UuICovCisgICAgICBleGl0IChl
cnJubyk7CiAgICAgICBicmVhazsKICAgICAgIAogICAgIGNhc2UgMDogIC8q
IENoaWxkICovCg==


------_=_NextPart_001_01C69186.6842C1E4
Content-Type: text/plain; charset=us-ascii

--
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/
------_=_NextPart_001_01C69186.6842C1E4--

- Raw text -