[PATCH] D64671: [clang-tidy] New check: misc-init-local-variables
Jussi Pakkanen via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Jul 30 05:34:39 PDT 2019
jpakkane marked 2 inline comments as done.
jpakkane added a comment.
In D64671#1603427 <https://reviews.llvm.org/D64671#1603427>, @alexfh wrote:
> A general comment: "misc" is a sort of a heap of checks that otherwise don't have a good home. This one would probably better go to bugprone (or maybe there's a relevant CERT or C++ Core Guidelines rule?).
The closest I could find is ES.20 "Always initialize an object": https://github.com/isocpp/CppCoreGuidelines/blob/master/CppCoreGuidelines.md#es20-always-initialize-an-object
Most other guidelines seem to be formulated as "don't read uninitialized memory" such as: https://cwe.mitre.org/data/definitions/457.html
So a potential name for this could be "cppcoreguidelines-init-variables". However that rule is about initializing all variables, even those in structs and classes. This is about local variables.
Related to this, there seems to be a talk at CppCon about doing something similar to this MR: https://cppcon2019.sched.com/event/SfYc/killing-uninitialized-memory-prot
CHANGES SINCE LAST ACTION
More information about the cfe-commits