]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commitdiff
Re: resbin: don't pass NULL as printf %s arg
authorAlan Modra <amodra@gmail.com>
Sat, 2 Aug 2025 00:42:21 +0000 (10:12 +0930)
committerAlan Modra <amodra@gmail.com>
Sat, 2 Aug 2025 00:50:15 +0000 (10:20 +0930)
Commit c6c8d0b82175 went completely the wrong way.  "key" needs to be
NULL as that reads a different type of data.

PR 33244
* resbin.c (get_version_header): Don't pass a NULL key on to
toosmall.
(bin_to_res_version): Restore NULL key cases.

binutils/resbin.c

index 889126e9e3265826e0efeb1e1e4fcea8fe5e6eab..fa77cd43d23f3839a30877fd9534cd241d16745e 100644 (file)
@@ -1052,7 +1052,7 @@ get_version_header (windres_bfd *wrbfd, const bfd_byte *data,
 {
   if (length < 8)
     {
-      toosmall (key);
+      toosmall (key ? key : _("version header"));
       return false;
     }
 
@@ -1250,7 +1250,7 @@ bin_to_res_version (windres_bfd *wrbfd, const bfd_byte *data,
 
              vst = res_alloc (sizeof (rc_ver_stringtable));
 
-             if (!get_version_header (wrbfd, data, length, "version stringtable",
+             if (!get_version_header (wrbfd, data, length, NULL,
                                       &vst->language, &stverlen, &vallen,
                                       &type, &off))
                return NULL;
@@ -1284,7 +1284,7 @@ bin_to_res_version (windres_bfd *wrbfd, const bfd_byte *data,
 
                  vs = res_alloc (sizeof (rc_ver_stringinfo));
 
-                 if (!get_version_header (wrbfd, data, length, "version string",
+                 if (!get_version_header (wrbfd, data, length, NULL,
                                           &vs->key, &sverlen, &vallen,
                                           &type, &off))
                    return NULL;
@@ -1348,7 +1348,7 @@ bin_to_res_version (windres_bfd *wrbfd, const bfd_byte *data,
          data += off;
          length -= off;
 
-         if (!get_version_header (wrbfd, data, length, "version varfileinfo",
+         if (!get_version_header (wrbfd, data, length, NULL,
                                   &vi->u.var.key, &verlen, &vallen,
                                   &type, &off))
            return NULL;