From: Alan Modra <amodra@gmail.com>
Date: Thu, 27 Dec 2018 23:04:28 +0000 (+1030)
Subject: PR23966, mingw failure due to 32-bit long
X-Git-Tag: binutils-2_32~238
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=cb87d9f1a49986b2eb92ac381444f4cc7c9d8a4f;p=platform%2Fupstream%2Fbinutils.git

PR23966, mingw failure due to 32-bit long

	PR 23966
	* libbfd.c (SSIZE_MAX): Define.
	(bfd_malloc, bfd_realloc): Don't cast size to long to check for
	"negative" values, compare against SSIZE_MAX instead.
---

diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index 84be63b..8a073f6 100644
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,3 +1,10 @@
+2018-12-28  Alan Modra  <amodra@gmail.com>
+
+	PR 23966
+	* libbfd.c (SSIZE_MAX): Define.
+	(bfd_malloc, bfd_realloc): Don't cast size to long to check for
+	"negative" values, compare against SSIZE_MAX instead.
+
 2018-12-23  H.J. Lu  <hongjiu.lu@intel.com>
 
 	* elf32-i386.c (elf_i386_rtype_to_howto): Remove the unused bfd
diff --git a/bfd/libbfd.c b/bfd/libbfd.c
index 7c45d52..305ee22 100644
--- a/bfd/libbfd.c
+++ b/bfd/libbfd.c
@@ -254,6 +254,10 @@ _bfd_dummy_target (bfd *ignore_abfd ATTRIBUTE_UNUSED)
 
 /* Allocate memory using malloc.  */
 
+#ifndef SSIZE_MAX
+#define SSIZE_MAX ((size_t) -1 >> 1)
+#endif
+
 void *
 bfd_malloc (bfd_size_type size)
 {
@@ -262,7 +266,7 @@ bfd_malloc (bfd_size_type size)
 
   if (size != sz
       /* This is to pacify memory checkers like valgrind.  */
-      || ((signed long) sz) < 0)
+      || sz > SSIZE_MAX)
     {
       bfd_set_error (bfd_error_no_memory);
       return NULL;
@@ -304,7 +308,7 @@ bfd_realloc (void *ptr, bfd_size_type size)
 
   if (size != sz
       /* This is to pacify memory checkers like valgrind.  */
-      || ((signed long) sz) < 0)
+      || sz > SSIZE_MAX)
     {
       bfd_set_error (bfd_error_no_memory);
       return NULL;