failed to extract .cab file: incorrect checksum detected

[Nafallo Bjälevik]

nafallo@manticore:~$ fwupdmgr refresh
Fetching metadata https://s3.amazonaws.com/lvfsbucket/downloads/firmware.xml.gz
Downloading… [***************************************]
Fetching signature https://s3.amazonaws.com/lvfsbucket/downloads/firmware.xml.gz.asc

nafallo@manticore:~$ fwupdmgr update
Downloading RQR12.07_B0029 for Unifying Receiver...
Fetching firmware https://fwupd.org/downloads/938fec082652c603a1cdafde7cd25d76baadc70d-Logitech-Unifying-RQR12.07_B0029.cab
Downloading… [***************************************]
Updating RQR12.07_B0029 on Unifying Receiver...
failed to extract .cab file: incorrect checksum detected

nafallo@manticore:~$ fwupdmgr get-updates|grep Checksum
Update Checksum: SHA1(d0d33e760ab6eeed6f11b9f9bd7e83820b29e970)

nafallo@manticore:~$ sha1sum .cache/fwupdmgr/938fec082652c603a1cdafde7cd25d76baadc70d-Logitech-Unifying-RQR12.07_B0029.cab
d0d33e760ab6eeed6f11b9f9bd7e83820b29e970 .cache/fwupdmgr/938fec082652c603a1cdafde7cd25d76baadc70d-Logitech-Unifying-RQR12.07_B0029.cab

[Nafallo Bjälevik (nafallo)]

This could well be a bug in the library extracting the .cab files. I'm not sure.

[Nafallo Bjälevik (nafallo)]

nafallo@manticore:~$ gcab -x .cache/fwupdmgr/938fec082652c603a1cdafde7cd25d76baadc70d-Logitech-Unifying-RQR12.07_B0029.cab
gcab: error during extraction: incorrect checksum detected

Looks like the problem will be in libgcab.

[Nafallo Bjälevik (nafallo)]

After downgrading to Artful gcab and libgcab:

nafallo@manticore:~$ gcab -x .cache/fwupdmgr/938fec082652c603a1cdafde7cd25d76baadc70d-Logitech-Unifying-RQR12.07_B0029.cab
nafallo@manticore:~$

[Richard Hughes (richard-hughes)]

What architecture is this? If it's big endian you need my fixes from git: https://git.gnome.org//browse/gcab

[Nafallo Bjälevik (nafallo)]

I found https://github.com/hughsie/fwupd/issues/318 , and it looks like the fix for big endian may have caused the same issue for little endian. Subscribing Mario for insights.

[Stephen Kitt (steve-sk2)]

Version 0.7-5 in Bionic already includes https://git.gnome.org/browse/gcab/commit/?h=wip/hughsie/meson&id=18557a4c7811355987abf9efe91c61b1d17327b8 (in fact, that’s pretty much the only difference between the Artful package and the Bionic package).

[Nafallo Bjälevik (nafallo)]

@richard-hughes: amd64
@steve-sk2: indeed. 0.7-5 have the incorrect checksum on little-endian (amd64), whilst 0.7-4 works.

[Richard Hughes (richard-hughes)]

I've pushed the upstream fix as https://git.gnome.org/browse/gcab/commit/?id=cbfff7a2175c7f970761c800411c46eed41129b4

[Nafallo Bjälevik (nafallo)]

Confirmed Richard's commit fixes this, once you restart the fwupd daemon as well.
Stephen, are you happy to pull the patch into sid?

[Stephen Kitt (steve-sk2)]

Done, thanks for the investigation (and Richard for the fix)!

[Launchpad Janitor (janitor)]

This bug was fixed in the package gcab - 0.7-6

---------------
gcab (0.7-6) unstable; urgency=medium

  * Apply upstream fix to the fix for checkstum calculations on big-endian
    and little-endian platforms. LP: #1737723.
  * Standards-Version 4.1.2, no change required.

 -- Stephen Kitt <email address hidden> Tue, 12 Dec 2017 20:36:48 +0100