Page MenuHomeFreeBSD
Differential D41512

libc: Add unit tests for N2630 and possible collateral damage.
ClosedPublic
Actions

Authored by des on Aug 19 2023, 11:47 PM.
Tags
None
Referenced Files
F175299146: D41512.id126552.diff
Fri, Oct 9, 6:53 PM
F175299102: D41512.id126557.diff
Fri, Oct 9, 6:52 PM
F175298984: D41512.id126331.diff
Fri, Oct 9, 6:51 PM
F175298963: D41512.id126595.diff
Fri, Oct 9, 6:51 PM
F175298853: D41512.id126242.diff
Fri, Oct 9, 6:50 PM
F175293570: D41512.diff
Fri, Oct 9, 5:53 PM
Unknown Object (File)
Sat, Oct 3, 12:40 PM
Unknown Object (File)
Thu, Oct 1, 8:22 PM
View All Files
Subscribers
emaste
imp

Details

Reviewers
ngie
imp
emaste
Commits
rGb9385720f34b: libc: Add unit tests for N2630 and possible collateral damage.

Diff Detail

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

Event Timeline

des created this revision.Aug 19 2023, 11:47 PM
Herald added a reviewer: ngie. · View Herald TranscriptAug 19 2023, 11:47 PM
Herald added a subscriber: imp. · View Herald Transcript
des requested review of this revision.Aug 19 2023, 11:47 PM
Harbormaster completed remote builds in B53177: Diff 126242.Aug 19 2023, 11:47 PM
des added a parent revision: D41511: libc: Implement N2630..Aug 19 2023, 11:47 PM
emaste added a subscriber: emaste.Aug 19 2023, 11:59 PM
imp accepted this revision.Aug 21 2023, 5:17 PM

Only issue I see, and it might not even be an issue, is assuming sizeof(int) == 4.

lib/libc/tests/stdio/snprintf_test.c
47

these assume 32-bit int. Is that sane wrt CHERI?

This revision is now accepted and ready to land.Aug 21 2023, 5:17 PM
emaste accepted this revision.Aug 21 2023, 5:27 PM

Tests lgtm, will leave it to @ngie to comment on the netbsd-tests changes/approach.

des updated this revision to Diff 126331.Aug 21 2023, 9:23 PM

A few more cases.

This revision now requires review to proceed.Aug 21 2023, 9:23 PM
Harbormaster completed remote builds in B53213: Diff 126331.Aug 21 2023, 9:23 PM
des updated this revision to Diff 126552.Aug 26 2023, 5:09 PM

Improve sscanf() tests.

Harbormaster completed remote builds in B53301: Diff 126552.Aug 26 2023, 5:09 PM
des updated this revision to Diff 126557.Aug 26 2023, 6:29 PM

Fully test sscanf() and swscanf() changes.

Harbormaster completed remote builds in B53303: Diff 126557.Aug 26 2023, 6:29 PM
imp accepted this revision.Aug 28 2023, 3:28 PM

These look good to me, though I did not verify the tables were right

This revision is now accepted and ready to land.Aug 28 2023, 3:28 PM
Closed by commit rGb9385720f34b: libc: Add unit tests for N2630 and possible collateral damage. (authored by des). · Explain WhyAug 28 2023, 3:39 PM
This revision was automatically updated to reflect the committed changes.
des added a commit: rGb9385720f34b: libc: Add unit tests for N2630 and possible collateral damage..
des mentioned this in rGbae932999e0f: libc: Implement N2630..Sep 7 2023, 8:29 PM

Revision Contents
Changeset List

PathSize
contrib/
netbsd-tests/
lib/
libc/
stdlib/
6 lines
lib/
libc/
tests/
stdio/
3 lines
138 lines
249 lines
250 lines

Diff 126595

View Options

contrib/netbsd-tests/lib/libc/stdlib/t_strtol.c

Loading...
View Options

lib/libc/tests/stdio/Makefile

Loading...
View Options

lib/libc/tests/stdio/snprintf_test.c

Loading...
View Options

lib/libc/tests/stdio/sscanf_test.c

Loading...
View Options

lib/libc/tests/stdio/swscanf_test.c

Loading...