[PATCH] D91000: [clang-tidy] Add cert-msc24-msc33-c checker.

Fütő Gergely via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 11 03:03:12 PST 2022


futogergely updated this revision to Diff 407819.
futogergely marked an inline comment as done.
futogergely retitled this revision from "[clang-tidy] Add cert-msc24-c checker." to "[clang-tidy] Add cert-msc24-msc33-c checker.".
futogergely edited the summary of this revision.
futogergely added a comment.

I changed the class name: ObsolescentFunctionsCheck->UnsafeFunctionsCheck.
Since MSC33-C is also included, I changed the checker name to cert-msc24-msc33-c.
I added the following functions from CheckSecuritySyntaxOnly under option 'ReportMoreUnsafeFunctions': bcmp, bcopy, bzero, getpw, vfork. Since there is a replacement suggested there, I added the replacement suggestions also.
I did not add tmpnam, tmpfile, mktemp, mkstemp, rand..() to the checker, because there are separate CERT rules for these.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D91000/new/

https://reviews.llvm.org/D91000

Files:
  clang-tools-extra/clang-tidy/cert/CERTTidyModule.cpp
  clang-tools-extra/clang-tidy/cert/CMakeLists.txt
  clang-tools-extra/clang-tidy/cert/UnsafeFunctionsCheck.cpp
  clang-tools-extra/clang-tidy/cert/UnsafeFunctionsCheck.h
  clang-tools-extra/docs/ReleaseNotes.rst
  clang-tools-extra/docs/clang-tidy/checks/cert-msc24-msc33-c.rst
  clang-tools-extra/docs/clang-tidy/checks/list.rst
  clang-tools-extra/test/clang-tidy/checkers/cert-msc24-msc33-c.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D91000.407819.patch
Type: text/x-patch
Size: 29391 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20220211/aec5b4de/attachment-0001.bin>


More information about the cfe-commits mailing list