Bug 1608301 - msiextract broken with updated lbgcab1
Summary: msiextract broken with updated lbgcab1
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: gcab
Version: 28
Hardware: Unspecified
OS: Linux
unspecified
medium
Target Milestone: ---
Assignee: Marc-Andre Lureau
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2018-07-25 09:31 UTC by Fritz Elfert
Modified: 2018-08-20 09:25 UTC (History)
5 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2018-08-20 09:25:02 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
fix (1.78 KB, patch)
2018-07-25 10:56 UTC, Marc-Andre Lureau
no flags Details | Diff

Description Fritz Elfert 2018-07-25 09:31:34 UTC
User-Agent:       Mozilla/5.0 (X11; Fedora; Linux x86_64; rv:61.0) Gecko/20100101 Firefox/61.0
Build Identifier: 

msitools (which depends on libgcab1-1.1-1 on Fedora 28) mostly produces garbage when extracting an msi file.

Reproducible: Always

Steps to Reproduce:
1. Fetch this example: https://github.com/billziss-gh/winfsp/releases/download/v1.3/winfsp-1.3.18160.msi
2. Run msiextract winfsp-1.3.18160.msi
3. Examine ./Program\ Files/WinFsp/inc/fuse/*.h, all of them should be ASCII but at least fuse.h is some binary garbage
Actual Results:  
Most of the files are not extracted correctly

Expected Results:  
All files in the msi should be extracted correctly

If I fetch the old gcab-0.7 src rpm from Fedora27, build and install libgcab1 ant libgcab1-devel, then rpmbuild msitools against this, msiextract starts working again. So this appears to be some incompatibility to the newer libgcab1-1.1

Comment 1 Marc-Andre Lureau 2018-07-25 10:56:52 UTC
Created attachment 1470500 [details]
fix

here is a fix for the regression. However, I am not sure if it's really correct yet. I'll ask Richard his review

Comment 2 Fritz Elfert 2018-08-02 11:08:46 UTC
Hi Marc-Andre,

I have tested your patch and it appears to work fine.

Any progress regarding that review?
Shall I open another Bug against the gcab package?

Comment 3 Fritz Elfert 2018-08-19 21:20:20 UTC
Can you PLEASE trigger a build with this patch applied?


Note You need to log in before you can comment on or make changes to this bug.