aboutsummaryrefslogtreecommitdiff
path: root/gdb/source.c
diff options
context:
space:
mode:
authorJoel Brobecker <brobecker@gnat.com>2010-01-08 13:54:40 +0000
committerJoel Brobecker <brobecker@gnat.com>2010-01-08 13:54:40 +0000
commitf91e5ac34e150aeb5030c1005a510a231449c33c (patch)
treeb5fd1ec7d001b8d3f486ab84d66a480cd7db4f3a /gdb/source.c
parent2de75e717cf6e60bcaf9ec42e92bb6a9d43d96d5 (diff)
downloadgdb-f91e5ac34e150aeb5030c1005a510a231449c33c.zip
gdb-f91e5ac34e150aeb5030c1005a510a231449c33c.tar.gz
gdb-f91e5ac34e150aeb5030c1005a510a231449c33c.tar.bz2
GDB crash with empty executable name (MinGW).
* source.c (openp): Add assert that parameter string is not NULL. if parameter string is an empty string, then return with a failure immediately.
Diffstat (limited to 'gdb/source.c')
-rw-r--r--gdb/source.c14
1 files changed, 14 insertions, 0 deletions
diff --git a/gdb/source.c b/gdb/source.c
index fcfce65..2090326 100644
--- a/gdb/source.c
+++ b/gdb/source.c
@@ -707,6 +707,20 @@ openp (const char *path, int opts, const char *string,
/* The open syscall MODE parameter is not specified. */
gdb_assert ((mode & O_CREAT) == 0);
+ gdb_assert (string != NULL);
+
+ /* A file with an empty name cannot possibly exist. Report a failure
+ without further checking.
+
+ This is an optimization which also defends us against buggy
+ implementations of the "stat" function. For instance, we have
+ noticed that a MinGW debugger built on Windows XP 32bits crashes
+ when the debugger is started with an empty argument. */
+ if (string[0] == '\0')
+ {
+ errno = ENOENT;
+ return -1;
+ }
if (!path)
path = ".";