From 2cd8cc0b66ca297bce4d08e4f712f86d9f1a2fb6 Mon Sep 17 00:00:00 2001 From: Tom Tromey Date: Wed, 19 Dec 2018 11:44:28 -0700 Subject: [PATCH] Fix a crash in jit.c A user at Mozilla pointed out a crash in jit.c. In his situation, an inferior using the JIT API exec'd an executable that did not use it. This caused an assertion failure when jit.c:free_objfile_data called delete_breakpoint with NULL. This patch fixes the problem in the obvious way. New test case included. gdb/ChangeLog 2018-12-28 Tom Tromey * jit.c (free_objfile_data): Only delete breakpoint if non-null. gdb/testsuite/ChangeLog 2018-12-28 Tom Tromey Simon Marchi * gdb.base/jit-exec.exp: New file. * gdb.base/jit-exec.c: New file. * gdb.base/jit-execd.c: New file. --- gdb/ChangeLog | 4 +++ gdb/jit.c | 3 +- gdb/testsuite/ChangeLog | 7 ++++ gdb/testsuite/gdb.base/jit-exec.c | 28 ++++++++++++++++ gdb/testsuite/gdb.base/jit-exec.exp | 52 +++++++++++++++++++++++++++++ gdb/testsuite/gdb.base/jit-execd.c | 22 ++++++++++++ 6 files changed, 115 insertions(+), 1 deletion(-) create mode 100644 gdb/testsuite/gdb.base/jit-exec.c create mode 100644 gdb/testsuite/gdb.base/jit-exec.exp create mode 100644 gdb/testsuite/gdb.base/jit-execd.c diff --git a/gdb/ChangeLog b/gdb/ChangeLog index c2c9f87477..d14a09373d 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,7 @@ +2018-12-28 Tom Tromey + + * jit.c (free_objfile_data): Only delete breakpoint if non-null. + 2018-12-28 Tom Tromey * NEWS: Mention terminal styling. diff --git a/gdb/jit.c b/gdb/jit.c index e6b3cc25ca..9ae5d4a908 100644 --- a/gdb/jit.c +++ b/gdb/jit.c @@ -1464,7 +1464,8 @@ free_objfile_data (struct objfile *objfile, void *data) if (ps_data != NULL && ps_data->objfile == objfile) { ps_data->objfile = NULL; - delete_breakpoint (ps_data->jit_breakpoint); + if (ps_data->jit_breakpoint != NULL) + delete_breakpoint (ps_data->jit_breakpoint); ps_data->cached_code_address = 0; } } diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog index b846b0c7af..3da846fcab 100644 --- a/gdb/testsuite/ChangeLog +++ b/gdb/testsuite/ChangeLog @@ -1,3 +1,10 @@ +2018-12-28 Tom Tromey + Simon Marchi + + * gdb.base/jit-exec.exp: New file. + * gdb.base/jit-exec.c: New file. + * gdb.base/jit-execd.c: New file. + 2018-12-28 Tom Tromey * gdb.base/style.exp: Update test to check for address styling. diff --git a/gdb/testsuite/gdb.base/jit-exec.c b/gdb/testsuite/gdb.base/jit-exec.c new file mode 100644 index 0000000000..85b5334ee7 --- /dev/null +++ b/gdb/testsuite/gdb.base/jit-exec.c @@ -0,0 +1,28 @@ +/* This testcase is part of GDB, the GNU debugger. + + Copyright 2018 Free Software Foundation, Inc. + + This program is free software; you can redistribute it and/or modify + it under the terms of the GNU General Public License as published by + the Free Software Foundation; either version 3 of the License, or + (at your option) any later version. + + This program is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + GNU General Public License for more details. + + You should have received a copy of the GNU General Public License + along with this program. If not, see . */ + +/* Simple standalone program using the JIT API. */ + +#include "jit-simple-jit.c" +#include + +int +main (int argc, char **argv) +{ + execl (PROGRAM, PROGRAM, (char *) 0); + return 99; +} diff --git a/gdb/testsuite/gdb.base/jit-exec.exp b/gdb/testsuite/gdb.base/jit-exec.exp new file mode 100644 index 0000000000..9dbf53c500 --- /dev/null +++ b/gdb/testsuite/gdb.base/jit-exec.exp @@ -0,0 +1,52 @@ +# Copyright 2018 Free Software Foundation, Inc. + +# This program is free software; you can redistribute it and/or modify +# it under the terms of the GNU General Public License as published by +# the Free Software Foundation; either version 3 of the License, or +# (at your option) any later version. +# +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. +# +# You should have received a copy of the GNU General Public License +# along with this program. If not, see . + +# Regression test for a jit.c bug. Previously it would crash if an +# inferior that used the JIT API then exec'd a program that did not +# use it. + +if { ![istarget "*-linux*"] } then { + return +} + +standard_testfile jit-exec.c + +set testfile2 "jit-execd" +set srcfile2 ${testfile2}.c +set binfile2 [standard_output_file ${testfile2}] + +set compile_options [list debug additional_flags=-DPROGRAM=\"$binfile2\"] + +if {[gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" \ + executable $compile_options] != ""} { + untested "failed to compile" + return -1 +} + +if {[gdb_compile "${srcdir}/${subdir}/${srcfile2}" "${binfile2}" \ + executable $compile_options] != ""} { + untested "failed to compile" + return -1 +} + +clean_restart $binfile + +if {![runto_main]} { + fail "can't run to main" + return +} + +delete_breakpoints +gdb_test "continue" "Inferior .* exited normally.*" diff --git a/gdb/testsuite/gdb.base/jit-execd.c b/gdb/testsuite/gdb.base/jit-execd.c new file mode 100644 index 0000000000..6afe539cb4 --- /dev/null +++ b/gdb/testsuite/gdb.base/jit-execd.c @@ -0,0 +1,22 @@ +/* This testcase is part of GDB, the GNU debugger. + + Copyright 2018 Free Software Foundation, Inc. + + This program is free software; you can redistribute it and/or modify + it under the terms of the GNU General Public License as published by + the Free Software Foundation; either version 3 of the License, or + (at your option) any later version. + + This program is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + GNU General Public License for more details. + + You should have received a copy of the GNU General Public License + along with this program. If not, see . */ + +int +main (int argc, char **argv) +{ + return 0; +} -- 2.34.1