Mailing-List: contact cygwin-developers-help AT sourceware DOT cygnus DOT com; run by ezmlm List-Subscribe: List-Archive: List-Post: List-Help: , Sender: cygwin-developers-owner AT sources DOT redhat DOT com Delivered-To: mailing list cygwin-developers AT sources DOT redhat DOT com Date: Fri, 7 Sep 2001 12:19:43 -0400 From: Christopher Faylor To: cygwin-developers AT cygwin DOT com Subject: Re: I know CVS is hosed (fix checked in) (did it work?) (anyone there?) Message-ID: <20010907121943.A27950@redhat.com> Reply-To: cygwin-developers AT cygwin DOT com Mail-Followup-To: cygwin-developers AT cygwin DOT com References: <20010906113843 DOT A30174 AT redhat DOT com> <20010906154534 DOT A32372 AT redhat DOT com> <20010906210535 DOT A8264 AT redhat DOT com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20010906210535.A8264@redhat.com> User-Agent: Mutt/1.3.21i On Thu, Sep 06, 2001 at 09:05:35PM -0400, Christopher Faylor wrote: >On Thu, Sep 06, 2001 at 03:45:34PM -0400, Christopher Faylor wrote: >>On Thu, Sep 06, 2001 at 11:38:43AM -0400, Christopher Faylor wrote: >>>While attempting to cut down on the size of the 1.3.3 DLL, I uncovered a few >>>problems with cygheap. I couldn't track them down before I went to bed. >>> >>>The symptom is that applications die in cfree. The problem manifests quickly >>>in a process which execs a process which execs a process. >>> >>>I haven't seen the problem that Egor reported with free but it probably is not >>>related to the cygheap problem. It probably is somehow related to the new >>>code in sigproc which allocates the zombie array dynamically. >> >>The fix for this turned out to be very trivial. It was just the requirement >>for a null pointer check. Doh. >> >>I've checked the fix in. >> >>I'd again appreciate feedback on whether this fixes the problem or not. It >>seems to for me. >> >>This is not a fix for the rsync problem that was recently reported. > >Did this patch work? It seems to for me. The last we heard, another similar patch to this one didn't work for Corinna. I'd like to get confirmation that this one works ok. cgf