2007-08-10 Michael Snyder <msnyder@access-company.com>
[deliverable/binutils-gdb.git] / gdb / ChangeLog
index 64708ad3b5d1cb018ab0d22813d791137653db75..b998c2eb56ee4828500476afc175634d3331182e 100644 (file)
@@ -1,3 +1,505 @@
+2007-08-10  Michael Snyder  <msnyder@access-company.com>
+
+       * gdbtypes.c: Coding standard cleanup.
+       * gdbtypes.c: Comment/whitespace cleanup.
+
+       * stabsread.c (read_huge_number): Attempt to compute value before
+       values that it depends on.
+
+       * linespec.c (decode_dollar): Dead code, ptr can't be non-null.
+       (decode_objc): Use "NULL" instead of 0.
+       (find_method): Ditto.
+       (decode_all_digits): Ditto.
+       (decode_dollar): Ditto.
+
+       * mi/mi-main.c (mi_cmd_data_evaluate_expression): Resource leak.
+
+       * mi/mi-cmd-var.c (mi_cmd_var_list_children): Memory leak.
+
+       * solib-svr4.c (enable_break): Don't free tmp_pathname until
+       after closing bfd.
+
+       * completer.c: Comment/whitespace cleanup.
+
+2007-08-10  Joel Brobecker  <brobecker@adacore.com>
+
+       * Makefile.in (i386nbsd-nat.o): Add missing dependency.
+
+2007-08-10  Maxim Grigoriev  <maxim2405@gmail.com>
+
+       * remote-fileio.c (remote_fileio_resize_fd_map): Initialize newly
+       allocated file descriptors.
+
+2007-08-10  Joel Brobecker  <brobecker@adacore.com>
+
+       * Makefile.in: Minor cleanup throughout; add some missing variables,
+       add some missing rules, remove some rules that are no longer needed,
+       and fix the dependencies in several rules.
+
+2007-08-10  Ludovic Courtès  <ludo@gnu.org>
+
+       * Makefile.in (SFILES): Add scm-{exp,lang,valprint}.c.
+       (scm_lang_h, scm_tags_h): New.
+       (COMMON_OBS): Add scm-{exp,lang,valprint}.o.
+       (scm-exp.o, scm-lang.o, scm-valprint.o): New targets.
+       * defs.h (enum language): Add `language_scm'.
+
+       * MAINTAINERS: Add Emacs local variables to use UTF-8 upon
+       opening.
+
+2007-08-09  Ludovic Courtès  <ludo@gnu.org>
+
+       * MAINTAINERS (Write After Approval): Add myself.
+
+2007-08-09  Michael Snyder  <msnyder@access-company.com>
+
+       * solib.c (solib_open): Memory leak -- openp returns xmalloc buffer.
+
+2007-08-09  Joel Brobecker  <brobecker@adacore.com>
+
+        * solib-som.c (som_relocate_section_addresses): Stop saving
+        the $CODE$ section in the so_list structure.
+
+2007-08-08  Maxim Grigoriev  <maxim2405@gmail.com>
+
+       * xtensa-tdep.h (XTENSA_MAX_COPROCESSOR): New.
+       (xtensa_register_group_t): Add entries for coprocessors.
+       * xtensa-tdep.c (xtensa_init_reggroups): New register groups.
+       (xtensa_add_reggroups): Likewise.
+       (xtensa_register_reggroup_p): Likewise.
+       (xtensa_coprocessor_register_group): New function.
+       (xtensa_cp): New.
+
+2007-08-08  Jan Kratochvil  <jan.kratochvil@redhat.com>
+
+       * serial.c (serial_open): Fix the OPEN parameter macro expansion.
+
+2007-08-08  Michael Snyder  <msnyder@access-company.com>
+
+       * target.c (target_read_string): Guard against null.
+
+       * varobj.c (value_of_root): Move alloc after return to avoid leak.
+
+       * tui/tui-layout.c (tui_set_layout): Dead code, dead variable.
+
+       * top.c (command_line_input): Memory leak.
+
+       * solib-svr4.c (open_symbol_file_object): Memory leak.
+       (svr4_current_sos): Ditto.
+       (enable_break): Ditto.
+
+       * cp-valprint.c (cp_print_hpacc_virtual_table_entries): Memory leak.
+
+       * dwarf2read.c (add_partial_symbol): Memory leak.
+
+2007-08-06  Michael Snyder  <msnyder@access-company.com>
+
+       * ada-lang.c (desc_bounds): Comparison of function address to NULL.
+
+2007-08-05  Jim Blandy  <jimb@codesourcery.com>
+
+       * macroexp.c (init_buffer): Remove testing code that overrides the
+       caller's length guess.
+       (gather_arguments): Use a larger initial size, now that the vector
+       growth code has been exercised.
+
+2007-08-05  Pedro Alves  <pedro_alves@portugalmail.pt>
+
+       * solib-target.c (solib_target_relocate_section_addresses): Add
+       orig_delta to addr_high.
+
+2007-08-04  Michael Snyder  <msnyder@access-company.com>
+
+       * remote-fileio.c (remote_fileio_func_write): Memory leak.
+
+       * breakpoint.c (print_one_breakpoint): Off by one error.
+
+       * tracepoint.c (add_register): Off by one error.
+       (stringify_collection_list): Free malloc buffer.
+
+2007-08-03  Michael Snyder  <msnyder@access-company.com>
+
+       * mi-cmd-var.c (mi_cmd_var_delete): Remove unused variable,
+       stop memory leak, straighten out cleanups.
+
+       * jv-lang.c (java_link_class_type): Guard against NULL.
+
+2007-08-02  Michael Snyder  <msnyder@access-company.com>
+
+       * gdbtypes.c (create_set_type): Test should only be done within
+       the preceeding if block.  Otherwise, variable is uninitialized.
+
+       * gdbtypes.c (check_typedef): Guard NULL.
+
+2007-08-01  Michael Snyder  <msnyder@access-company.com>
+
+       * cli/cli-decode.c (lookup_cmd): Check for null earlier, to 
+       avoid dereference in lookup_cmd_1.
+
+       * tui/tui-data.c (tui_alloc_content): Move assign out of if, 
+       clean up long lines.
+       (tui_alloc_generic_win_info): Tidy by using XMALLOC macro.
+       (tui_alloc_win_info): Ditto.
+       (tui_add_content_elements): Ditto.
+       * tui/tui-file.c (tui_file_magic): Ditto.
+
+2007-07-31  Michael Snyder  <msnyder@access-company.com>
+
+       * breakpoint.c (breakpoint_init_inferior): Add 'else' to 'if'.
+       True and false paths are mutually exclusive.
+
+       * event-top.c (command_line_handler): Add pedantic return.
+
+       * f-valprint.c (info_common_command): Bail out to prevent null
+       pointer deref.  Break up a long line.
+
+       * exec.c (xfer_memory): Remove redundant condition from 'if'.
+
+       * symfile.c (reread_separate_symbols): Free xmalloced memory.
+
+       * printcmd.c (build_address_symbolic): Remove dead code and dead 
+       variable.
+
+2007-07-31  Daniel Jacobowitz  <dan@codesourcery.com>
+
+       * linespec.c (minsym_found): Advance to the next line if possible.
+
+2007-07-31  Pedro Alves  <pedro_alves@portugalmail.pt>
+
+       * arm-wince-tdep.c (arm_wince_init_abi): Remove svr4 related call.
+       * config/arm/wince.mt (TDEPFILES): Remove solib-legacy.o and
+       solib-svr4.o, and add solib-target.o
+
+2007-07-27  Michael Snyder  <msnyder@access-company.com>
+
+       * tui/tui-data.c (tui_alloc_generic_win_info): Wrong arg for sizeof.
+
+2007-07-26  Maciej W. Rozycki  <macro@linux-mips.org>
+
+       * MAINTAINERS (Write After Approval): Add myself.
+
+2007-07-26  Maciej W. Rozycki  <macro@mips.com>
+
+       * Makefile.in (MAKEHTMLFLAGS): Also search the current directory
+       for include files.
+
+2007-07-25  Maciej W. Rozycki  <macro@mips.com>
+
+       * Makefile.in (MAKEHTMLFLAGS): Remove "-glossary".
+
+2007-07-24  Michael Snyder  <msnyder@access-company.com>
+
+       * solib-svr4.c (svr4_fetch_objfile_link_map): Null pointer check of
+       'buffer' must cover both branches that call strcmp (Coverity).
+
+       * stack.c (print_frame_args): Check return value of lookup_symbol.
+
+       * ax-gdb.c (find_field): Guard against null ptr.
+
+2007-07-24  Ulrich Weigand  <uweigand@de.ibm.com>
+
+       * regformats/reg-spu.dat: Fix order of npc, id registers.
+
+2007-07-24  Ulrich Weigand  <uweigand@de.ibm.com>
+
+       * target.c (memory_xfer_partial): Accesses to unmapped overlay
+       sections should always go to the executable file.
+
+2004-07-20  Chris Dearman  <chris@mips.com>
+
+       * mips-tdep.c (heuristic_proc_start): Add more MIPS16 function
+       prologue instructions.
+
+2007-07-20  Maciej W. Rozycki  <macro@mips.com>
+
+       * mips-tdep.c (mips_next_pc): Use is_mips16_addr() instead of
+       a direct test.
+
+2007-07-20  Chris Dearman  <chris@mips.com>
+            Maciej W. Rozycki  <macro@mips.com>
+
+       * mips-tdep.c (fetch_mips_16): Fix typos and reformat the
+       description.
+
+2007-07-17  Pedro Alves  <pedro_alves@portugalmail.pt>
+           Daniel Jacobowitz  <dan@codesourcery.com>
+
+       * config/i386/cygwin.mt (TDEPFILES): Add solib-target.o.
+       * coff-pe-read.c (read_pe_exported_syms): Delete verbose
+       printf.
+       * NEWS: Mention gdbserver DLL support.
+
+2007-07-17  Daniel Jacobowitz  <dan@codesourcery.com>
+
+       * dwarf2read.c (dwarf_decode_lines): Detect address size mismatches.
+
+2007-07-16  H.J. Lu  <hongjiu.lu@intel.com>
+
+       * i386-linux-nat.c (fetch_regs): Work around gcc 3.4 alias
+       warning bug.
+
+2007-07-13  Kevin Buettner  <kevinb@redhat.com>
+
+       * mep-tdep.c (mep_analyze_prologue): Update comment for BRA
+       instruction case.
+
+2007-07-12  Kevin Buettner  <kevinb@redhat.com>
+
+       * mep-tdep.c (IS_BRA, BRA_DISP): New macros.
+       (mep_analyze_prologue): Add case for BRA instruction.
+
+2007-07-12  Ulrich Weigand  <uweigand@de.ibm.com>
+
+       * solib-svr4.c (LM_ADDR_CHECK): Relax prelink recognition heuristic.
+
+2007-07-10  Nick Roberts  <nickrob@snap.net.nz>
+
+       * breakpoint.c: Include "top.h".
+       (breakpoint_1): Don't set convenience variable $_ if server prefix
+       is used.
+       (_initialize_breakpoint): Describe this behaviour in command help.
+
+2007-07-08  Pedro Alves  <pedro_alves@portugalmail.pt>
+
+       * solib-target.c (library_list_start_segment): Cast address to
+       CORE_ADDR.
+
+2007-07-06  Mark Kettenis  <kettenis@gnu.org>
+
+       * bsd-uthread.c (bsd_uthread_wait): Don't try to fetch thread IDs
+       for terminated processes.
+
+2007-07-05  Michael Snyder  <msnyder@access-company.com>
+
+       * event-top.c (cli_command_loop): Prompt string can (and should)
+       be freed after call to readline (Coverity).  Also move local var
+       declarations into block where they are used.
+
+       * tui/tui-interp.c (tui_command_loop): Prompt string can (and
+       should) be freed after call to readline (Coverity).  Also move
+       local var declarations into block where they are used.
+
+2007-07-03  Andreas Schwab  <schwab@suse.de>
+
+       * linux-nat.c (linux_nat_info_proc_cmd): Fix parsing of columns in
+       /proc/../stat.
+
+2007-07-03  Paul Gilliam  <pgilliam@us.ibm.com>
+           Thiago Bauermann  <bauerman@br.ibm.com>
+           Joseph S. Myers  <joseph@codesourcery.com>
+           Daniel Jacobowitz  <dan@codesourcery.com>
+
+       * remote.c (remote_check_symbols): Use
+       gdbarch_convert_from_func_ptr_addr.
+       * infcall.c (find_function_addr): Handle function descriptors
+       without debugging information.
+       * ppc-linux-tdep.c (ppc_linux_convert_from_func_ptr_addr): Renamed
+       from ppc64_linux_convert_from_func_ptr_addr.  Handle -msecure-plt.
+       (ppc_linux_init_abi): Always set convert_from_func_ptr_addr.
+       * solib-svr4.c (solib_break_names): Remove "._dl_debug_state".
+       (bfd_lookup_symbol): Do not take a SECT_FLAGS argument.  Always
+       allow SEC_CODE and SEC_DATA.
+       (enable_break): Update calls.  Pass current_target to solib_add.
+       Use gdbarch_convert_from_func_ptr_addr.
+
+2007-07-03  Ilko Iliev  <iliev@ronetix.at>
+           Daniel Jacobowitz  <dan@codesourcery.com>
+
+       * symfile.c (print_transfer_performance): Avoid integer overflow.
+       Use larger units.
+
+2007-07-03  Markus Deuling  <deuling@de.ibm.com>
+
+       * cp-namespace.c (lookup_symbol_file): Add block to
+       lookup_symbol_global call.
+       * Makefile.in (solist_h): Add dependency on symtab header.
+       (symtab.o): Add dependency on solist header.
+       * solib.c (solib_global_lookup): New function.
+       * solib-svr4.c (scan_dyntag): Likewise.
+       (elf_locate_base): Call helper routine scan_dyntag.
+       (elf_lookup_lib_symbol): New function.
+       (_initialize_svr4_solib): Add elf_lookup_lib_symbol to svr4_so_ops.
+       * solist.h (symtab.h): New include.
+       (struct target_so_ops): New member lookup_lib_global_symbol.
+       (solib_global_lookup): New prototype.
+       * symtab.c: New include solist.h.
+       (lookup_objfile_from_block): New function.
+       (lookup_global_symbol_from_objfile): New function.
+       (basic_lookup_symbol_nonlocal): Add block to lookup_symbol_global call.
+       (lookup_symbol_global): Call library-specific lookup procedure.
+       * symtab.h (lookup_global_symbol_from_objfile): New prototype.
+
+       * NEWS: Document framework.
+
+2007-07-02  Daniel Jacobowitz  <dan@codesourcery.com>
+
+       * target-descriptions.c (tdesc_create_reg): Do not set reg->type
+       to NULL.
+
+       * cli/cli-script.c (build_command_line): Update NULL check.
+
+2007-07-02  Michael Snyder  <msnyder@access-company.com>
+
+       * p-exp.y (yylex): Memory leak, 'uptokstart' must be freed (Coverity).
+
+2007-07-02  Daniel Jacobowitz  <dan@codesourcery.com>
+
+       * Makefile.in (XMLFILES): Add library-list.dtd.
+       (ALLDEPFILES): Add solib-target.o.
+       (solib-target.o): New rule.
+       * remote.c (PACKET_qXfer_libraries): New constant.
+       (remote_protocol_features): Add qXfer:libraries:read.
+       (remote_wait): Recognize library stop replies.
+       (remote_async_wait): Likewise.  Fix typo.
+       (remote_xfer_partial): Handle TARGET_OBJECT_LIBRARIES.
+       (init_remote_async_ops): Fix typo.
+       (_initialize_remote): Register "set remote library-info-packet".
+       * solib-som.c (som_current_sos): Set addr_low and addr_high.
+       * solib-target.c: New file.
+       * solib.c (solib_map_sections): Use addr_low and addr_high instead
+       of textsection.
+       (info_sharedlibrary_command): Likewise.
+       (solib_add_library, solib_remove_library): New.
+       * solist.h (struct so_list): Replace textsection with addr_low and
+       addr_high.
+       * target.h (enum target_object): Add TARGET_OBJECT_LIBRARIES.
+       * NEWS: Describe new qXfer:libraries:read and shared library
+       event support.
+       * features/library-list.dtd: New.
+
+2007-07-02  Daniel Jacobowitz  <dan@codesourcery.com>
+
+       * infrun.c (inferior_ignoring_startup_exec_events): Delete.
+       (start_remote): Use STOP_QUIETLY_REMOTE.
+       (handle_inferior_event): Do not condition TARGET_WAITKIND_LOADED
+       support on a SOLIB_ADD definition.  Update breakpoints_inserted.
+       Update to match shared library event breakpoint support.  Only
+       resume if appropriate.  Handle STOP_QUIETLY_REMOTE.
+       (normal_stop): Handle TARGET_WAITKIND_LOADED.
+       * fork-child.c (startup_inferior): Do not set
+       inferior_ignoring_startup_exec_events
+       * inferior.h (inferior_ignoring_startup_exec_events): Delete
+       declaration.
+       (enum stop_kind): Improve documentation.  Add STOP_QUIETLY_REMOTE.
+
+2007-07-02  Markus Deuling  <deuling@de.ibm.com>
+
+       * breakpoint.c (insert_bp_location): Remove dead code
+       (DISABLE_UNSETTABLE_BREAK).
+       (disable_breakpoints_in_shlibs)
+       (disable_breakpoints_in_unloaded_shlib): Likewise (comment).
+
+2007-07-02  Daniel Jacobowitz  <dan@codesourcery.com>
+
+       * breakpoint.c (reattach_breakpoints): Do not use remove_breakpoint.
+       Call insert_bp_location.
+
+2007-07-01  H.J. Lu  <hongjiu.lu@intel.com>
+
+       * core-regset.c (fetch_core_registers): Work around gcc 3.4
+       alias warning bug.
+
+2007-07-01  Daniel Jacobowitz  <dan@codesourcery.com>
+
+       * minsyms.c (lookup_minimal_symbol_by_pc_section): Search fewer
+       objfiles.
+
+2007-07-01  Daniel Jacobowitz  <dan@codesourcery.com>
+
+       * top.c (gdb_readline_wrapper_line): Call rl_callback_handler_remove.
+       (struct gdb_readline_wrapper_cleanup): Remove prompt_orig.
+       (gdb_readline_wrapper_cleanup): Do not reset the prompt.
+       (gdb_readline_wrapper): Do not save the prompt.  Pass our prompt
+       to display_gdb_prompt.
+
+2007-07-01  Daniel Jacobowitz  <dan@codesourcery.com>
+
+       PR symtab/2161
+       * target.c (memory_xfer_partial): Do not continue past targets with
+       all memory.
+
+2007-06-30  Andreas Schwab  <schwab@suse.de>
+
+       * m68k-tdep.c (m68k_ps_type): New.
+       (m68k_init_types): New.
+       (m68k_register_type): Use m68k_ps_type for PS register.
+       (_initialize_m68k_tdep): Call m68k_init_types.
+
+       * m68k-tdep.c (m68k_gdbarch_init): Don't infer coldfire flavour
+       from the generic m68k arch.
+
+2007-06-28  Michael Snyder  <msnyder@access-company.com>
+
+       * m2-typeprint.c (m2_print_type): Move pointer ref after null test
+       (Coverity).
+
+       * linux-thread-db.c (thread_db_get_thread_local_address): Add
+       gdb_assert before using return value of find_thread_pid (Coverity).
+
+       * source.c (unset_substitute_path_command): Plug leak (Coverity).
+
+       * cli/cli-script.c (build_command_line): Add null pointer guard 
+       (Coverity).
+       
+2007-06-27  Daniel Jacobowitz  <dan@codesourcery.com>
+
+       * linux-thread-db.c (thread_db_attach): Delete.
+       (thread_db_detach): Typo fix.  Call target_mourn_inferior
+       instead of fixing up proc_handle.
+       (have_threads_callback, have_threads): New functions.
+       (thread_db_wait): Remove dead proc_handle.pid check.  Only
+       translate PTIDs if we have registered threads.  Check for new
+       threads if we have none.
+       (thread_db_create_inferior, thread_db_post_startup_inferior): Delete.
+       (find_new_threads_callback): Only enable event reporting if TID == 0.
+       (same_ptid_callback): New.
+       (thread_db_get_thread_local_address): Check for new threads.
+       (init_thread_db_ops): Don't set to_attach, to_create_inferior,
+       or to_post_startup_inferior.
+
+2007-06-22  Ulrich Weigand  <uweigand@de.ibm.com>
+
+       * infrun.c (adjust_pc_after_break): Do not assume software single-step
+       is always active if SOFTWARE_SINGLE_STEP_P is true.
+       (resume): Use gdbarch_software_single_step[_p] instead of
+       SOFTWARE_SINGLE_STEP[_P].
+       (handle_inferior_event): Do not check for SOFTWARE_SINGLE_STEP_P.
+
+       * gdbarch.sh (software_single_step): Remove target macro.
+       * gdbarch.h, gdbarch.c: Regenerate.
+
+2007-06-22  Ulrich Weigand  <uweigand@de.ibm.com>
+
+       * gdbarch.sh (gdbarch_swap_ftype, deprecated_register_gdbarch_swap,
+       DEPRECATED_REGISTER_GDBARCH_SWAP): Remove.
+       (struct gdbarch_swap, struct gdbarch_swap_registration, 
+       struct gdbarch_swap_registry, gdbarch_swap_registry,
+       current_gdbarch_swap_init_hack, current_gdbarch_swap_out_hack,
+       current_gdbarch_swap_in_hack): Remove.
+       (find_arch_by_info): Do not call current_gdbarch_swap_init_hack.
+       (gdbarch_find_by_info): Do not call current_gdbarch_swap_in_hack
+       and current_gdbarch_swap_out_hack, update current_gdbarch directly.
+       (deprecated_current_gdbarch_select_hack): Likewise.
+       * gdbarch.h, gdbarch.c: Regenerate.
+
+2007-06-22  Ulrich Weigand  <uweigand@de.ibm.com>
+
+       * infrun.c (clear_proceed_status): Clean up stop_registers.
+       (normal_stop): Allocate regcache for stop_registers.
+       (struct inferior_status): Remove stop_registers member.
+       (save_inferior_status): Do not save stop_registers.
+       (restore_inferior_status): Do not restore stop_registers.
+       (discard_inferior_status): Do not discard stop_registers.
+       (build_infrun): Remove.
+       (_initialize_infrun): Do not swap stop_registers.
+
+2007-06-22  Ulrich Weigand  <uweigand@de.ibm.com>
+
+       * remote.c (remote_address_masked): If remote_address_size is zero,
+       default to target address size.
+       (build_remote_gdbarch_data): Remove.
+       (_initialize_remote): Do not swap remote_address_size.
+
 2007-06-22  Ulrich Weigand  <uweigand@de.ibm.com>
 
        * gdbtypes.h (builtin_type_true_char, builtin_type_void,
@@ -5783,4 +6285,5 @@ mode: change-log
 left-margin: 8
 fill-column: 74
 version-control: never
+coding: utf-8
 End:
This page took 0.029968 seconds and 4 git commands to generate.