Mailing-List:	contact cygwin-help AT cygwin DOT com; run by ezmlm
List-Subscribe:	<mailto:cygwin-subscribe AT cygwin DOT com>
List-Archive:	<http://sources.redhat.com/ml/cygwin/>
List-Post:	<mailto:cygwin AT cygwin DOT com>
List-Help:	<mailto:cygwin-help AT cygwin DOT com>, <http://sources.redhat.com/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
From:	"Robert Collins" <robert DOT collins AT syncretize DOT net>
To:	"'Pavel Tsekov'" <cygwin AT cygwin DOT com>, "'Max Bowsher'" <maxb AT ukf DOT net>
Subject:	RE: Diagnosis of setup crash (was: Re: Re[3]: cygwin download incomplete - try again - loop?)
Date:	Mon, 10 Jun 2002 20:26:01 +1000
Message-ID:	<006e01c21069$3871d570$0200a8c0@lifelesswks>
MIME-Version:	1.0
X-Priority:	3 (Normal)
X-MSMail-Priority:	Normal
In-reply-to:	<14910202790.20020610120910@syntrex.com>
Importance:	Normal
X-MimeOLE:	Produced By Microsoft MimeOLE V6.00.2600.0000

> -----Original Message-----
> From: cygwin-owner AT cygwin DOT com 
> [mailto:cygwin-owner AT cygwin DOT com] On Behalf Of Pavel Tsekov
> Sent: Monday, 10 June 2002 8:09 PM
> To: Max Bowsher
> Cc: cygwin AT cygwin DOT com
> Subject: Re: Diagnosis of setup crash (was: Re: Re[3]: cygwin 
> download incomplete - try again - loop?)
> 
> 
> MB> The crash occurs because rfc1738_do_escape is passed 
> url=0x0. This zero pointer
> MB> is gereated in rfc1738_escape_part by url.cstr_oneuse(), 
> where url = String("").
> 
> MB> rfc_escape_part("") is called from check_for_cached(). 
> The "" value comes from
> pkgsource.sites[n]->>key. The site objects with a key of "" 
> are created somewhere
> MB> in the yacc/lex stuff (of which I know nothing) based 
> upon the mirror value
> MB> produced in IniParseFindVisitor::visitFile(). Changing < 
> mirror = String (); >
> MB> to < mirror = String ("."); > on line 69 of 
> IniParseFindVisitor.cc eliminates
> MB> the crash.
> 
> Well, it seems like this is a result of the fix applied for the
> previous bug. Btw IMO this can be fixed by skipping the ini_init()
> part and all its dependencies when the mirror is determined to be
> empty. I.e. the previous fix is OK , but some more code has to be
> moved to the block "prefix.size() > 0".

Hmm, it shouldn't need anymore than what I've checked in.

Rob


--
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
Bug reporting:         http://cygwin.com/bugs.html
Documentation:         http://cygwin.com/docs.html
FAQ:                   http://cygwin.com/faq/

- Raw text -