readelf: replace xmalloc with malloc in slurp_relr_relocs

Using xmalloc makes the null check redundant since failing allocation
will exit the program. Instead use malloc and let the error be
conveyed up the call chain.

diff --git a/binutils/ChangeLog b/binutils/ChangeLog
index 888d453321b553657f0f9a99fd96e3811108591e..8ff4e059bf932da20cb2014346cdb77fd5953c7a 100644 (file)
--- a/binutils/ChangeLog
+++ b/binutils/ChangeLog
@@ -1,3 +1,8 @@
+2022-06-22  Marcus Nilsson <brainbomb@gmail.com>
+
+       * readelf.c: (slurp_relr_relocs) Use malloc instead of xmalloc
+        when allocating space for relocations.
+
 2022-06-21  Kumar N, Bhuvanendra via Binutils  <Kavitha.Natarajan@amd.com>
 
        * dwarf.h (struct debug_info): Add rnglists_base field.

diff --git a/binutils/readelf.c b/binutils/readelf.c
index 4c0a2a347674d2a733c7fcd99a0e7145f29800ff..fe0d27decc3baef9831be4cbcc13e6dffb6676af 100644 (file)
--- a/binutils/readelf.c
+++ b/binutils/readelf.c
@@ -1401,7 +1401,7 @@ slurp_relr_relocs (Filedata * filedata,
            size++;
     }
 
-  *relrsp = (bfd_vma *) xmalloc (size * sizeof (bfd_vma));
+  *relrsp = (bfd_vma *) malloc (size * sizeof (bfd_vma));
   if (*relrsp == NULL)
     {
       free (relrs);