[Crash-utility] [PATCH] GDB started inside crash should not read any .gdbinit
-----Original Message-----
> -----Original Message-----
> > I don't think we want .gdbinit files to impact functions in
> > crash. Example of bad behavior, a .gdbinit file with:
> > set logging on
> >
> > then when starting crash there is segmentation fault.
>
> Thanks for the report and patch.
>
> Reading .gdbinit is the intended functionality of crash:
>
> crash> define func
> crash: restricted gdb command: define
> "define" may only be used in a .gdbinit file or in a command file.
> The .gdbinit file is read automatically during crash initialization.
> ...
>
> and I would like to avoid changing the default behavior, so how about
> adding --no_gdbinit option like --no_crashrc ?
Ok, so there is a need to read .gdbinit file for GDB embedded in
crash. Also it looks for the file in same default path as regular GDB.
Adding a --no_gdbinit option, then there is a possibility to avoid
segmentation fault. It could be good enough.
or if "set logging on" is the only reason to drop .gdbinit, stop
execution of "set logging on" ? "gdb set logging on" is also bad..
It might be a bit hard to do this though.
I am not sure if there are other commands that cause problems.
crash> gdb set logging on
Copying output to gdb.txt.
crash> ptype
<segmentation violation in gdb>
crash>
Thanks,
Kazu
>
> (btw, the crash mailing list accepts only emails posted by member.
> I accepted this email but if someone replies only to the list,
> you will miss it. Please subscribe to the list if possible.)
I was actually subscribed, but with slightly different email domain. I
think its solved now.
Thanks
Johan
>
> Thanks,
> Kazu
>
> > ---
> > gdb_interface.c | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/gdb_interface.c b/gdb_interface.c
> > index f4f4dd3..cfa9f1e 100644
> > --- a/gdb_interface.c
> > +++ b/gdb_interface.c
> > @@ -42,6 +42,7 @@ gdb_main_loop(int argc, char **argv)
> > {
> > argc = 1;
> >
> > + argv[argc++] = "--nx";
> > if (pc->flags & SILENT) {
> > if (pc->flags & READNOW)
> > argv[argc++] = "--readnow";
> > --
> > 2.17.1
> >
> >
> >
> > --
> > Crash-utility mailing list
> > Crash-utility(a)redhat.com
> > https://listman.redhat.com/mailman/listinfo/crash-utility
>
>
> --
> Crash-utility mailing list
> Crash-utility(a)redhat.com
> https://listman.redhat.com/mailman/listinfo/crash-utility