elf: search dynamic tags within sections, not segment
Metadata
Current evaluation
Fixed ELF dynamic-tag parsing by iterating dynamic sections instead of segments, avoiding invalid data produced by patchelf mangling.
Resolution: closed resolved
Reason: PR #3149 was merged in commit 32a399c4f, changing tag discovery to use dynamic sections and resolving the patchelf-related parsing problem tracked by LP #1879519.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Due to yet more patchelf mangling, snapcraft was tripping on
invalid tag data. We can avoid this by iteration over the
dynamic sections.
LP: 1879519
Signed-off-by: Chris Patterson <chris.patterson@canonical.com>
- [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `./runtests.sh static`?
- [ ] Have you successfully run `./runtests.sh tests/unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Fixed ELF dynamic-tag parsing by iterating dynamic sections instead of segments, avoiding invalid data produced by patchelf mangling. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged ELF parsing fix in PR #3149 (32a399c4f), iterating dynamic sections instead of segments to avoid invalid tag data after patchelf mangling. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged fix resolving snapcraft failures on invalid tag data caused by patchelf mangling. Resolved by iterating over dynamic sections instead of segments. Successfully merged into master. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into master. Updates ELF dynamic tag parsing to iterate over sections instead of segments, resolving invalid tag data errors from patchelf mangling and fixing LP 1879519. |
Update history
No update history recorded yet.
Related work
-
Related To:
snapcore/snapcraft#3111
(confidence 55%)
The preceding commit also addressed ELF parsing after patchelf mangling.
Related issues
No related issues found above the similarity threshold.