> -----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@xxxxxxxxxx > > > https://listman.redhat.com/mailman/listinfo/crash-utility > > > > > > -- > > Crash-utility mailing list > > Crash-utility@xxxxxxxxxx > > https://listman.redhat.com/mailman/listinfo/crash-utility -- Crash-utility mailing list Crash-utility@xxxxxxxxxx https://listman.redhat.com/mailman/listinfo/crash-utility