diff options
author | DonĂ¡t Nagy <donat.nagy@ericsson.com> | 2025-09-08 13:16:24 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2025-09-08 13:16:24 +0200 |
commit | a80c393a9c498279a1ec9fd630535b9ff139b49f (patch) | |
tree | 88617bd2d3f9801efb09468fde72eb0f3b3d25e7 /llvm/test/DebugInfo/Generic/assignment-tracking | |
parent | 408a2e7cee3f4bcdcbc4d3dbb13fea3ecd5788d4 (diff) | |
download | llvm-main.zip llvm-main.tar.gz llvm-main.tar.bz2 |
Previously the analyzer had an undocumented top-level checker group
called `valist` which offered several checkers to detect use of
uninitialized `va_list` objects and leaks of `va_list`s.
As the responsibilities of these checkers were messily intertwined and
`va_list` is a rarely used language feature, this commit simplifies the
situation by consolidating these checkers into a single checker which
will be called `security.VAList`.
Note that I'm choosing the capitalization `VAList` to be consistent with
the example of the AST node type `VAArgExpr`. I updated many variable
names to ensure that `ValistChecker.cpp` uses this spelling everywhere
(in CamelCased names). I'm planning to rename `ValistChecker.cpp` to
`VAListChecker.cpp` in a follow-up commit.
This commit also adds documentation for this checker in checkers.rst.
Among the test files I preserved the existing separation but I
eliminated some duplicated cases now that there is no way to separately
enable the old sub-checkers.
For the background of this change see also the discourse thread
https://discourse.llvm.org/t/clean-up-valist-checkers/85277/3
Diffstat (limited to 'llvm/test/DebugInfo/Generic/assignment-tracking')
0 files changed, 0 insertions, 0 deletions