SOLVED bug processing zip file with errors

[Fred Broughton]

I'll pass this on to Ned. Wish I had had the same luck, but mine still dies.

Phred


-----Original Message-----
From: Mariano Absatz [mailto:mailscanner at LISTS.COM.AR] 
Sent: Wednesday, June 30, 2004 16:25
To: MAILSCANNER at JISCMAIL.AC.UK
Subject: SOLVED bug processing zip file with errors

WOW!!! IT WORKED!!!
(the line above is to increase my SA scoring ;-)

FTR, the Archive::Zip version 1.11alpha1 that Ned sent thru Phred worked OK 
with my corrupt test zip in my RH6.1+ZMailer 2.99.56+MS 4.29-7 setup (I can't 
put alpha releases in the Fedora box, but I'm eagerly waitin for a more 
formal release to install).

The MailScanner log is at 
http://baby.com.ar/MailScanner/err_zip/mailscanner_now_it_worked.log

Thanx a lot Ned & Phred (& Julian, who apparently had previously sent Ned 
patches for this).

Guys... this seems to me like Open Source at its best :-D

El 30 Jun 2004 a las 18:02, Mariano Absatz escribió:

> Thanx a lot Fred for contacting Ned...
> 
> After debuggin MailScanner and Archive::Zip (adding 'print STDERR' here and 
> there) I saw that the problem was Archive::Zip looping and not MailScanner.
> 
> Will try this version of Archive::Zip right away and tell you about it...
> 
> El 30 Jun 2004 a las 15:44, Fred Broughton escribió:
> 
> > I have an alpha release of Archive::Zip rev 1.11 from Ned Konz and his
> > OK to share with the list. We should try to give him some feedback ASAP.
> > Here are his notes.
> > 
> > Hi Fred,
> > 
> > I added a patch from Julian Field that is supposed to prevent looping on
> > certain corrupt files.
> > 
> > I have also added code to deal with a certain sub-type of the Zip files
> > that is used by (for instance) Java JAR files in some cases.
> > 
> > I've attached it; can you see if it makes a difference?
> > 
> > Thanks,
> > -- 
> > Ned Konz
> > http://bike-nomad.com
> > 
> > Phred Wrote> Excellent! Is it OK with you if I share this in the
> > MailScanner list?
> > 
> > Sure. The code could stand a few more eyes before I release it.
> > 
> > I sent a copy to the Amavis people too but haven't heard back.
> > 
> > The changes since 1.10 include:
> > 
> >   - Fix to avoid endless looping on certain corrupt zip files
> >   - Handling of zip files with data descriptors (like some Java JAR
> > files)
> >   - Handle unicode strings as input to Archive::Zip::Member::contents(),
> > 
> >     Archive::Zip::Member::contents(), and 
> > Archive::Zip::Archive::zipfileComment()
> > 
> > Ned

--
Mariano Absatz
El Baby
----------------------------------------------------------
Pentiums melt in your PC, not in your hand.

-------------------------- MailScanner list ----------------------
To leave, send    leave mailscanner    to jiscmail at jiscmail.ac.uk
Before posting, please see the Most Asked Questions at
http://www.mailscanner.biz/maq/     and the archives at
http://www.jiscmail.ac.uk/lists/mailscanner.html

-------------------------- MailScanner list ----------------------
To leave, send    leave mailscanner    to jiscmail at jiscmail.ac.uk
Before posting, please see the Most Asked Questions at
http://www.mailscanner.biz/maq/     and the archives at
http://www.jiscmail.ac.uk/lists/mailscanner.html