diff options
| -rw-r--r-- | tools/CIndex/CIndex.cpp | 6 | ||||
| -rw-r--r-- | tools/c-index-test/c-index-test.c | 3 |
2 files changed, 6 insertions, 3 deletions
diff --git a/tools/CIndex/CIndex.cpp b/tools/CIndex/CIndex.cpp index 9389139116..1f9be0d0c7 100644 --- a/tools/CIndex/CIndex.cpp +++ b/tools/CIndex/CIndex.cpp @@ -552,7 +552,8 @@ const char *clang_getDeclSpelling(CXDecl AnonDecl) NamedDecl *ND = static_cast<NamedDecl *>(AnonDecl); if (ObjCMethodDecl *OMD = dyn_cast<ObjCMethodDecl>(ND)) { - return OMD->getSelector().getAsString().c_str(); + // FIXME: Memory leak! We need to move to a CXString or other API. + return ::strdup(OMD->getSelector().getAsString().c_str()); } if (ObjCCategoryImplDecl *CIMP = dyn_cast<ObjCCategoryImplDecl>(ND)) // No, this isn't the same as the code below. getIdentifier() is non-virtual @@ -650,7 +651,8 @@ const char *clang_getCursorSpelling(CXCursor C) ObjCMessageExpr *OME = dyn_cast<ObjCMessageExpr>( static_cast<Stmt *>(C.stmt)); assert(OME && "clang_getCursorLine(): Missing message expr"); - return OME->getSelector().getAsString().c_str(); + // FIXME: Memory leak! We need to move to a CXString or other API. + return ::strdup(OME->getSelector().getAsString().c_str()); } case CXCursor_VarRef: case CXCursor_FunctionRef: diff --git a/tools/c-index-test/c-index-test.c b/tools/c-index-test/c-index-test.c index 82653357b1..cc4080e388 100644 --- a/tools/c-index-test/c-index-test.c +++ b/tools/c-index-test/c-index-test.c @@ -4,6 +4,7 @@ #include <stdlib.h> #include <stdio.h> #include <string.h> +#include <assert.h> #ifdef _MSC_VER char *basename(const char* path) @@ -86,7 +87,7 @@ static void TranslationUnitVisitor(CXTranslationUnit Unit, CXCursor Cursor, printf("// CHECK: %s:%d:%d: ", basename(clang_getCursorSource(Ref)), curLine, curColumn); PrintCursor(Ref); - printf(" [Context:%s]\n", clang_getDeclSpelling(Ref.decl)); + printf(" [Context:%s]\n", clang_getDeclSpelling(Ref.decl)); } startBuf++; } |