MachOImageSymbolTableReader: permit indirect symbols to exist among external defined symbols

snapshot/mac/mach_o_image_reader_test.cc

Index: snapshot/mac/mach_o_image_reader_test.cc
diff --git a/snapshot/mac/mach_o_image_reader_test.cc b/snapshot/mac/mach_o_image_reader_test.cc
index 23510df4add248e1f89b010b58b9bf1233e56d12..56650140a2a48db6b54246af05bf437f396fe2ba 100644
--- a/snapshot/mac/mach_o_image_reader_test.cc
+++ b/snapshot/mac/mach_o_image_reader_test.cc
@@ -393,14 +393,8 @@ void ExpectSymbol(const Nlist* entry,
 
   uint32_t entry_type = entry->n_type & N_TYPE;
   if ((entry->n_type & N_STAB) == 0 && (entry->n_type & N_PEXT) == 0 &&
-      entry_type != N_UNDF && entry_type != N_PBUD &&
+      (entry_type == N_ABS || entry_type == N_SECT) &&
       (entry->n_type & N_EXT) == 1) {
-    // Note that this catches more symbols than MachOImageSymbolTableReader
-    // does. This test looks for all external defined symbols, but the
-    // implementation excludes indirect (N_INDR) symbols. This is intentional,
-    // because indirect symbols are currently not seen in the wild, but if they
-    // begin to be used more widely, this test is expected to catch them so that
-    // a decision can be made regarding whether support ought to be implemented.
     mach_vm_address_t actual_address;
     ASSERT_TRUE(
         actual_image->LookUpExternalDefinedSymbol(name, &actual_address));