diff options
| author | Benjamin Kramer <benny.kra@googlemail.com> | 2009-11-14 12:08:24 +0000 |
|---|---|---|
| committer | Benjamin Kramer <benny.kra@googlemail.com> | 2009-11-14 12:08:24 +0000 |
| commit | f4c511b026a8c9d11392c5ce05ff0a6de28d8172 (patch) | |
| tree | 73f8290ea8de557db9c0c4f5f83c08128af18340 /clang/lib/Analysis/VLASizeChecker.cpp | |
| parent | ea956ddf0b664de624b5ca046f5d9daf87ada727 (diff) | |
| download | bcm5719-llvm-f4c511b026a8c9d11392c5ce05ff0a6de28d8172.tar.gz bcm5719-llvm-f4c511b026a8c9d11392c5ce05ff0a6de28d8172.zip | |
Change *BugReport constructors to take StringRefs.
- Eliminates many calls to std::string.c_str()
- Fixes an invalid read in ReturnStackAddressChecker due to an unsafe call to
StringRef.data() which doesn't guarantee null-termination.
llvm-svn: 88779
Diffstat (limited to 'clang/lib/Analysis/VLASizeChecker.cpp')
| -rw-r--r-- | clang/lib/Analysis/VLASizeChecker.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/clang/lib/Analysis/VLASizeChecker.cpp b/clang/lib/Analysis/VLASizeChecker.cpp index 5cb700ed67b..799a73e293c 100644 --- a/clang/lib/Analysis/VLASizeChecker.cpp +++ b/clang/lib/Analysis/VLASizeChecker.cpp @@ -64,7 +64,7 @@ void VLASizeChecker::PreVisitDeclStmt(CheckerContext &C, const DeclStmt *DS) { "garbage value as its size"); EnhancedBugReport *report = - new EnhancedBugReport(*BT_undef, BT_undef->getName().c_str(), N); + new EnhancedBugReport(*BT_undef, BT_undef->getName(), N); report->addRange(SE->getSourceRange()); report->addVisitorCreator(bugreporter::registerTrackNullOrUndefValue, SE); C.EmitReport(report); @@ -84,7 +84,7 @@ void VLASizeChecker::PreVisitDeclStmt(CheckerContext &C, const DeclStmt *DS) { "size"); EnhancedBugReport *report = - new EnhancedBugReport(*BT_zero, BT_zero->getName().c_str(), N); + new EnhancedBugReport(*BT_zero, BT_zero->getName(), N); report->addRange(SE->getSourceRange()); report->addVisitorCreator(bugreporter::registerTrackNullOrUndefValue, SE); C.EmitReport(report); |
