From: Joel Brobecker Date: Wed, 4 Oct 2006 21:36:39 +0000 (+0000) Subject: * somread.c (som_symtab_read): Avoid using alloca for potentially X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=f31b3751745fb4e229e571e25ece906194efc52f;p=binutils-gdb.git * somread.c (som_symtab_read): Avoid using alloca for potentially large buffers. --- diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 971a05bd07f..c6bf6b3a880 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,8 @@ +2006-10-04 Joel Brobecker + + * somread.c (som_symtab_read): Avoid using alloca for potentially + large buffers. + 2006-10-04 Daniel Jacobowitz * arch-utils.h (gdbarch_info_fill): Remove duplicate prototype. diff --git a/gdb/somread.c b/gdb/somread.c index c187d13c1dc..f17094d354f 100644 --- a/gdb/somread.c +++ b/gdb/somread.c @@ -88,15 +88,22 @@ som_symtab_read (bfd *abfd, struct objfile *objfile, number_of_symbols = bfd_get_symcount (abfd); - /* FIXME (alloca): could be quite large. */ - buf = alloca (symsize * number_of_symbols); + /* Allocate a buffer to read in the debug info. + We avoid using alloca because the memory size could be so large + that we could hit the stack size limit. */ + buf = xmalloc (symsize * number_of_symbols); + make_cleanup (xfree, buf); bfd_seek (abfd, obj_som_sym_filepos (abfd), SEEK_SET); val = bfd_bread (buf, symsize * number_of_symbols, abfd); if (val != symsize * number_of_symbols) error (_("Couldn't read symbol dictionary!")); - /* FIXME (alloca): could be quite large. */ - stringtab = alloca (obj_som_stringtab_size (abfd)); + /* Allocate a buffer to read in the som stringtab section of + the debugging info. Again, we avoid using alloca because + the data could be so large that we could potentially hit + the stack size limitat. */ + stringtab = xmalloc (obj_som_stringtab_size (abfd)); + make_cleanup (xfree, stringtab); bfd_seek (abfd, obj_som_str_filepos (abfd), SEEK_SET); val = bfd_bread (stringtab, obj_som_stringtab_size (abfd), abfd); if (val != obj_som_stringtab_size (abfd))