Page MenuHomeFreeBSD
Differential D42490

certctl: Convert line endings before inspecting files.
ClosedPublic
Actions

Authored by des on Nov 7 2023, 1:14 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, May 22, 11:35 AM
Unknown Object (File)
Sun, May 19, 9:11 AM
Unknown Object (File)
Sun, May 19, 2:56 AM
Unknown Object (File)
Wed, May 8, 6:14 AM
Unknown Object (File)
Apr 23 2024, 1:09 AM
Unknown Object (File)
Apr 23 2024, 1:09 AM
Unknown Object (File)
Apr 23 2024, 1:09 AM
Unknown Object (File)
Apr 23 2024, 12:57 AM
View All 32 Files
Subscribers
imp
michaelo

Details

Reviewers
allanjude
Group Reviewers
security
Commits
rGf7d16a627efa: certctl: Convert line endings before inspecting files.
Summary

This ensures that certificate files or bundles with DOS or Mac line
endings are recognized as such and handled identically to those with
Unix line endings.

PR: 274952

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

des created this revision.Nov 7 2023, 1:14 PM
Herald added a subscriber: imp. · View Herald TranscriptNov 7 2023, 1:14 PM
des requested review of this revision.Nov 7 2023, 1:14 PM
Harbormaster completed remote builds in B54312: Diff 129811.Nov 7 2023, 1:14 PM
allanjude accepted this revision.Nov 7 2023, 7:28 PM

reviewed-by: allanjude

This revision is now accepted and ready to land.Nov 7 2023, 7:28 PM
Closed by commit rGf7d16a627efa: certctl: Convert line endings before inspecting files. (authored by des). · Explain WhyNov 7 2023, 7:54 PM
This revision was automatically updated to reflect the committed changes.
des added a commit: rGf7d16a627efa: certctl: Convert line endings before inspecting files..
michaelo added a comment.Nov 7 2023, 10:07 PM

While technically I have no objection, I still consider that the split approach is wrong as layed out in the PR: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=274952

Revision Contents
Changeset List

PathSize
usr.sbin/
certctl/
9 lines

Diff 129836

View Options

usr.sbin/certctl/certctl.sh

Loading...