]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commitdiff
PR32829, SEGV on objdump function debug_type_samep
authorAlan Modra <amodra@gmail.com>
Tue, 1 Apr 2025 12:06:54 +0000 (22:36 +1030)
committerAlan Modra <amodra@gmail.com>
Tue, 1 Apr 2025 13:15:46 +0000 (23:45 +1030)
u.kenum is always non-NULL, see debug_make_enum_type.

PR 32829
* debug.c (debug_type_samep): Correct incomplete enum test.
(debug_write_type): Remove dead code.

binutils/debug.c

index dcc8ccde4c5255e58720dda20d35e1aa589597cb..465b18e7c0aaac0f4d282045ee2251f7bddb59e9 100644 (file)
@@ -2554,9 +2554,6 @@ debug_write_type (struct debug_handle *info,
     case DEBUG_KIND_UNION_CLASS:
       return debug_write_class_type (info, fns, fhandle, type, tag);
     case DEBUG_KIND_ENUM:
-      if (type->u.kenum == NULL)
-       return (*fns->enum_type) (fhandle, tag, (const char **) NULL,
-                                 (bfd_signed_vma *) NULL);
       return (*fns->enum_type) (fhandle, tag, type->u.kenum->names,
                                type->u.kenum->values);
     case DEBUG_KIND_POINTER:
@@ -3097,9 +3094,9 @@ debug_type_samep (struct debug_handle *info, struct debug_type_s *t1,
       break;
 
     case DEBUG_KIND_ENUM:
-      if (t1->u.kenum == NULL)
-       ret = t2->u.kenum == NULL;
-      else if (t2->u.kenum == NULL)
+      if (t1->u.kenum->names == NULL)
+       ret = t2->u.kenum->names == NULL;
+      else if (t2->u.kenum->names == NULL)
        ret = false;
       else
        {