Re: [PATCH] ld: testsuite: Fix integer overflow in ld/testsuite/ld-elf/init-mixed.c.
Project / Subsystem
binutils / ld
Date
2026-07-20
Proposer
Jan Beulich <jbeulich@suse.com>
Source type
public_inbox
Consensus
Proposed
Sentiment
—/10
Technical tradeoffs
- • Correctness of test cases.
- • Portability and robustness of code relying on assumed integer sizes.
All attributes
- project
- binutils
- subsystem
- ld
- patch_id
- —
- discussion_id
- 9c15be6c-77a8-4817-9fa7-7c88e9f638eb@suse.com
- source_type
- public_inbox
- title
- Re: [PATCH] ld: testsuite: Fix integer overflow in ld/testsuite/ld-elf/init-mixed.c.
- headline
- ld: testsuite: Fix integer overflow in ld/testsuite/ld-elf/init-mixed.c.
- tldr
- Jan Beulich agrees with a patch fixing an integer overflow in a binutils linker test case, noting such assumptions are common.
- proposer
- Jan Beulich <jbeulich@suse.com>
- consensus
- Proposed
- outcome
- proposed
- sentiment_score
- —
- technical_tradeoffs
-
- • Correctness of test cases.
- • Portability and robustness of code relying on assumed integer sizes.
- series_id
- —
- series_role
- standalone
- series_parts
- []
- tags
-
- • binutils
- • ld
- • testsuite
- • integer overflow
- • c
- bugzilla_url
- —
- date
- 2026-07-20T00:00:00.000Z
Re: [PATCH] ld: testsuite: Fix integer overflow in ld/testsuite/ld-elf/init-mixed.c.
Jan Beulich acknowledges a patch intended to fix an integer overflow in a binutils linker test case (ld/testsuite/ld-elf/init-mixed.c). The original patch noted that the code assumed int was 32 bits, which isn’t always true. Beulich agrees with the problem but points out that similar assumptions about int size are prevalent throughout the binutils source code.