[PATCH, libiberty]: Add a couple of missing casts
Gary Benson
gbenson@redhat.com
Wed Nov 13 16:09:00 GMT 2013
Richard Biener wrote:
> On Tue, Nov 12, 2013 at 8:55 PM, Ian Lance Taylor <iant@google.com> wrote:
> > On Tue, Nov 12, 2013 at 11:24 AM, Uros Bizjak <ubizjak@gmail.com> wrote:
> > >
> > > This was uncovered by x86 lto-profiledbootstrap. The patch allows
> > > lto-profiledbootstrap to proceed further.
> > >
> > > 2013-11-12 Uros Bizjak <ubizjak@gmail.com>
> > >
> > > * cp-demangle.c (d_copy_templates): Cast result of malloc
> > > to (struct d_print_template *).
> > > (d_print_comp): Cast result of realloc to (struct d_saved scope *).
> > >
> > > Tested on x86_64-pc-linux-gnu.
> > >
> > > OK for mainline?
> >
> > The patch is OK, but this code is troubling. I obviously should
> > have looked at it earlier. The C++ demangler is sometimes used in
> > panic situations, when malloc is not available. The interface was
> > designed to be usable without requiring malloc, by passing in a
> > sufficiently large buffer. I'm concerned that we apparently now
> > require malloc to work.
>
> That indeed looks like an important regression - Gary, can you
> please work to fix this?
I'm on it.
Thanks,
Gary
--
http://gbenson.net/
More information about the Gcc-patches
mailing list