]> git.proxmox.com Git - mirror_edk2.git/commit
MdePkg/DebugLib: Flag post-_ASSERT() as unreachable for analyzers.
authorMarvin H?user <Marvin.Haeuser@outlook.com>
Sun, 19 Jun 2016 01:29:39 +0000 (09:29 +0800)
committerLiming Gao <liming.gao@intel.com>
Wed, 22 Jun 2016 01:37:12 +0000 (09:37 +0800)
commit90eda6fc279744c3ab200c149e4b75ed0c5dabf0
tree3b59a235a4b160a085ca2a0ec3c17f15d32e78dc
parent001f28caefbb2b5b05435adf36fc4e695b7b0f66
MdePkg/DebugLib: Flag post-_ASSERT() as unreachable for analyzers.

When a variable is compared to NULL as part of an ASSERT() call, the
code implies that the variable may be NULL. Certain Static Analyzers,
such as the Clang Static Analyzer, then issue false warnings for any
untested access of that pointer, even though that defeats the
intention behind ASSERT(). This patch flags the position after
_ASSERT() as unreachable for Static Analyzers to indicate that if the
variable is NULL, execution cannot pass.
As the ANALYZER-prefixed version is used, the executables will not
have any code stripped during compilation.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Marvin Haeuser <Marvin.Haeuser@outlook.com>
Reviewed-by: Michael Kinney <michael.d.kinney@intel.com>
Reviewed-by: Liming Gao <liming.gao@intel.com>
MdePkg/Include/Library/DebugLib.h