Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
* No Subject
@ 1999-03-08  7:10 Andris Pavenis
  1999-03-08 11:15 ` none Jim Blandy
                   ` (3 more replies)
  0 siblings, 4 replies; 32+ messages in thread
From: Andris Pavenis @ 1999-03-08  7:10 UTC (permalink / raw)
  To: gdb-patches

Hi!

Tried gdb-4.17.85 on i586-pc-linux-gnu (with glibc-2.1) and met following
problem that was no with gdb-4.17:

Some global symbols are defined in gdb/main.c (eg. gdb_stdout,
gdb-stderr, etc). As the result if I'm building all except main.c in
object library to use with some IDE that uses gdb for debugging 
(rhide-1.4.7),  I'm getting unresolved references. 

So I'm suggesting rather ugly hack (moving these definitions to top.c
and puting there static constructor that initializes gdb_std*). I agree does
not look nice, but I'm sending it only to point to possible problem.
I cannot use _initialize_* here as files must be defined before running
all _initialize_* procedures.

An alternative idea could be duplicating init code and global vrariables in
rhide.

Andris


--- main.c~1	Wed Feb 24 22:55:05 1999
+++ main.c	Mon Mar  8 16:38:44 1999
@@ -54,18 +54,6 @@
 
 int display_space;
 
-/* Whether this is the command line version or not */
-int tui_version = 0;
-
-/* Whether xdb commands will be handled */
-int xdb_commands = 0;
-
-/* Whether dbx commands will be handled */
-int dbx_commands = 0;
-
-GDB_FILE *gdb_stdout;
-GDB_FILE *gdb_stderr;
-
 /* Whether to enable writing into executable and core files */
 extern int write_files;
 
@@ -161,20 +149,6 @@
 
   getcwd (gdb_dirbuf, sizeof (gdb_dirbuf));
   current_directory = gdb_dirbuf;
