Page MenuHomeFreeBSD
Differential D45917

kdb_sysctl_trap: suppress gcc -Warray-bounds
ClosedPublic
Actions

Authored by rlibby on Jul 8 2024, 3:58 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Oct 5, 10:48 PM
Unknown Object (File)
Mon, Oct 5, 8:56 AM
Unknown Object (File)
Mon, Oct 5, 8:40 AM
Unknown Object (File)
Sun, Oct 4, 1:43 PM
Unknown Object (File)
Sat, Oct 3, 9:01 PM
Unknown Object (File)
Sat, Oct 3, 8:59 PM
Unknown Object (File)
Sat, Oct 3, 10:48 AM
Unknown Object (File)
Fri, Sep 18, 4:14 AM
View All Files
Subscribers
imp
olce

Details

Reviewers
kib
Commits
rGff1c63671b73: kdb_sysctl_trap: suppress gcc -Warray-bounds
Summary

gcc diagnosed a dereference of 0x10 with -Warray-bounds, which is
entirely sensible, except that this is a deliberate trap. Throw gcc off
with a volatile pointer.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 58566
Build 55454: arc lint + arc unit

Event Timeline

rlibby created this revision.Jul 8 2024, 3:58 PM
Herald added subscribers: olce, imp. · View Herald TranscriptJul 8 2024, 3:58 PM
rlibby requested review of this revision.Jul 8 2024, 3:58 PM
Harbormaster completed remote builds in B58566: Diff 140692.Jul 8 2024, 3:58 PM
rlibby added a parent revision: D45916: kern_exec.c: quiet gcc -Wmaybe-uninitialized.Jul 8 2024, 3:58 PM
rlibby added a child revision: D45918: ddb: make db_error reliably no-return.
kib accepted this revision.Jul 8 2024, 6:47 PM
kib added inline comments.
sys/kern/subr_kdb.c
241

I believe this is more style-ish

This revision is now accepted and ready to land.Jul 8 2024, 6:47 PM
rlibby added inline comments.Jul 8 2024, 7:56 PM
sys/kern/subr_kdb.c
241

We seem to have a mix of both for both *volatile and *const. I'll take your suggestion.

rlibby updated this revision to Diff 140705.Jul 8 2024, 8:34 PM

kib: style fixup

This revision now requires review to proceed.Jul 8 2024, 8:34 PM
Harbormaster completed remote builds in B58575: Diff 140705.Jul 8 2024, 8:34 PM
This revision was not accepted when it landed; it landed in state Needs Review.Jul 9 2024, 6:46 PM
Closed by commit rGff1c63671b73: kdb_sysctl_trap: suppress gcc -Warray-bounds (authored by rlibby). · Explain Why
This revision was automatically updated to reflect the committed changes.
rlibby added a commit: rGff1c63671b73: kdb_sysctl_trap: suppress gcc -Warray-bounds.

Revision Contents
Changeset List

PathSize
sys/
kern/
2 lines

Diff 140692

View Options

sys/kern/subr_kdb.c

Loading...