[BusyBox 0001617]: unzip bug in 1.7.3

[bugs at busybox.net]

The following issue has been CLOSED 
====================================================================== 
http://busybox.net/bugs/view.php?id=1617 
====================================================================== 
Reported By:                jocke
Assigned To:                BusyBox
====================================================================== 
Project:                    BusyBox
Issue ID:                   1617
Category:                   Other
Reproducibility:            always
Severity:                   minor
Priority:                   urgent
Status:                     closed
Resolution:                 reopened
Fixed in Version:           
====================================================================== 
Date Submitted:             11-28-2007 16:53 PST
Last Modified:              11-29-2007 00:20 PST
====================================================================== 
Summary:                    unzip bug in 1.7.3
Description: 
I can't unzip the attached(or any other zip file) zip file with bb 1.7.3
I get:
# > unzip tst.zip 
Archive:  tst.zip
unzip: can't open '': No such file or directory

zip file file is created by:
# >zip -v
Copyright (C) 1990-2005 Info-ZIP
Type 'zip "-L"' for software license.
This is Zip 2.31 (March 8th 2005), by Info-ZIP.

====================================================================== 

---------------------------------------------------------------------- 
 jocke - 11-28-07 16:58  
---------------------------------------------------------------------- 
Forgot to mention that this is on PowerPC 

---------------------------------------------------------------------- 
 vda - 11-28-07 22:47  
---------------------------------------------------------------------- 
This is fixed in svn. The problem is that endianness was handled improperly
and all BE targets were broken.

1.8.2 should work too, can you test it? 

---------------------------------------------------------------------- 
 jocke - 11-28-07 23:58  
---------------------------------------------------------------------- 
I could test but that would not help much as we are close
to release and I really don't want to do a such an upgrade now.

I found this in SVN:
http://www.busybox.net/cgi-bin/viewcvs.cgi/trunk/busybox/archival/unzip.c?rev=20295&r1=20219&r2=20295
Is that enough to fix the problem? 

---------------------------------------------------------------------- 
 vda - 11-29-07 00:20  
---------------------------------------------------------------------- 
Yes, this is the fix. Or just copy unzip.c over from 1.8.2 (you will fix
some fallout, but it is typically trivial, no thinking involved). 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
11-28-07 16:53  jocke          New Issue                                    
11-28-07 16:53  jocke          Status                   new => assigned     
11-28-07 16:53  jocke          Assigned To               => BusyBox         
11-28-07 16:53  jocke          File Added: tst.zip                          
11-28-07 16:58  jocke          Note Added: 0002987                          
11-28-07 22:47  vda            Status                   assigned => closed  
11-28-07 22:47  vda            Note Added: 0002988                          
11-28-07 22:47  vda            Resolution               open => fixed       
11-28-07 23:58  jocke          Status                   closed => feedback  
11-28-07 23:58  jocke          Resolution               fixed => reopened   
11-28-07 23:58  jocke          Note Added: 0002989                          
11-29-07 00:20  vda            Status                   feedback => closed  
11-29-07 00:20  vda            Note Added: 0002990                          
======================================================================