amd64 build of unshield fails to extract files due to checksum error

I've created a debdiff of this patch to create a 0.5.3-ubuntu1 version
build and debdiff looks Ok to me, however I've not been able to fully test as I don't have a installshield cab at hand

There is a mention of this problem on ubuntuforums and I've attached the amd64 deb packages that include the above patch if anyone wants them

http://ubuntuforums.org/showthread.php?t=660489

Thank you for your debdiff.
Could you explain the steps required to trigger this bug and how to test your fix?

Hi Luca,

   To test it you'll need to have an installshield cab file (different format than the more common microsoft cab file) on a amd64 version of unix
   I haven't been able to test my changes completely as I haven't found that type of cab file

   once you have such a file the command
     unshield x file.cab
   should generate following error :
[unshield_file_save:710] MD5 checksum failure for file 66 (extractedfilename)
Failed to extract file 'extractedfilename'.

  the corrected version should be able to extract the files correctly

Hi,

  I've finally managed to find an installshield .cab file and tried it with the current version repos on amd64 and it works for that file, ..., still looking for a cab file that reproduces this problem
  if anyone has one please attach it to this bug

I've tried with a second installshield .cab file and again, the current version in the ubuntu repos works correctly on amd64

I propose to unsubscribe sponsors for universe until someone can attach a .cab that causes this issue

Hi all,

I don't have my failing cab file anymore, but I found more examples! The trick is to get a Installshield .cab file and .hdr file in the same directory otherwise I get "Failed to open ~/data1.cab as an InstallShield Cabinet File"

I don't have attached a cab file, so we don't start illegal distribution here, but a quick google will give multiple files to test:
http://www.google.com/search?q=%22index+of%22+data1.cab+data2.cab+setup.inx
Just go to a site and download data1.cab and data1.hdr and test unshield.

I do have attached a script to test it with a vpn client I found. And I have attached the output of what the result is in the text file.

output file

Hi,

   dirk, thanks for the links

   I tried with the data1.cab, data1.hdr and data2.cab files and can now reproduce the problem with the current version in the ubuntu repos on 64bit
   I also tried with the amended version (using the debdiff I attached previously) and there the files extracted correctly - so on amd64 the change seems to work
   debdiff : https://bugs.launchpad.net/ubuntu/+source/unshield/+bug/172901/comments/2

   I also tried the old and the new version on 32 bit and both worked there - so the debdiff seems to be OK to me and tested (on 64 and 32 bit)

  the files I used to test can be found here : http://160.78.48.20/vpn/software/Windows/InstallShield/

   Koen

It might be a good idea to propose this patch upstream. unshield is maintained by the SynCE project on SourceForge (http://synce.sf.net), or you can chat with the developers (see http://www.synce.org/moin/SynceChat).

This bug was fixed in the package unshield - 0.5-3ubuntu1

---------------
unshield (0.5-3ubuntu1) hardy; urgency=low

  * debian/rules : amended to include dpatch patch system
  * debian : added patches directory
  * debian/patches : added 00list containing 01_amd64-fix.dpatch
  * debian/patches : added 01_amd64-fix.dpatch (LP: #172901)
      linked to http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=350210
  * debian/roles : changed maintainer as per DebianMaintainerField spec
  * debian/control : added depend on dpatch

 -- Koen Beek <email address hidden> Wed, 16 Jan 2008 01:44:39 +0100

Very nice debdiff. Keep up the good work and thank you for your contribution to Ubuntu.

Hi hggdh,

  according to http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=350210#20 the patch has already been committed upstream

   Koen