MatCat.OpenSource/8sa1-binutils-gdb
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Restore procfs.c compilation

Browse Source 
Since c8fbd44a01 (gdb: remove
target_is_pushed free function), procfs.c compilation is broken, which
went unnoticed for lack of functioning buildbots:

/vol/src/gnu/gdb/hg/master/dist/gdb/procfs.c: In member function 'virtual void procfs_target::attach(const char*, int)':
/vol/src/gnu/gdb/hg/master/dist/gdb/procfs.c:1772:8: error: 'inf' was not declared in this scope; did you mean 'info'?
 1772 |   if (!inf->target_is_pushed (this))
      |        ^~~
      |        info
/vol/src/gnu/gdb/hg/master/dist/gdb/procfs.c: In member function 'virtual void procfs_target::create_inferior(const char*, const string&, char**, int)':
/vol/src/gnu/gdb/hg/master/dist/gdb/procfs.c:2865:8: error: 'inf' was not declared in this scope; did you mean 'info'?
 2865 |   if (!inf->target_is_pushed (this))
      |        ^~~
      |        info

Fixed by defining inf.  Tested on amd64-pc-solaris2.11 and
sparcv9-sun-solaris2.11.

2021-03-29  Rainer Orth  <ro@CeBiTec.Uni-Bielefeld.DE>

	gdb:
	* procfs.c (procfs_target::attach): Define inf.
	Use it.
	(procfs_target::create_inferior): Likewise.
This commit is contained in:
Rainer Orth 2021-03-29 13:26:35 +02:00
parent bbe1eca622
commit 25b48839b3
2 changed files with 11 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
gdb/ChangeLog
View File

@ -1,3 +1,10 @@
2021-03-29 Rainer Orth <ro@CeBiTec.Uni-Bielefeld.DE>
gdb:
* procfs.c (procfs_target::attach): Define inf.
Use it.
(procfs_target::create_inferior): Likewise.
2021-03-28 Tom Tromey <tom@tromey.com> 2021-03-28 Tom Tromey <tom@tromey.com>
* elfread.c (can_lazily_read_symbols): Move to dwarf2/read.c. * elfread.c (can_lazily_read_symbols): Move to dwarf2/read.c.

6
gdb/procfs.c
View File

@ -1768,10 +1768,11 @@ procfs_target::attach (const char *args, int from_tty)
error (_("Attaching GDB to itself is not a good idea...")); error (_("Attaching GDB to itself is not a good idea..."));
/* Push the target if needed, ensure it gets un-pushed it if attach fails. */ /* Push the target if needed, ensure it gets un-pushed it if attach fails. */
inferior *inf = current_inferior ();
target_unpush_up unpusher; target_unpush_up unpusher;
if (!inf->target_is_pushed (this)) if (!inf->target_is_pushed (this))
{ {
current_inferior ()->push_target (this); inf->push_target (this);
unpusher.reset (this); unpusher.reset (this);
} }
@ -2862,8 +2863,9 @@ procfs_target::create_inferior (const char *exec_file,
shell_file = tryname; shell_file = tryname;
} }
inferior *inf = current_inferior ();
if (!inf->target_is_pushed (this)) if (!inf->target_is_pushed (this))
current_inferior ()->push_target (this); inf->push_target (this);
pid = fork_inferior (exec_file, allargs, env, procfs_set_exec_trap, pid = fork_inferior (exec_file, allargs, env, procfs_set_exec_trap,
NULL, procfs_pre_trace, shell_file, NULL); NULL, procfs_pre_trace, shell_file, NULL);