-
-  gdb_file_size = sizeof(GDB_FILE);
-
-  gdb_stdout = (GDB_FILE *)xmalloc (gdb_file_size);
-  gdb_stdout->ts_streamtype = afile;
-  gdb_stdout->ts_filestream = stdout;
-  gdb_stdout->ts_strbuf = NULL;
-  gdb_stdout->ts_buflen = 0;
-
-  gdb_stderr = (GDB_FILE *)xmalloc (gdb_file_size);
-  gdb_stderr->ts_streamtype = afile;
-  gdb_stderr->ts_filestream = stderr;
-  gdb_stderr->ts_strbuf = NULL;
-  gdb_stderr->ts_buflen = 0;
 
   /* Parse arguments and options.  */
   {
--- top.c~1	Fri Jan 29 11:46:03 1999
+++ top.c	Mon Mar  8 16:37:03 1999
@@ -53,6 +53,19 @@
 
 extern void initialize_utils PARAMS ((void));
 
+
+/* Whether this is the command line version or not */
+int tui_version = 0;
+
+/* Whether xdb commands will be handled */
+int xdb_commands = 0;
+
+/* Whether dbx commands will be handled */
+int dbx_commands = 0;
+
+GDB_FILE *gdb_stdout;
+GDB_FILE *gdb_stderr;
+
 /* Prototypes for local functions */
 
 static void dont_repeat_command PARAMS ((char *, int));
@@ -467,6 +480,28 @@
 SIGJMP_BUF error_return;
 /* Where to go for return_to_top_level (RETURN_QUIT).  */
 SIGJMP_BUF quit_return;
+
+
+
+void __attribute__((constructor))
+_init_gdb_stdio (void)
+{
+  int gdb_file_size;
+  gdb_file_size = sizeof(GDB_FILE);
+
+  gdb_stdout = (GDB_FILE *)xmalloc (gdb_file_size);
+  gdb_stdout->ts_streamtype = afile;
+  gdb_stdout->ts_filestream = stdout;
+  gdb_stdout->ts_strbuf = NULL;
+  gdb_stdout->ts_buflen = 0;
+
+  gdb_stderr = (GDB_FILE *)xmalloc (gdb_file_size);
+  gdb_stderr->ts_streamtype = afile;
+  gdb_stderr->ts_filestream = stderr;
+  gdb_stderr->ts_strbuf = NULL;
+  gdb_stderr->ts_buflen = 0;
+}
+
 
 /* Return for reason REASON.  This generally gets back to the command
    loop, but can be caught via catch_errors.  */


^ permalink raw reply	[flat|nested] 32+ messages in thread
* No Subject
@ 1998-04-06  4:43 Philippe De Muyter
  1998-04-06 14:16 ` Stan Shebs
  0 siblings, 1 reply; 32+ messages in thread
From: Philippe De Muyter @ 1998-04-06  4:43 UTC (permalink / raw)
  To: gdb-patches, gdb-testers

Am I the only one to have (sometimes) the problem of finding a non-executable
with the same name earlier in my path than the program I want to debug ?

Here is a patch.

I also removed the obsolete (now that we have gdb_string.h) strstr
declaration comment.

Mon Apr  6 13:20:21 1998  Philippe De Muyter  <phdm@macqel.be>

	* source.c (openp): Accept only files that match prot.
	* exec.c (exec_file_command): Request openp to find an executable
	file.

--- ./gdb/source.c	Mon Apr  6 11:22:17 1998
+++ ./gdb/source.c	Mon Apr  6 09:49:08 1998
@@ -83,11 +83,6 @@
 
 static void find_source_lines PARAMS ((struct symtab *, int));
 
-/* If we use this declaration, it breaks because of fucking ANSI "const" stuff
-   on some systems.  We just have to not declare it at all, have it default
-   to int, and possibly botch on a few systems.  Thanks, ANSIholes... */
-/* extern char *strstr(); */
-
 /* Path of directories to search for source files.
    Same format as the PATH environment variable's value.  */
 
@@ -521,7 +516,7 @@ openp (path, try_cwd_first, string, mode
     {
       int i;
       filename = string;
-      fd = open (filename, mode, prot);
+      fd = open (filename, mode);
       if (fd >= 0)
 	goto done;
       for (i = 0; string[i]; i++)
@@ -570,8 +565,10 @@ openp (path, try_cwd_first, string, mode
       strcat (filename+len, SLASH_STRING);
       strcat (filename, string);
 
-      fd = open (filename, mode);
-      if (fd >= 0) break;
+      /* Accept only files matching prot. */
+      if ((prot == 0 || access(filename, prot) == 0)
+      	  && (fd = open (filename, mode)) >= 0)
+	break;
     }
 
  done:
--- ./gdb/exec.c	Mon Apr  6 11:22:18 1998
+++ ./gdb/exec.c	Sat Apr  4 09:37:50 1998
@@ -32,6 +32,12 @@
 #endif
 
 #include <fcntl.h>
+#ifdef HAVE_UNISTD_H
+#include <unistd.h>
+#endif
+#ifndef X_OK
+#define X_OK	1
+#endif
 #include "gdb_string.h"
 
 #include "gdbcore.h"
@@ -185,8 +191,8 @@ exec_file_command (args, from_tty)
       make_cleanup (free, filename);
       
       scratch_chan = openp (getenv ("PATH"), 1, filename, 
-			    write_files? O_RDWR|O_BINARY: O_RDONLY|O_BINARY, 0,
-			    &scratch_pathname);
+			    write_files? O_RDWR|O_BINARY: O_RDONLY|O_BINARY,
+			    X_OK, &scratch_pathname);
 #if defined(__GO32__) || defined(_WIN32)
       if (scratch_chan < 0)
       {


^ permalink raw reply	[flat|nested] 32+ messages in thread
end of thread, other threads:[~1999-04-01  0:00 UTC | newest]

Thread overview: 32+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
1999-03-08  7:10 No Subject Andris Pavenis
1999-03-08 11:15 ` none Jim Blandy
1999-04-01  0:00   ` libgdb (was none) Robert Hoehne
1999-03-08 14:31     ` Robert Hoehne
1999-04-01  0:00     ` Stan Shebs
1999-03-10 15:39       ` Stan Shebs
1999-03-10 16:29       ` Todd Whitesel
1999-04-01  0:00         ` Todd Whitesel
1999-03-14  4:18       ` Robert Hoehne
1999-04-01  0:00         ` DJGPP support (was libgdb) Stan Shebs
1999-03-14 14:41           ` Stan Shebs
1999-04-01  0:00         ` libgdb (was none) Robert Hoehne
1999-04-01  0:00   ` none Stan Shebs
1999-03-08 13:09     ` none Stan Shebs
1999-03-09 12:54     ` none Robert Hoehne
1999-04-01  0:00       ` none Robert Hoehne
1999-04-01  0:00   ` none Jim Blandy
     [not found] ` <99031011141900.03735.cygnus.patches.gdb@hal>
1999-03-10 16:31   ` libgdb.a Andrew Cagney
1999-04-01  0:00     ` libgdb.a J.T. Conklin
1999-03-11 14:29       ` libgdb.a J.T. Conklin
1999-04-01  0:00     ` libgdb.a Andris Pavenis
1999-03-11  0:27       ` libgdb.a Andris Pavenis
1999-04-01  0:00     ` libgdb.a Todd Whitesel
1999-03-10 16:56       ` libgdb.a Todd Whitesel
1999-03-11 12:40       ` libgdb.a Stan Shebs
1999-04-01  0:00         ` libgdb.a Stan Shebs
1999-04-01  0:00     ` libgdb.a Andrew Cagney
1999-04-01  0:00 ` libgdb.a Andris Pavenis
1999-03-10  1:14   ` libgdb.a Andris Pavenis
1999-04-01  0:00 ` No Subject Andris Pavenis
  -- strict thread matches above, loose matches on Subject: below --
1998-04-06  4:43 Philippe De Muyter
1998-04-06 14:16 ` Stan Shebs

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox