[Lldb-commits] [lldb] r164844 - /lldb/trunk/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCTypeVendor.cpp

Sean Callanan scallanan at apple.com
Fri Sep 28 13:47:48 PDT 2012


Author: spyffe
Date: Fri Sep 28 15:47:47 2012
New Revision: 164844

URL: http://llvm.org/viewvc/llvm-project?rev=164844&view=rev
Log:
Fixed a bug where if something went wrong while
constructing the ObjCInterfaceDecl for an ISA,
we'd continue and try to use that Decl anyway,
possibly causing a crash.

Modified:
    lldb/trunk/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCTypeVendor.cpp

Modified: lldb/trunk/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCTypeVendor.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCTypeVendor.cpp?rev=164844&r1=164843&r2=164844&view=diff
==============================================================================
--- lldb/trunk/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCTypeVendor.cpp (original)
+++ lldb/trunk/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCTypeVendor.cpp Fri Sep 28 15:47:47 2012
@@ -640,6 +640,8 @@
                 log->Printf("AOCTV::FT [%u] Couldn't get the Objective-C interface for isa 0x%llx",
                             current_id,
                             (uint64_t)isa);
+            
+            break;
         }
         
         clang::QualType new_iface_type = ast_ctx->getObjCInterfaceType(iface_decl);
@@ -651,6 +653,8 @@
                         current_id,
                         dumper.GetCString(),
                         (uint64_t)isa);
+            
+            break;
         }
         
         types.push_back(ClangASTType(ast_ctx, new_iface_type.getAsOpaquePtr()));





More information about the lldb-commits mailing list