public inbox for cygwin@cygwin.com
 help / color / mirror / Atom feed
* RE: setup.exe local install with 2.218.2.4 fails, 2.78.2.15works
@ 2002-05-14  6:35 Robert Collins
  2002-05-14  6:53 ` new 2.218.2.6 setup also freezes Sander Timmermans
  0 siblings, 1 reply; 2+ messages in thread
From: Robert Collins @ 2002-05-14  6:35 UTC (permalink / raw)
  To: Kilroy, David, Cygwin

There is a newer version available ... .6 that has such a fix in it
already.

Rob

> -----Original Message-----
> From: Kilroy, David [mailto:david.kilroy@detroitdiesel.com] 
> Sent: Tuesday, May 14, 2002 10:51 PM
> To: Cygwin
> Subject: RE: setup.exe local install with 2.218.2.4 fails, 
> 2.78.2.15works
> 
> 
> > It seems that all of these bug reports are on Win2K and XP Pro.
> > Has anyone successfully tested the recent setup.exe releases 
> > (since the MD5 updates) on either of these platforms?
> 
> I can confirm that setup 2.218.2.4 works on my Win2k SP2 box. 
> Both install from internet and install from local directory.
> 
> I did get the stack and parse errors in previous md5 setup snapshots.
> 
> In 2.218.2.4 you will also see a crash if you have a file in 
> the local cache directory with a 10 letter filename _and_ the 
> string 'setup.ini' in the full path/filename (Unfortunate if 
> you edit setup.ini with emacs). This may or may not have been 
> fixed in .5 and .6; if it hasn't, see description below.
> 
> 
> Dave.
> 
> 
> 
> line nos from 2.218.2.4 src posted to setup_snapshots.
> ini.cc: find_routine
> 
> line 68:
>   if (!strstr (path, "setup.ini"))
>     return;
> 
> ...
> 
> line 85:
>   /* Attempt to unescape the string */
>   path[strlen(path) -10] = '\0';
>   String mirror = rfc1738_unescape_part (path);
> 
> tries to put null terminator at the last separator.
> 
> If at this point you are at the root of the cache, and you 
> are looking at setup.ini, the byte _before_ path is touched 
> (which may affect further recursion of directories, see 
> find.cc). The file is then parsed. Packages added to the 
> mirror 'setup.ini.
> 
> If we are looking at setup.ini in a cached mirror directory, 
> everything works fine.
> 
> If strlen(path) is 10 at this point (as with the file 
> setup.ini~) the null is placed at the first char of path, 
> causing the rfc1738 code to segfault (passing it a null string).
> 
> We attempt to process other files that match and do not cause 
> this fault. If it is indeed a setup backup, the package 
> entries are added to the database under a bad? mirror name 
> (hence files will not be found if that package is selected?).
> 
> Possible fix off the top of my head (which matches files _only_ called
> setup.ini):
> 
> line 68:
> 	if(!strcmp(path + strlen(path)- 9, "setup.ini"))
> 
> 
> --
> 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/
> 
> 

--
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/

^ permalink raw reply	[flat|nested] 2+ messages in thread

* new 2.218.2.6 setup also freezes
  2002-05-14  6:35 setup.exe local install with 2.218.2.4 fails, 2.78.2.15works Robert Collins
@ 2002-05-14  6:53 ` Sander Timmermans
  0 siblings, 0 replies; 2+ messages in thread
From: Sander Timmermans @ 2002-05-14  6:53 UTC (permalink / raw)
  To: Cygwin

WIN2000 SP2
after reading the package directory from ftp it leaves only the window with:
"This space intentionally left blank". 100% CPU 100% disappointment :(

Sander

> -----Original Message-----
> From: cygwin-owner@cygwin.com [mailto:cygwin-owner@cygwin.com]On Behalf
> Of Robert Collins
> Sent: 14 May 2002 14:58
> To: Kilroy, David; Cygwin
> Subject: RE: setup.exe local install with 2.218.2.4 fails,
> 2.78.2.15works
>
>
> There is a newer version available ... .6 that has such a fix in it
> already.
>
> Rob
>
> > -----Original Message-----
> > From: Kilroy, David [mailto:david.kilroy@detroitdiesel.com]
> > Sent: Tuesday, May 14, 2002 10:51 PM
> > To: Cygwin
> > Subject: RE: setup.exe local install with 2.218.2.4 fails,
> > 2.78.2.15works
> >
> >
> > > It seems that all of these bug reports are on Win2K and XP Pro.
> > > Has anyone successfully tested the recent setup.exe releases
> > > (since the MD5 updates) on either of these platforms?
> >
> > I can confirm that setup 2.218.2.4 works on my Win2k SP2 box.
> > Both install from internet and install from local directory.
> >
> > I did get the stack and parse errors in previous md5 setup snapshots.
> >
> > In 2.218.2.4 you will also see a crash if you have a file in
> > the local cache directory with a 10 letter filename _and_ the
> > string 'setup.ini' in the full path/filename (Unfortunate if
> > you edit setup.ini with emacs). This may or may not have been
> > fixed in .5 and .6; if it hasn't, see description below.
> >
> >
> > Dave.
> >
> >
> >
> > line nos from 2.218.2.4 src posted to setup_snapshots.
> > ini.cc: find_routine
> >
> > line 68:
> >   if (!strstr (path, "setup.ini"))
> >     return;
> >
> > ...
> >
> > line 85:
> >   /* Attempt to unescape the string */
> >   path[strlen(path) -10] = '\0';
> >   String mirror = rfc1738_unescape_part (path);
> >
> > tries to put null terminator at the last separator.
> >
> > If at this point you are at the root of the cache, and you
> > are looking at setup.ini, the byte _before_ path is touched
> > (which may affect further recursion of directories, see
> > find.cc). The file is then parsed. Packages added to the
> > mirror 'setup.ini.
> >
> > If we are looking at setup.ini in a cached mirror directory,
> > everything works fine.
> >
> > If strlen(path) is 10 at this point (as with the file
> > setup.ini~) the null is placed at the first char of path,
> > causing the rfc1738 code to segfault (passing it a null string).
> >
> > We attempt to process other files that match and do not cause
> > this fault. If it is indeed a setup backup, the package
> > entries are added to the database under a bad? mirror name
> > (hence files will not be found if that package is selected?).
> >
> > Possible fix off the top of my head (which matches files _only_ called
> > setup.ini):
> >
> > line 68:
> > 	if(!strcmp(path + strlen(path)- 9, "setup.ini"))
> >
> >
> > --
> > 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/
> >
> >
>
> --
> 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/
>
>


--
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/

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2002-05-14 13:16 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2002-05-14  6:35 setup.exe local install with 2.218.2.4 fails, 2.78.2.15works Robert Collins
2002-05-14  6:53 ` new 2.218.2.6 setup also freezes Sander Timmermans

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).