Author: dim
Date: Thu Dec 11 08:05:59 2014
New Revision: 275699
URL: https://svnweb.freebsd.org/changeset/base/275699

Log:
  MFC r275477:
  
  Pull in r174303 from upstream gcc trunk (by Jason Merrill):
  
    PR c++/48211
    * name-lookup.h (cp_class_binding): Make base a pointer.
    * name-lookup.c (new_class_binding): Adjust.
    (poplevel_class): Adjust.
  
  This fixes a potential segfault when compiling gold, a part of the
  devel/binutils port, with gcc.  See also the upstream bug report:
  
  https://gcc.gnu.org/bugzilla/show_bug.cgi?id=48211
  
  Thanks to Jason Merrill, Tom Callaway and Red Hat legal for approving
  the use of this patch under the GNU GPL, version 2 or later.

Modified:
  stable/10/contrib/gcc/cp/name-lookup.c
  stable/10/contrib/gcc/cp/name-lookup.h
Directory Properties:
  stable/10/   (props changed)

Changes in other areas also in this revision:
Modified:
  stable/7/contrib/gcc/cp/name-lookup.c
  stable/7/contrib/gcc/cp/name-lookup.h
  stable/8/contrib/gcc/cp/name-lookup.c
  stable/8/contrib/gcc/cp/name-lookup.h
  stable/9/contrib/gcc/cp/name-lookup.c
  stable/9/contrib/gcc/cp/name-lookup.h
Directory Properties:
  stable/7/contrib/gcc/   (props changed)
  stable/8/contrib/gcc/   (props changed)
  stable/9/contrib/gcc/   (props changed)

Modified: stable/10/contrib/gcc/cp/name-lookup.c
==============================================================================
--- stable/10/contrib/gcc/cp/name-lookup.c      Thu Dec 11 06:52:10 2014        
(r275698)
+++ stable/10/contrib/gcc/cp/name-lookup.c      Thu Dec 11 08:05:59 2014        
(r275699)
@@ -319,35 +319,11 @@ new_class_binding (tree name, tree value
   cp_class_binding *cb;
   cxx_binding *binding;
 
-  if (VEC_length (cp_class_binding, scope->class_shadowed))
-    {
-      cp_class_binding *old_base;
-      old_base = VEC_index (cp_class_binding, scope->class_shadowed, 0);
-      if (VEC_reserve (cp_class_binding, gc, scope->class_shadowed, 1))
-       {
-         /* Fixup the current bindings, as they might have moved.  */
-         size_t i;
-
-         for (i = 0;
-              VEC_iterate (cp_class_binding, scope->class_shadowed, i, cb);
-              i++)
-           {
-             cxx_binding **b;
-             b = &IDENTIFIER_BINDING (cb->identifier);
-             while (*b != &old_base[i].base)
-               b = &((*b)->previous);
-             *b = &cb->base;
-           }
-       }
-      cb = VEC_quick_push (cp_class_binding, scope->class_shadowed, NULL);
-    }
-  else
     cb = VEC_safe_push (cp_class_binding, gc, scope->class_shadowed, NULL);
 
   cb->identifier = name;
-  binding = &cb->base;
+  cb->base = binding = cxx_binding_make (value, type);
   binding->scope = scope;
-  cxx_binding_init (binding, value, type);
   return binding;
 }
 
@@ -2501,7 +2477,10 @@ poplevel_class (void)
       for (i = 0;
           VEC_iterate (cp_class_binding, level->class_shadowed, i, cb);
           ++i)
-       IDENTIFIER_BINDING (cb->identifier) = cb->base.previous;
+       {
+         IDENTIFIER_BINDING (cb->identifier) = cb->base->previous;
+         cxx_binding_free (cb->base);
+       }
       ggc_free (level->class_shadowed);
       level->class_shadowed = NULL;
     }

Modified: stable/10/contrib/gcc/cp/name-lookup.h
==============================================================================
--- stable/10/contrib/gcc/cp/name-lookup.h      Thu Dec 11 06:52:10 2014        
(r275698)
+++ stable/10/contrib/gcc/cp/name-lookup.h      Thu Dec 11 08:05:59 2014        
(r275699)
@@ -144,7 +144,7 @@ typedef enum tag_scope {
 
 typedef struct cp_class_binding GTY(())
 {
-  cxx_binding base;
+  cxx_binding *base;
   /* The bound name.  */
   tree identifier;
 } cp_class_binding;
_______________________________________________
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to