sys: add __null_terminated and bound the hash.h helpers
Needs RevisionPublic
Actions

Authored by abhijeetsharma2002_gmail.com on Tue, Nov 10, 6:47 PM.

Project Tags

None

Referenced Files

	F177448092: D61401.1794425739.diff
	Tue, Nov 10, 7:35 PM

	F177448036: D61401.1794425660.diff
	Tue, Nov 10, 7:34 PM

Subscribers

imp

Details

Reviewers

rpaulo
emaste
imp

Summary

Add __null_terminated and its forge macro to cdefs.h, size hash32_buf()
by len and walk the string hashes as NUL-terminated. No change without
the flag.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 78953
Build 75836: arc lint + arc unit

Event Timeline

abhijeetsharma2002_gmail.com created this revision.Tue, Nov 10, 6:47 PM

Herald added a subscriber: imp. · View Herald TranscriptTue, Nov 10, 6:47 PM

abhijeetsharma2002_gmail.com requested review of this revision.Tue, Nov 10, 6:47 PM

Harbormaster completed remote builds in B78953: Diff 192148.Tue, Nov 10, 6:47 PM

abhijeetsharma2002_gmail.com added a parent revision: D60966: tests: add a kyua test for the bounds-safety soft traps.Tue, Nov 10, 6:48 PM

abhijeetsharma2002_gmail.com added a child revision: D61402: tcp: build tcp_fastopen.c under bounds-safety.

I'm not sure tagging every string in the kernel with __null_terminated is viable. Can you explain this a little more?

sys/sys/cdefs.h
399	Can you document both of these in cdefs(9)?

This revision now requires changes to proceed.Tue, Nov 10, 8:17 PM

Revision Contents
Changeset List

Path

Size

sys/

cdefs.h

6 lines

hash.h

21 lines

Diff 192148

View Options

sys/sys/cdefs.h