Fix handling of corrupt STABS enum type strings.

	PR binutils/21157
	* stabs.c (parse_stab_enum_type): Check for corrupt NAME:VALUE
	pairs.
	(parse_number): Exit early if passed an empty string.

diff --git a/binutils/ChangeLog b/binutils/ChangeLog
index 5d60a78a159b34a4a8c1a857e91cd7a9d63c0c3a..99774b6b934c503216d8610fae52bc83d066c766 100644 (file)
--- a/binutils/ChangeLog
+++ b/binutils/ChangeLog
@@ -1,3 +1,10 @@
+2017-02-14  Nick Clifton  <nickc@redhat.com>
+
+       PR binutils/21157
+       * stabs.c (parse_stab_enum_type): Check for corrupt NAME:VALUE
+       pairs.
+       (parse_number): Exit early if passed an empty string.
+
 2017-02-14  Nick Clifton  <nickc@redhat.com>
 
        PR binutils/21155

diff --git a/binutils/stabs.c b/binutils/stabs.c
index d3fc4afe242f1086743ea161e83672ca6d340810..3861f83857cc999066a6fdd0e8ea90e1be55030d 100644 (file)
--- a/binutils/stabs.c
+++ b/binutils/stabs.c
@@ -232,6 +232,10 @@ parse_number (const char **pp, bfd_boolean *poverflow)
 
   orig = *pp;
 
+  /* Stop early if we are passed an empty string.  */
+  if (*orig == 0)
+    return (bfd_vma) 0;
+
   errno = 0;
   ul = strtoul (*pp, (char **) pp, 0);
   if (ul + 1 != 0 || errno == 0)
@@ -1975,9 +1979,17 @@ parse_stab_enum_type (void *dhandle, const char **pp)
       bfd_signed_vma val;
 
       p = *pp;
-      while (*p != ':')
+      while (*p != ':' && *p != 0)
        ++p;
 
+      if (*p == 0)
+       {
+         bad_stab (orig);
+         free (names);
+         free (values);
+         return DEBUG_TYPE_NULL;
+       }
+
       name = savestring (*pp, p - *pp);
 
       *pp = p + 1;