← Back to issue list

elf: use surrogate escape when decoding readelf output

View original Github issue

Metadata

Project
snapcraft
Number
#1903
Type
pull request
State
merged
Author
come-maiz
Labels
—
Created
Updated
Closed

Current evaluation

Merged a one-line change to use surrogate escape when decoding readelf output, resolving Launchpad bug 1746639. The fix handles non-UTF-8 characters in ELF parsing and was approved without formal review.

Suggested action: —

No scores available.

Issue body

- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)? - [x] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)? - [x] Have you successfully run `./runtests.sh static`? - [x] Have you successfully run `./runtests.sh unit`? ----- Fixes [LP:1746639](https://bugs.launchpad.net/snapcraft/+bug/1746639)

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b — — Merged a one-line change to use surrogate escape when decoding readelf output, resolving Launchpad bug 1746639. The fix handles non-UTF-8 characters in ELF parsing and was approved without formal review.
qwen3.6-35b-a3b-mtp-q6 — — Merged changes to the ELF module that implement surrogate escape for decoding readelf output, resolving Launchpad bug LP:1746639.
qwen3.6-35b-a3b-mtp-q6 — — Merged pull request implementing surrogate escape handling for decoding readelf output, resolving Launchpad bug LP:1746639. The change prevents decoding errors when parsing ELF binaries containing non-UTF-8 characters.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#1899 elf: readelf dependency snapcraft closed Merged. The change ensures the readelf dependency is properly added and utilized based on the execution environment. Approved by one reviewer with passing checks.
72%