From e080a68fce6b8e73c9869a6016d726f820dee7fc Mon Sep 17 00:00:00 2001 From: Jim Kingdon Date: Sat, 15 Oct 1994 18:13:47 +0000 Subject: [PATCH] * exec.c (map_vmap): Cast return from xmalloc to its proper type, not to PTR. * symfile.c (reread_symbols): Include bfd_errmsg string in error message if bfd_close fails. * exec.c (exec_close), solib.c (clear_solib), corelow.c (core_close), objfiles.c (free_objfile), irix5-nat.c (clear_solib), remote-utils.c (gr_load_image): Check for errors from bfd_close. * solib.c (look_for_base), remote-utils.c (gr_load_image), remote-udi.c (download), corelow.c (core_open), symfile.c (symfile_bfd_open), symfile.c (generic_load): Add comment regarding error from bfd_close. * remote-udi.c (download), remote-utils.c (gr_load_image): Add comment about bogus handling of errors from bfd_openr. * exec.c (exec_close): Add comment regarding memory leak and dangling reference to vp->name. --- gdb/ChangeLog | 20 ++++++++++++++++++++ gdb/remote-udi.c | 5 +++++ 2 files changed, 25 insertions(+) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 3abb0956bb..e6309872f8 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,23 @@ +Fri Oct 14 10:29:08 1994 Jim Kingdon (kingdon@lioth.cygnus.com) + + * exec.c (map_vmap): Cast return from xmalloc to its proper type, + not to PTR. + + * symfile.c (reread_symbols): Include bfd_errmsg string in error + message if bfd_close fails. + * exec.c (exec_close), solib.c (clear_solib), corelow.c + (core_close), objfiles.c (free_objfile), irix5-nat.c + (clear_solib), remote-utils.c (gr_load_image): + Check for errors from bfd_close. + * solib.c (look_for_base), remote-utils.c (gr_load_image), + remote-udi.c (download), corelow.c (core_open), symfile.c + (symfile_bfd_open), symfile.c (generic_load): Add comment + regarding error from bfd_close. + * remote-udi.c (download), remote-utils.c (gr_load_image): Add + comment about bogus handling of errors from bfd_openr. + * exec.c (exec_close): Add comment regarding memory leak and + dangling reference to vp->name. + Sat Oct 15 03:43:00 1994 Peter Schauer (pes@regent.e-technik.tu-muenchen.de) * eval.c (evaluate_subexp): Make fnptr a LONGEST instead diff --git a/gdb/remote-udi.c b/gdb/remote-udi.c index e57cad0051..c2366fdf8d 100644 --- a/gdb/remote-udi.c +++ b/gdb/remote-udi.c @@ -1139,8 +1139,13 @@ download(load_arg_string, from_tty) pbfd = bfd_openr (filename, gnutarget); if (!pbfd) + /* FIXME: should be using bfd_errmsg, not assuming it was + bfd_error_system_call. */ perror_with_name (filename); + /* FIXME: should be checking for errors from bfd_close (for one thing, + on error it does not free all the storage associated with the + bfd). */ make_cleanup (bfd_close, pbfd); QUIT;