Page MenuHomeFreeBSD
Differential D58439

bsd.clang-analyze.mk: silence Annex K (DeprecatedOrUnsafeBufferHandling) by default
Needs ReviewPublic
Actions

Authored by siva on Fri, Jul 24, 5:49 PM.
Tags
None
Referenced Files
F164533484: D58439.diff
Sat, Aug 1, 9:43 PM
Unknown Object (File)
Fri, Jul 31, 2:37 PM
Unknown Object (File)
Fri, Jul 31, 12:28 PM
Unknown Object (File)
Fri, Jul 31, 10:59 AM
Unknown Object (File)
Thu, Jul 30, 4:18 PM
Unknown Object (File)
Wed, Jul 29, 10:58 AM
Unknown Object (File)
Wed, Jul 29, 2:31 AM
Unknown Object (File)
Tue, Jul 28, 7:09 PM
View All 12 Files
Subscribers
bdrewery
emaste
imp
markj

Details

Reviewers
None
Group Reviewers
srcmgr
Summary

This introduces a new make variable CLANG_ANALYZE_SILENCED_CHECKERS that
silences a given list of checkers (from https://clang.llvm.org/docs/analyzer/checkers.html).

Diff Detail

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

Event Timeline

siva created this revision.Fri, Jul 24, 5:49 PM
Herald added subscribers: emaste, imp, bdrewery. · View Herald TranscriptFri, Jul 24, 5:49 PM
siva requested review of this revision.Fri, Jul 24, 5:49 PM
Harbormaster completed remote builds in B75099: Diff 182623.Fri, Jul 24, 5:49 PM
markj added a subscriber: markj.Fri, Jul 24, 8:01 PM
markj added inline comments.
share/mk/bsd.clang-analyze.mk
47

I think some comment explaining why we suppress this would be worthwhile.

Also this feels like too large a hammer. This flag also suppresses warnings about use of sprintf()... but that's quite different from fprintf(). That's really silly. :(

And to top it off, we don't even implement fprintf_s() or sprintf_s() as far as I can see.

Revision Contents
Changeset List

PathSize
share/
mk/
9 lines

Diff 182623

View Options

share/mk/bsd.clang-analyze.mk

Loading...