Merge pull request #8036 from augusto2112/next-c338e175a

[lldb] Add nullptr checks in all usages of ThreadSafeReflectionContext

Author: augusto2112

lldb/source/Plugins/LanguageRuntime/Swift/SwiftLanguageRuntimeDynamicTypeResolution.cpp

@@ -237,6 +237,8 @@ SwiftLanguageRuntimeImpl::GetMemoryReader() {
         m_process, [&](swift::remote::RemoteAbsolutePointer pointer) {
           ThreadSafeReflectionContext reflection_context =
               GetReflectionContext();
+          if (!reflection_context)
+            return pointer;
           return reflection_context->StripSignedPointer(pointer);
         }));
   }
@@ -653,6 +655,9 @@ SwiftLanguageRuntimeImpl::GetNumChildren(CompilerType type,
       return {};
 
     ThreadSafeReflectionContext reflection_ctx = GetReflectionContext();
+    if (!reflection_ctx)
+      return {};
+
     LLDBTypeInfoProvider tip(*this, *ts);
     auto *cti = reflection_ctx->GetClassInstanceTypeInfo(
         tr, &tip, ts->GetDescriptorFinder());
@@ -728,6 +733,9 @@ SwiftLanguageRuntimeImpl::GetNumFields(CompilerType type,
       return 0;
     case ReferenceKind::Strong:
       ThreadSafeReflectionContext reflection_ctx = GetReflectionContext();
+      if (!reflection_ctx)
+        return {};
+
       LLDBTypeInfoProvider tip(*this, *ts);
       auto *cti = reflection_ctx->GetClassInstanceTypeInfo(
           tr, &tip, ts->GetDescriptorFinder());
@@ -861,6 +869,9 @@ SwiftLanguageRuntimeImpl::GetIndexOfChildMemberWithName(
                                            child_indexes);
     case ReferenceKind::Strong: {
       ThreadSafeReflectionContext reflection_ctx = GetReflectionContext();
+      if (!reflection_ctx)
+        return {false, {}};
+
       LLDBTypeInfoProvider tip(*this, *ts);
       // `current_tr` iterates the class hierarchy, from the current class, each
       // superclass, and ends on null.
@@ -1721,6 +1732,9 @@ bool SwiftLanguageRuntimeImpl::GetDynamicTypeAndAddress_Protocol(
   swift::remote::RemoteAddress remote_existential(existential_address);
 
   ThreadSafeReflectionContext reflection_ctx = GetReflectionContext();
+  if (!reflection_ctx)
+    return false;
+
   auto pair = reflection_ctx->ProjectExistentialAndUnwrapClass(
       remote_existential, *protocol_typeref,
       tss->GetTypeSystemSwiftTypeRef().GetDescriptorFinder());
@@ -2184,22 +2198,23 @@ SwiftLanguageRuntimeImpl::GetValueType(ValueObject &in_value,
       // Read the value witness table and check if the data is inlined in
       // the existential container or not.
       swift::remote::RemoteAddress remote_existential(existential_address);
-      ThreadSafeReflectionContext reflection_ctx = GetReflectionContext();
-      llvm::Optional<bool> is_inlined =
-          reflection_ctx->IsValueInlinedInExistentialContainer(
-              remote_existential);
-
-      if (use_local_buffer)
-        PopLocalBuffer();
-
-      // An error has occurred when trying to read value witness table,
-      // default to treating it as pointer.
-      if (!is_inlined.has_value())
-        return Value::ValueType::LoadAddress;
-
-      // Inlined data, same as static data.
-      if (*is_inlined)
-        return static_value_type;
+      if (ThreadSafeReflectionContext reflection_ctx = GetReflectionContext()) {
+        llvm::Optional<bool> is_inlined =
+            reflection_ctx->IsValueInlinedInExistentialContainer(
+                remote_existential);
+
+        if (use_local_buffer)
+          PopLocalBuffer();
+
+        // An error has occurred when trying to read value witness table,
+        // default to treating it as pointer.
+        if (!is_inlined.has_value())
+          return Value::ValueType::LoadAddress;
+
+        // Inlined data, same as static data.
+        if (*is_inlined)
+          return static_value_type;
+      }
 
       // If the data is not inlined, we have a pointer.
       return Value::ValueType::LoadAddress;