diff options
| author | Joel Brobecker <brobecker@gnat.com> | 2012-11-26 17:41:44 +0000 |
|---|---|---|
| committer | Joel Brobecker <brobecker@gnat.com> | 2012-11-26 17:41:44 +0000 |
| commit | 17d6efc44b4779a1a8a95645daf293a587ef3da9 (patch) | |
| tree | e27dfa9d556359109c17f73855dafb8e6889931c /gdb/testsuite | |
| parent | 210bbc17ac115d12dbde59b5de97b8547e7465c8 (diff) | |
| download | gdb-17d6efc44b4779a1a8a95645daf293a587ef3da9.zip gdb-17d6efc44b4779a1a8a95645daf293a587ef3da9.tar.gz gdb-17d6efc44b4779a1a8a95645daf293a587ef3da9.tar.bz2 | |
Do not crash when calling GDB with empty executable name.
% gdb -q ''
... or ...
% gdb -q
(gdb) file ''
... both cause GDB to crash with an invalid free. This is because
exec_file_attach is attempting to free a string that has not been
allocated. The string is only allocated if openp is successful.
But in the case of this obviously invalid filename, openp fails,
and leaves scratch_pathname uninitialized, thus causing the xfree
to fail.
The fix is to enable the associated cleanup after we have verified
that openp was successful.
gdb/ChangeLog (By Keith Seitz <keiths@redhat.com>):
* exec.c (exec_file_attach): Move cleanup after verifying that
memory has in fact been allocated.
gdb/testsuite/ChangeLog:
* gdb.base/empty_exe.exp: New testcase.
Diffstat (limited to 'gdb/testsuite')
| -rw-r--r-- | gdb/testsuite/ChangeLog | 4 | ||||
| -rw-r--r-- | gdb/testsuite/gdb.base/empty_exe.exp | 26 |
2 files changed, 30 insertions, 0 deletions
diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog index 907bec3..b625a7b 100644 --- a/gdb/testsuite/ChangeLog +++ b/gdb/testsuite/ChangeLog @@ -1,3 +1,7 @@ +2012-11-26 Joel Brobecker <brobecker@adacore.com> + + * gdb.base/empty_exe.exp: New testcase. + 2012-11-20 Mike Frysinger <vapier@gentoo.org> * gdb.base/completion.exp: Add test for help aliases completion. diff --git a/gdb/testsuite/gdb.base/empty_exe.exp b/gdb/testsuite/gdb.base/empty_exe.exp new file mode 100644 index 0000000..7c55554 --- /dev/null +++ b/gdb/testsuite/gdb.base/empty_exe.exp @@ -0,0 +1,26 @@ +# Copyright 2012 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 <http://www.gnu.org/licenses/>. + +gdb_exit +gdb_start + +# Make sure that the "file" command rejects an empty filename, +# rather than crash. +gdb_test "file ''" \ + ": No such file or directory\\." + +# And to make extra sure that GDB is still alive, do a quick +# sanity check. +gdb_test "print 1" "= 1" |