Page MenuHomeFreeBSD
Differential D47286

lib/libc/string: add memset_explicit() for compliance with C23
ClosedPublic
Actions

Authored by fuz on Oct 25 2024, 5:48 PM.
Tags
None
Referenced Files
F168018479: D47286.id146474.diff
Tue, Aug 25, 10:56 PM
F168006061: D47286.diff
Tue, Aug 25, 9:20 PM
Unknown Object (File)
Tue, Aug 25, 2:01 AM
Unknown Object (File)
Fri, Aug 21, 11:04 AM
Unknown Object (File)
Fri, Aug 21, 11:04 AM
Unknown Object (File)
Wed, Aug 19, 6:35 AM
Unknown Object (File)
Tue, Aug 18, 5:36 PM
Unknown Object (File)
Mon, Aug 17, 11:17 PM
View All Files
Subscribers
imp

Details

Reviewers
kevans
emaste
Commits
rG8983acc8dee5: lib/libc/string: apply SSP hardening and tests to memset_explicit
rG007871c35628: lib/libc/string: add memset_explicit() for compliance with C23
Summary

Patterned after explicit_bzero sans the hook.
SSP hardening is provided, too.
The function is documented in memset(3).

Test Plan

passes newly added unit tests

Diff Detail

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

Event Timeline

fuz created this revision.Oct 25 2024, 5:48 PM
Herald added a subscriber: imp. · View Herald TranscriptOct 25 2024, 5:48 PM
fuz requested review of this revision.Oct 25 2024, 5:48 PM
Harbormaster completed remote builds in B60194: Diff 145466.Oct 25 2024, 5:48 PM
fuz added a parent revision: D47284: contrib/mandoc: add -isoC-2024 and -svid1 to mdoc(7).Oct 25 2024, 5:49 PM
fuz updated this revision to Diff 146441.Nov 14 2024, 12:39 PM
  • lib/libc/string/memset_explicit.c: add weak hook to confuse LTO inlining

As requested by @emaste.

Harbormaster completed remote builds in B60557: Diff 146441.Nov 14 2024, 12:39 PM
kevans added a comment.Nov 14 2024, 4:30 PM

fortify bits look good, thanks

include/string.h
74

I believe

kevans accepted this revision.Nov 14 2024, 4:30 PM
This revision is now accepted and ready to land.Nov 14 2024, 4:30 PM
Closed by commit rG007871c35628: lib/libc/string: add memset_explicit() for compliance with C23 (authored by fuz). · Explain WhyNov 14 2024, 10:11 PM
This revision was automatically updated to reflect the committed changes.
fuz added a commit: rG007871c35628: lib/libc/string: add memset_explicit() for compliance with C23.
fuz added a commit: rG8983acc8dee5: lib/libc/string: apply SSP hardening and tests to memset_explicit.

Revision Contents
Changeset List

PathSize
include/
3 lines
lib/
libc/
string/
4 lines
4 lines
18 lines
26 lines

Diff 146474

View Options

include/string.h

Loading...
View Options

lib/libc/string/Makefile.inc

Loading...
View Options

lib/libc/string/Symbol.map

Loading...
View Options

lib/libc/string/memset.3

Loading...
View Options

lib/libc/string/memset_explicit.c

Loading...