Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test_firmware_zip fails with 2.3.3 upgrade #566

Closed
mweinelt opened this issue Oct 10, 2021 · 1 comment
Closed

test_firmware_zip fails with 2.3.3 upgrade #566

mweinelt opened this issue Oct 10, 2021 · 1 comment

Comments

@mweinelt
Copy link

mweinelt commented Oct 10, 2021

We (NixOS) are just in the process of upgrading from 2.2.2 to 2.3.3 and the firmware.zip test is failing.

FAIL: Test: Open firmware.zip, scan for signatures
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/nix/store/7vl0qcb8gkxllmrzgsdvnw3yyca51rcb-python3.9-nose-1.3.7/lib/python3.9/site-packages/nose/case.py", line 197, in runTest
    self.test(*self.arg)
  File "/build/source/testing/tests/test_firmware_zip.py", line 32, in test_firmware_zip
    eq_(len(scan_result[0].results), len(expected_results))
AssertionError: 4 != 2

These are the descriptions of the scan_result:

Zip archive data, at least v1.0 to extract, name: dir655_revB_FW_203NA/
Zip archive data, at least v2.0 to extract, compressed size: 6395868, uncompressed size: 6422554, name: dir655_revB_FW_203NA/DIR655B1_FW203NAB02.bin
Zip archive data, at least v2.0 to extract, compressed size: 14243, uncompressed size: 61440, name: dir655_revB_FW_203NA/dir655_revB_release_notes_203NA.doc
End of Zip archive, footer length: 22

and this is what was expected:

    expected_results = [
	[0, 'Zip archive data, at least v1.0 to extract, name: dir655_revB_FW_203NA/'],
	[6410581, 'End of Zip archive, footer length: 22'],
    ]

Looks related to this change: dd4f2ef

mweinelt added a commit to NixOS/nixpkgs that referenced this issue Oct 10, 2021
mweinelt added a commit to NixOS/nixpkgs that referenced this issue Oct 10, 2021
@gyakovlev
Copy link

we hit same on gentoo. reverting second hunk, that touches testing/tests/test_firmware_zip.py was enough to fix, we did not revert the traversal test.

gentoo-bot pushed a commit to gentoo/gentoo that referenced this issue Oct 28, 2021
also partially revert test result change
ReFirmLabs/binwalk#566

Closes: https://bugs.gentoo.org/820359
Signed-off-by: Georgy Yakovlev <gyakovlev@gentoo.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants