[PATCH] D27726: [analyzer] Refer to macro names in diagnostics for macros representing a literal
This revision was automatically updated to reflect the committed changes. Closed by commit rL289884: [analyzer] Refer to macro names in diagnostics for macros representing a literal (authored by zaks). Changed prior to commit: https://reviews.llvm.org/D27726?vs=81277=81669#toc Repository: rL LLVM https://reviews.llvm.org/D27726 Files: cfe/trunk/include/clang/StaticAnalyzer/Core/BugReporter/BugReporterVisitor.h cfe/trunk/lib/StaticAnalyzer/Core/BugReporterVisitors.cpp cfe/trunk/test/Analysis/Inputs/system-header-simulator-objc.h cfe/trunk/test/Analysis/Inputs/system-header-simulator.h cfe/trunk/test/Analysis/diagnostics/macros.cpp cfe/trunk/test/Analysis/diagnostics/macros.m Index: cfe/trunk/include/clang/StaticAnalyzer/Core/BugReporter/BugReporterVisitor.h === --- cfe/trunk/include/clang/StaticAnalyzer/Core/BugReporter/BugReporterVisitor.h +++ cfe/trunk/include/clang/StaticAnalyzer/Core/BugReporter/BugReporterVisitor.h @@ -245,6 +245,7 @@ const ExplodedNode *N); bool patternMatch(const Expr *Ex, +const Expr *ParentEx, raw_ostream , BugReporterContext , BugReport , Index: cfe/trunk/test/Analysis/Inputs/system-header-simulator.h === --- cfe/trunk/test/Analysis/Inputs/system-header-simulator.h +++ cfe/trunk/test/Analysis/Inputs/system-header-simulator.h @@ -102,3 +102,11 @@ void _exit(int status) __attribute__ ((__noreturn__)); void _Exit(int status) __attribute__ ((__noreturn__)); +#define UINT32_MAX4294967295U +#define INT64_MIN(-INT64_MAX-1) +#define __DBL_MAX__ 1.7976931348623157e+308 +#define DBL_MAX __DBL_MAX__ +#ifndef NULL +#define __DARWIN_NULL 0 +#define NULL __DARWIN_NULL +#endif \ No newline at end of file Index: cfe/trunk/test/Analysis/Inputs/system-header-simulator-objc.h === --- cfe/trunk/test/Analysis/Inputs/system-header-simulator-objc.h +++ cfe/trunk/test/Analysis/Inputs/system-header-simulator-objc.h @@ -17,7 +17,11 @@ typedef unsigned short unichar; typedef UInt16 UniChar; -#define NULL ((void *)0) +#ifndef NULL +#define __DARWIN_NULL ((void *)0) +#define NULL __DARWIN_NULL +#endif + #define nil ((id)0) enum { @@ -54,6 +58,7 @@ - (oneway void)release; - (id)autorelease; - (id)init; +@property (readonly, copy) NSString *description; @end @protocol NSCopying - (id)copyWithZone:(NSZone *)zone; @end @protocol NSMutableCopying - (id)mutableCopyWithZone:(NSZone *)zone; @end @protocol NSCoding - (void)encodeWithCoder:(NSCoder *)aCoder; Index: cfe/trunk/test/Analysis/diagnostics/macros.cpp === --- cfe/trunk/test/Analysis/diagnostics/macros.cpp +++ cfe/trunk/test/Analysis/diagnostics/macros.cpp @@ -0,0 +1,48 @@ +// RUN: %clang_cc1 -analyze -std=c++11 -analyzer-checker=core,osx -analyzer-output=text -verify %s + +#include "../Inputs/system-header-simulator.h" +#include "../Inputs/system-header-simulator-cxx.h" + +void testIntMacro(unsigned int i) { + if (i == UINT32_MAX) { // expected-note {{Assuming 'i' is equal to UINT32_MAX}} + // expected-note@-1 {{Taking true branch}} +char *p = NULL; // expected-note {{'p' initialized to a null pointer value}} +*p = 7; // expected-warning {{Dereference of null pointer (loaded from variable 'p')}} + // expected-note@-1 {{Dereference of null pointer (loaded from variable 'p')}} + } +} + +void testNULLMacro(int *p) { + if (p == NULL) { // expected-note {{Assuming 'p' is equal to NULL}} + // expected-note@-1 {{Taking true branch}} +*p = 7; // expected-warning {{Dereference of null pointer (loaded from variable 'p')}} + // expected-note@-1 {{Dereference of null pointer (loaded from variable 'p')}} + } +} + +void testnullptrMacro(int *p) { + if (p == nullptr) { // expected-note {{Assuming pointer value is null}} + // expected-note@-1 {{Taking true branch}} +*p = 7; // expected-warning {{Dereference of null pointer (loaded from variable 'p')}} + // expected-note@-1 {{Dereference of null pointer (loaded from variable 'p')}} + } +} + +// There are no path notes on the comparison to float types. +void testDoubleMacro(double d) { + if (d == DBL_MAX) { // expected-note {{Taking true branch}} + +char *p = NULL; // expected-note {{'p' initialized to a null pointer value}} +*p = 7; // expected-warning {{Dereference of null pointer (loaded from variable 'p')}} +// expected-note@-1 {{Dereference of null pointer (loaded from variable 'p')}} + } +} + +void testboolMacro(bool b, int *p) { + p = nullptr; // expected-note {{Null pointer value stored to 'p'}} + if (b ==
[PATCH] D27726: [analyzer] Refer to macro names in diagnostics for macros representing a literal
dcoughlin accepted this revision. dcoughlin added a comment. This revision is now accepted and ready to land. The diagnostics look great to me and the macro logic seems reasonable. Comment at: test/Analysis/diagnostics/macros.cpp:49 +} \ No newline at end of file Missing newline? https://reviews.llvm.org/D27726 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D27726: [analyzer] Refer to macro names in diagnostics for macros representing a literal
zaks.anna created this revision. zaks.anna added reviewers: dergachev.a, dcoughlin. zaks.anna added a subscriber: cfe-commits. When a macro expending to a literal is used in a comparison, use the macro name in the diagnostic rather than the literal. This improves readability of path notes. Added tests for various macro literals that could occur. Only BOOl, Int, and NULL tests have changed behavior with this patch. https://reviews.llvm.org/D27726 Files: include/clang/StaticAnalyzer/Core/BugReporter/BugReporterVisitor.h lib/StaticAnalyzer/Core/BugReporterVisitors.cpp test/Analysis/Inputs/system-header-simulator-objc.h test/Analysis/Inputs/system-header-simulator.h test/Analysis/diagnostics/macros.cpp test/Analysis/diagnostics/macros.m Index: test/Analysis/diagnostics/macros.m === --- /dev/null +++ test/Analysis/diagnostics/macros.m @@ -0,0 +1,31 @@ +// RUN: %clang_cc1 -analyze -analyzer-checker=core,osx -fblocks -analyzer-output=text -verify %s + +#include "../Inputs/system-header-simulator-objc.h" + +@interface NSDictionary : NSObject +- (NSUInteger)count; +- (id)objectForKey:(id)aKey; +- (NSEnumerator *)keyEnumerator; +@end +@interface NSMutableDictionary : NSDictionary +- (void)setObject:(id)anObject forKey:(id )aKey; +@end + +void testBOOLMacro(BOOL b) { + if (b == YES) { // expected-note {{Assuming 'b' is equal to YES}} + // expected-note@-1 {{Taking true branch}} +char *p = NULL;// expected-note {{'p' initialized to a null pointer value}} +*p = 7; // expected-warning {{Dereference of null pointer (loaded from variable 'p')}} + // expected-note@-1 {{Dereference of null pointer (loaded from variable 'p')}} + } +} + +void testNilMacro(NSMutableDictionary *d, NSObject *o) { + if (o == nil) // expected-note {{Assuming 'o' is equal to nil}} +// expected-note@-1 {{Taking true branch}} +[d setObject:o forKey:[o description]]; // expected-warning {{Key argument to 'setObject:forKey:' cannot be nil}} +// expected-note@-1 {{'description' not called because the receiver is nil}} +// expected-note@-2 {{Key argument to 'setObject:forKey:' cannot be nil}} + + return; +} Index: test/Analysis/diagnostics/macros.cpp === --- /dev/null +++ test/Analysis/diagnostics/macros.cpp @@ -0,0 +1,48 @@ +// RUN: %clang_cc1 -analyze -std=c++11 -analyzer-checker=core,osx -analyzer-output=text -verify %s + +#include "../Inputs/system-header-simulator.h" +#include "../Inputs/system-header-simulator-cxx.h" + +void testIntMacro(unsigned int i) { + if (i == UINT32_MAX) { // expected-note {{Assuming 'i' is equal to UINT32_MAX}} + // expected-note@-1 {{Taking true branch}} +char *p = NULL; // expected-note {{'p' initialized to a null pointer value}} +*p = 7; // expected-warning {{Dereference of null pointer (loaded from variable 'p')}} + // expected-note@-1 {{Dereference of null pointer (loaded from variable 'p')}} + } +} + +void testNULLMacro(int *p) { + if (p == NULL) { // expected-note {{Assuming 'p' is equal to NULL}} + // expected-note@-1 {{Taking true branch}} +*p = 7; // expected-warning {{Dereference of null pointer (loaded from variable 'p')}} + // expected-note@-1 {{Dereference of null pointer (loaded from variable 'p')}} + } +} + +void testnullptrMacro(int *p) { + if (p == nullptr) { // expected-note {{Assuming pointer value is null}} + // expected-note@-1 {{Taking true branch}} +*p = 7; // expected-warning {{Dereference of null pointer (loaded from variable 'p')}} + // expected-note@-1 {{Dereference of null pointer (loaded from variable 'p')}} + } +} + +// There are no path notes on the comparison to float types. +void testDoubleMacro(double d) { + if (d == DBL_MAX) { // expected-note {{Taking true branch}} + +char *p = NULL; // expected-note {{'p' initialized to a null pointer value}} +*p = 7; // expected-warning {{Dereference of null pointer (loaded from variable 'p')}} +// expected-note@-1 {{Dereference of null pointer (loaded from variable 'p')}} + } +} + +void testboolMacro(bool b, int *p) { + p = nullptr; // expected-note {{Null pointer value stored to 'p'}} + if (b == false) { // expected-note {{Assuming the condition is true}} +// expected-note@-1 {{Taking true branch}} +*p = 7; // expected-warning {{Dereference of null pointer (loaded from variable 'p')}} +// expected-note@-1 {{Dereference of null pointer (loaded from variable 'p')}} + } +} \ No newline at end of file Index: test/Analysis/Inputs/system-header-simulator.h === ---