diff options
author | Daniel Dunbar <daniel@zuster.org> | 2009-04-20 06:13:16 +0000 |
---|---|---|
committer | Daniel Dunbar <daniel@zuster.org> | 2009-04-20 06:13:16 +0000 |
commit | 69a79b1a60bb91e5fc4d867a033289e0b0231e9c (patch) | |
tree | 650c71142aa001a66cf700d33ee8435dafe7e154 /clang/lib/Basic/Diagnostic.cpp | |
parent | 428d4903326faec482ecbf64200ed62f47a53f48 (diff) | |
download | bcm5719-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.cpp | 13 |
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('\''); |