summaryrefslogtreecommitdiffstats
path: root/clang/lib/Basic/Diagnostic.cpp
diff options
context:
space:
mode:
authorDaniel Dunbar <daniel@zuster.org>2009-04-20 06:13:16 +0000
committerDaniel Dunbar <daniel@zuster.org>2009-04-20 06:13:16 +0000
commit69a79b1a60bb91e5fc4d867a033289e0b0231e9c (patch)
tree650c71142aa001a66cf700d33ee8435dafe7e154 /clang/lib/Basic/Diagnostic.cpp
parent428d4903326faec482ecbf64200ed62f47a53f48 (diff)
downloadbcm5719-llvm-69a79b1a60bb91e5fc4d867a033289e0b0231e9c.tar.gz
bcm5719-llvm-69a79b1a60bb91e5fc4d867a033289e0b0231e9c.zip
Don't crash in the diagnostic printer if we happen to get passed a
null string / identifier. llvm-svn: 69575
Diffstat (limited to 'clang/lib/Basic/Diagnostic.cpp')
-rw-r--r--clang/lib/Basic/Diagnostic.cpp13
1 files changed, 13 insertions, 0 deletions
diff --git a/clang/lib/Basic/Diagnostic.cpp b/clang/lib/Basic/Diagnostic.cpp
index 9986fc5a41b..d5d8da1e22a 100644
--- a/clang/lib/Basic/Diagnostic.cpp
+++ b/clang/lib/Basic/Diagnostic.cpp
@@ -714,6 +714,11 @@ FormatDiagnostic(llvm::SmallVectorImpl<char> &OutStr) const {
case Diagnostic::ak_c_string: {
const char *S = getArgCStr(ArgNo);
assert(ModifierLen == 0 && "No modifiers for strings yet");
+
+ // Don't crash if get passed a null pointer by accident.
+ if (!S)
+ S = "(null)";
+
OutStr.append(S, S + strlen(S));
break;
}
@@ -757,6 +762,14 @@ FormatDiagnostic(llvm::SmallVectorImpl<char> &OutStr) const {
case Diagnostic::ak_identifierinfo: {
const IdentifierInfo *II = getArgIdentifier(ArgNo);
assert(ModifierLen == 0 && "No modifiers for strings yet");
+
+ // Don't crash if get passed a null pointer by accident.
+ if (!II) {
+ const char *S = "(null)";
+ OutStr.append(S, S + strlen(S));
+ continue;
+ }
+
OutStr.push_back('\'');
OutStr.append(II->getName(), II->getName() + II->getLength());
OutStr.push_back('\'');
OpenPOWER on IntegriCloud