[Bug gdb/13175] gdb segfaults after two CLI commands (interpreternot correctly initialised?)

["cvs-commit at gcc dot gnu.org" <sourceware-bugzilla at sourceware dot org>]

http://sourceware.org/bugzilla/show_bug.cgi?id=13175

--- Comment #2 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> 2011-09-12 21:24:56 UTC ---
CVSROOT:    /cvs/src
Module name:    src
Changes by:    palves@sourceware.org    2011-09-12 21:24:52

Modified files:
    gdb            : ChangeLog interps.c interps.h 
    gdb/cli        : cli-interp.c 
    gdb/mi         : mi-common.h mi-interp.c 
    gdb/testsuite  : ChangeLog 
    gdb/testsuite/gdb.base: interp.exp 
    gdb/tui        : tui-interp.c tui-io.c 

Log message:
    gdb/
    2011-09-12  Pedro Alves  <pedro@codesourcery.com>
    Matt Rice  <ratmice@gmail.com>

    PR gdb/13175

    * interps.c (struct interp) <interpreter_out>: Delete field.
    (interp_new): Remove the data and uiout parameters and adjust.
    (interp_set): Only set the current_uiout from the interpreter's
    uiout after initializing the interpreter.  Adjust call to
    init_proc.
    (interp_ui_out): Adjust to call procs->ui_out_proc.
    (interp_data, interp_name): New.
    * interps.h (interp_init_ftype): Add `self' parameter.
    (interp_ui_out_ftype): New typedef.
    (struct interp_procs) <ui_out_proc>: New method pointer.
    (interp_new): Remove the data and uiout parameters.
    (interp_data, interp_name): Declare.
    * tui/tui-interp.c (tui_init): Adjust prototype.
    (tui_ui_out): New.
    (_initialize_tui_interp): Install tui_ui_out.  Don't instanciate
    tui_out here.  Adjust call to interp_new.
    * tui/tui-io.c (tui_initialize_io): Don't set current_uiout here.
    * cli/cli-interp.c (cli_interpreter_init): Adjust prototype.
    (cli_ui_out): New.
    (_initialize_cli_interp): Install it.  Adjust call to interp_new.
    * mi/mi-common.h (struct mi_interp) <uiout>: New field.
    * mi/mi-interp.c (mi_interpreter_init): Adjust prototype.
    Initialize mi->uiout depending on the mi_version as extracted from
    the interpreter's name.
    (mi_ui_out): New.
    (_initialize_mi_interp): Install mi_ui_out.  Adjust calls to
    interp_new.  Don't allocate the ui_out's of the interpreters here.

    gdb/testsuite/
    2011-09-12  Matt Rice  <ratmice@gmail.com>
    Pedro Alves  <pedro@codesourcery.com>

    PR gdb/13175

    * gdb.base/interp.exp: New tests.
    * gdb.base/interp.c: New file.

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gdb/ChangeLog.diff?cvsroot=src&r1=1.13322&r2=1.13323
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gdb/interps.c.diff?cvsroot=src&r1=1.42&r2=1.43
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gdb/interps.h.diff?cvsroot=src&r1=1.20&r2=1.21
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gdb/cli/cli-interp.c.diff?cvsroot=src&r1=1.23&r2=1.24
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gdb/mi/mi-common.h.diff?cvsroot=src&r1=1.10&r2=1.11
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gdb/mi/mi-interp.c.diff?cvsroot=src&r1=1.66&r2=1.67
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gdb/testsuite/ChangeLog.diff?cvsroot=src&r1=1.2850&r2=1.2851
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gdb/testsuite/gdb.base/interp.exp.diff?cvsroot=src&r1=1.10&r2=1.11
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gdb/tui/tui-interp.c.diff?cvsroot=src&r1=1.31&r2=1.32
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gdb/tui/tui-io.c.diff?cvsroot=src&r1=1.30&r2=1.31

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.