Bug 2336914

Summary: binwalk fails to build with Python 3.14: TypeError: unsupported operand type(s) for +=: 'NoneType' and 'int'
Product: [Fedora] Fedora Reporter: Karolina Surma <ksurma>
Component: binwalkAssignee: Adam Jackson <ajax>
Status: NEW --- QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: ajax, fti-bugs, ksurma, mhroncok, suraj.ghimire7, zbyszek
Target Milestone: ---Flags: fti-bugs: needinfo? (ajax)
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 2322407, 2339432, 2339435, 2371704    

Description Karolina Surma 2025-01-10 12:57:35 UTC
binwalk fails to build with Python 3.14.0a3.

4 tests fail with the same traceback
FAILED test_firmware_gzip.py::test_firmware_gzip - AssertionError
FAILED test_firmware_jffs2.py::test_firmware_jffs2 - AssertionError
FAILED test_firmware_squashfs.py::test_firmware_squashfs - AssertionError
FAILED test_firmware_zip.py::test_firmware_zip - AssertionError

Example failure:
______________________________ test_firmware_gzip ______________________________

    def test_firmware_gzip():
        '''
        Test: Open firmware.gzip, scan for signatures.
        Verify that only one gzip signature was detected.
        '''
        input_vector_file = os.path.join(os.path.dirname(__file__),
                                         "input-vectors",
                                         "firmware.gzip")
    
        scan_result = binwalk.scan(input_vector_file,
                                   signature=True,
                                   quiet=True)
    
        # Test number of modules used
        eq_(len(scan_result), 1)
    
        # There should be only one result
>       eq_(len(scan_result[0].results), 1)

test_firmware_gzip.py:23: 
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 

a = 0, b = 1

    def eq_(a, b):
>       assert a == b
E       AssertionError

_nose_shim.py:2: AssertionError
----------------------------- Captured stderr call -----------------------------

Signature Exception: unsupported operand type(s) for +=: 'NoneType' and 'int'
----------------------------------------------------------------------------------------------------
Traceback (most recent call last):
  File "/builddir/build/BUILD/binwalk-2.3.4-build/BUILDROOT/usr/lib/python3.14/site-packages/binwalk/core/module.py", line 595, in main
    retval = self.run()
  File "/builddir/build/BUILD/binwalk-2.3.4-build/BUILDROOT/usr/lib/python3.14/site-packages/binwalk/modules/signature.py", line 192, in run
    self.scan_file(fp)
    ~~~~~~~~~~~~~~^^^^
  File "/builddir/build/BUILD/binwalk-2.3.4-build/BUILDROOT/usr/lib/python3.14/site-packages/binwalk/modules/signature.py", line 150, in scan_file
    for r in self.magic.scan(data, dlen):
             ~~~~~~~~~~~~~~~^^^^^^^^^^^^
  File "/builddir/build/BUILD/binwalk-2.3.4-build/BUILDROOT/usr/lib/python3.14/site-packages/binwalk/core/magic.py", line 827, in scan
    tags = self._analyze(signature, offset)
  File "/builddir/build/BUILD/binwalk-2.3.4-build/BUILDROOT/usr/lib/python3.14/site-packages/binwalk/core/magic.py", line 584, in _analyze
    line_offset = self._do_math(offset, line_offset_text)
  File "/builddir/build/BUILD/binwalk-2.3.4-build/BUILDROOT/usr/lib/python3.14/site-packages/binwalk/core/magic.py", line 505, in _do_math
    o += offset
TypeError: unsupported operand type(s) for +=: 'NoneType' and 'int'
----------------------------------------------------------------------------------------------------

https://docs.python.org/3.14/whatsnew/3.14.html

For the build logs, see:
https://copr-be.cloud.fedoraproject.org/results/@python/python3.14/fedora-rawhide-x86_64/08480129-binwalk/

For all our attempts to build binwalk with Python 3.14, see:
https://copr.fedorainfracloud.org/coprs/g/python/python3.14/package/binwalk/

Testing and mass rebuild of packages is happening in copr.
You can follow these instructions to test locally in mock if your package builds with Python 3.14:
https://copr.fedorainfracloud.org/coprs/g/python/python3.14/

Let us know here if you have any questions.

Python 3.14 is planned to be included in Fedora 43.
To make that update smoother, we're building Fedora packages with all pre-releases of Python 3.14.
A build failure prevents us from testing all dependent packages (transitive [Build]Requires),
so if this package is required a lot, it's important for us to get it fixed soon.

We'd appreciate help from the people who know this package best,
but if you don't want to work on this now, let us know so we can try to work around it on our side.

Comment 1 Aoife Moloney 2025-02-26 13:22:11 UTC
This bug appears to have been reported against 'rawhide' during the Fedora Linux 42 development cycle.
Changing version to 42.

Comment 2 Karolina Surma 2025-06-11 15:52:26 UTC
*** Bug 2371682 has been marked as a duplicate of this bug. ***

Comment 3 Zbigniew Jędrzejewski-Szmek 2025-06-16 10:38:01 UTC
I'll skip the tests with binwalks in diffoscope for now, to allow a build to happen.

Comment 4 Fedora Fails To Install 2025-06-20 19:50:51 UTC
Hello,

Please note that this comment was generated automatically by https://pagure.io/releng/blob/main/f/scripts/ftbfs-fti/follow-policy.py
If you feel that this output has mistakes, please open an issue at https://pagure.io/releng/

This package fails to install and maintainers are advised to take one of the following actions:

 - Fix this bug and close this bugzilla once the update makes it to the repository.
   (The same script that posted this comment will eventually close this bugzilla
   when the fixed package reaches the repository, so you don't have to worry about it.)

or

 - Move this bug to ASSIGNED if you plan on fixing this, but simply haven't done so yet.

or

 - Orphan the package if you no longer plan to maintain it.


If you do not take one of these actions, the process at https://docs.fedoraproject.org/en-US/fesco/Fails_to_build_from_source_Fails_to_install/#_package_removal_for_long_standing_ftbfs_and_fti_bugs will continue.
This package may be orphaned in 7+ weeks.
This is the first reminder (step 3) from the policy.

Don't hesitate to ask for help on https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org/ if you are unsure how to fix this bug.