[emirmrkaljevic]

Fixes #16166

md5sum marks a line with an escaped file name by starting it with a \. The tagged parser skipped that marker, but the untagged and single-space parsers did not, so md5sum -c rejected its own output for names containing a backslash or a newline. At the same time, every name was unescaped, even on lines without the marker.

This records whether the line started with a \ (LineInfo::escaped, set by all three parsers through strip_escape_marker) and only unescapes the name in that case.

Test: test_check_escaped_file_names (escaped untagged and binary-mode lines, md5sum's own output, and a name that must not be unescaped). All test_md5sum and test_cksum tests and the uucore checksum unit tests pass, clippy and fmt are clean.

Found with behave.

[codspeed]

Merging this PR will not alter performance

1 warning, 1 note

⚠️ 124 benchmarks are dominated by system calls

System calls cannot be simulated consistently and are left out of the measure. The reported value understates the real cost. Switch to the Walltime instrument to measure them.

Learn more about system calls

See affected benchmarks

ℹ️ 1 benchmark has no flamegraph

The flamegraph wasn't generated: most of the time was spent in runtime internals, which would leave nothing useful to analyze.

See affected benchmarks

✅ 401 untouched benchmarks
⏩ 70 skipped benchmarks¹

---

_{Comparing emirmrkaljevic:checksum-escaped-lines (281747d) with main (88707ee)²}

Footnotes

1. 70 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩

2. No successful run was found on main (c9c790b) during the generation of this report, so 88707ee was used instead as the comparison base. There might be some changes unrelated to this pull request in this report. ↩

[sylvestre]

could use strip_escape_marker here too, no?

[emirmrkaljevic]

Yes, it fits: switched it to strip_escape_marker in 281747d, same behaviour, the tests still pass.

[sylvestre]

Thanks for your PR