X-Spam-Check-By: sourceware.org Date: Mon, 19 Jun 2006 11:46:36 +0200 From: Corinna Vinschen To: cygwin@cygwin.com Subject: Re: fork failure patch for cygrunsrv Message-ID: <20060619094636.GA3346@calimero.vinschen.de> Reply-To: cygwin@cygwin.com Mail-Followup-To: cygwin@cygwin.com References: <93BD0000E4D72D458F0E8CDE6BA971A833D8BE@CINMLVEM11.e2k.ad.ge.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <93BD0000E4D72D458F0E8CDE6BA971A833D8BE@CINMLVEM11.e2k.ad.ge.com> User-Agent: Mutt/1.4.2i Mailing-List: contact cygwin-help@cygwin.com; run by ezmlm Precedence: bulk List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: cygwin-owner@cygwin.com Mail-Followup-To: cygwin@cygwin.com Delivered-To: mailing list cygwin@cygwin.com On Jun 16 16:49, Hochstedler, Ben (GE Healthcare) wrote: > 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=kb;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. Thanks for the patch. I've uploaded a new cygrunsrv with this patch applied. Just... next time, please include a ChangeLog entry. Thanks, Corinna -- Corinna Vinschen Please, send mails regarding Cygwin to Cygwin Project Co-Leader cygwin AT cygwin DOT com Red Hat -- 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/