Index: lib/StaticAnalyzer/Checkers/RetainCountChecker.cpp =================================================================== --- lib/StaticAnalyzer/Checkers/RetainCountChecker.cpp +++ lib/StaticAnalyzer/Checkers/RetainCountChecker.cpp @@ -1929,10 +1929,12 @@ isa(E); } -static std::string describeRegion(const MemRegion *MR) { +static Optional describeRegion(const MemRegion *MR) { + if (const auto *VR = dyn_cast_or_null(MR)) + return std::string(cast(MR)->getDecl()->getName()); // Once we support more storage locations for bindings, // this would need to be improved. - return cast(MR)->getDecl()->getName(); + return None; } /// Returns true if this stack frame is for an Objective-C method that is a @@ -2399,9 +2401,9 @@ os << "Object leaked: "; - if (FirstBinding) { - os << "object allocated and stored into '" - << describeRegion(FirstBinding) << '\''; + Optional RegionDescription = describeRegion(FirstBinding); + if (RegionDescription) { + os << "object allocated and stored into '" << *RegionDescription << '\''; } else os << "allocated object"; @@ -2519,7 +2521,8 @@ UniqueingDecl = AllocNode->getLocationContext()->getDecl(); } -void CFRefLeakReport::createDescription(CheckerContext &Ctx, bool GCEnabled, bool IncludeAllocationLine) { +void CFRefLeakReport::createDescription(CheckerContext &Ctx, bool GCEnabled, + bool IncludeAllocationLine) { assert(Location.isValid() && UniqueingDecl && UniqueingLocation.isValid()); Description.clear(); llvm::raw_string_ostream os(Description); @@ -2528,8 +2531,9 @@ os << "(when using garbage collection) "; os << "of an object"; - if (AllocBinding) { - os << " stored into '" << describeRegion(AllocBinding) << '\''; + Optional RegionDescription = describeRegion(AllocBinding); + if (RegionDescription) { + os << " stored into '" << *RegionDescription << '\''; if (IncludeAllocationLine) { FullSourceLoc SL(AllocStmt->getLocStart(), Ctx.getSourceManager()); os << " (allocated on line " << SL.getSpellingLineNumber() << ")"; Index: test/Analysis/retaincountchecker-compoundregion.m =================================================================== --- test/Analysis/retaincountchecker-compoundregion.m +++ test/Analysis/retaincountchecker-compoundregion.m @@ -0,0 +1,25 @@ +// RUN: %clang_analyze_cc1 -analyzer-checker=core,osx -fblocks -verify -Wno-objc-root-class %s +typedef const void *CFTypeRef; +enum { kCFNumberSInt8Type = 1, kCFNumberSInt16Type = 2, + kCFNumberSInt32Type = 3, kCFNumberSInt64Type = 4, + kCFNumberFloat32Type = 5, kCFNumberFloat64Type = 6, + kCFNumberCharType = 7, kCFNumberShortType = 8, + kCFNumberIntType = 9, kCFNumberLongType = 10, + kCFNumberLongLongType = 11, kCFNumberFloatType = 12, + kCFNumberDoubleType = 13, kCFNumberCFIndexType = 14, + kCFNumberNSIntegerType = 15, kCFNumberCGFloatType = 16, + kCFNumberMaxType = 16 }; +typedef const struct __CFAllocator * CFAllocatorRef; +typedef signed long CFIndex; +typedef CFIndex CFNumberType; +typedef const struct __CFNumber * CFNumberRef; +extern CFNumberRef CFNumberCreate(CFAllocatorRef allocator, CFNumberType theType, const void *valuePtr); + +void foo(CFAllocatorRef allocator) { + int width = 0; + int height = 0; + CFTypeRef* values = (CFTypeRef[]){ + CFNumberCreate(allocator, kCFNumberSInt32Type, &width), //expected-warning-re{{Potential leak of an object{{$}}}} + CFNumberCreate(allocator, kCFNumberSInt32Type, &height), //expected-warning-re{{Potential leak of an object{{$}}}} + }; +}