Implement "info proc exe" for NetBSD

Message ID 20200412175308.13844-1-n54@gmx.com
State Superseded
Headers show
Series
  • Implement "info proc exe" for NetBSD
Related show

Commit Message

Kamil Rytarowski April 12, 2020, 5:53 p.m.
Add nbsd_pid_to_cwd() to query the program current directory.

gdb/ChangeLog:

	* nbsd-nat.c (nbsd_pid_to_cwd): Add.
	(nbsd_nat_target::info_proc): Add do_cwd.
---
 gdb/ChangeLog  |  5 +++++
 gdb/nbsd-nat.c | 27 +++++++++++++++++++++++++++
 2 files changed, 32 insertions(+)

--
2.25.0

Comments

Kamil Rytarowski April 12, 2020, 5:58 p.m. | #1
On 12.04.2020 19:59, Simon Marchi wrote:
> On 2020-04-12 1:53 p.m., Kamil Rytarowski wrote:

>> Add nbsd_pid_to_cwd() to query the program current directory.

>>

>> gdb/ChangeLog:

>>

>> 	* nbsd-nat.c (nbsd_pid_to_cwd): Add.

>> 	(nbsd_nat_target::info_proc): Add do_cwd.

>

> This was sent with the wrong title I suppose, so we should ignore this one?

>


Yes. I resent it with a new title.
Simon Marchi April 12, 2020, 5:59 p.m. | #2
On 2020-04-12 1:53 p.m., Kamil Rytarowski wrote:
> Add nbsd_pid_to_cwd() to query the program current directory.

> 

> gdb/ChangeLog:

> 

> 	* nbsd-nat.c (nbsd_pid_to_cwd): Add.

> 	(nbsd_nat_target::info_proc): Add do_cwd.


This was sent with the wrong title I suppose, so we should ignore this one?

Patch

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 3bbd5c3129d..9e49cf9cf97 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@ 
+2020-04-12  Kamil Rytarowski  <n54@gmx.com>
+
+	* nbsd-nat.c (nbsd_pid_to_cwd): Add.
+        (nbsd_nat_target::info_proc): Add do_cwd.
+
 2020-04-12  Kamil Rytarowski  <n54@gmx.com>

 	* nbsd-nat.c (nbsd_nat_target::info_proc): Add do_exe.
diff --git a/gdb/nbsd-nat.c b/gdb/nbsd-nat.c
index 05aedf8e3f1..628bf919a98 100644
--- a/gdb/nbsd-nat.c
+++ b/gdb/nbsd-nat.c
@@ -44,6 +44,21 @@  nbsd_nat_target::pid_to_exec_file (int pid)
   return buf;
 }

+/* Return the name of a file that can be opened to get the symbols for
+   the child process identified by PID.  */
+
+static std::string
+nbsd_pid_to_cwd (int pid)
+{
+  char buf[PATH_MAX];
+  size_t buflen;
+  int mib[4] = {CTL_KERN, KERN_PROC_ARGS, pid, KERN_PROC_CWD};
+  buflen = sizeof (buf);
+  if (sysctl (mib, ARRAY_SIZE (mib), buf, &buflen, NULL, 0))
+    return "";
+  return buf;
+}
+
 /* Generic thread (LWP) lister within a specified process.  The callback
    parameters is a C++ function that is called for each detected thread.  */

@@ -299,6 +314,7 @@  bool
 nbsd_nat_target::info_proc (const char *args, enum info_proc_what what)
 {
   pid_t pid;
+  bool do_cwd = false;
   bool do_exe = false;
   bool do_mappings = false;

@@ -310,6 +326,9 @@  nbsd_nat_target::info_proc (const char *args, enum info_proc_what what)
     case IP_EXE:
       do_exe = true;
       break;
+    case IP_CWD:
+      do_cwd = true;
+      break;
     default:
       error (_("Not supported on this target."));
     }
@@ -328,6 +347,14 @@  nbsd_nat_target::info_proc (const char *args, enum info_proc_what what)

   printf_filtered (_("process %d\n"), pid);

+  if (do_cwd)
+    {
+      std::string cwd = nbsd_pid_to_cwd (pid);
+      if (cwd != "")
+	printf_filtered ("cwd = '%s'\n", cwd.c_str ());
+      else
+	warning (_("unable to fetch current working directory"));
+    }
   if (do_exe)
     {
       const char *exe = pid_to_exec_file (pid);