From 4a31419eab30909a785019c3f73df7c7714965c4 Mon Sep 17 00:00:00 2001
From: "Kelvin M. Klann" <kmk3.code@protonmail.com>
Date: Sun, 12 Jan 2025 10:51:07 -0300
Subject: [PATCH] refactor: deduplicate definition of RUN_FIREJAIL_NAME_DIR

Reuse the definition from src/include/rundefs.h.

Related commits:

* 07c05e8a5 ("moved sandbox name to /run/firejail/name/<PID>",
  2016-02-19)
* 57ffc35a8 ("added sandbox name support in firemon", 2018-03-21)
---
 src/include/rundefs.h | 2 +-
 src/lib/common.c      | 6 +++---
 src/lib/pid.c         | 5 +----
 3 files changed, 5 insertions(+), 8 deletions(-)

diff --git a/src/include/rundefs.h b/src/include/rundefs.h
index 00eaa47c6..932942303 100644
--- a/src/include/rundefs.h
+++ b/src/include/rundefs.h
@@ -25,7 +25,7 @@
 #define RUN_FIREJAIL_DIR		RUN_FIREJAIL_BASEDIR "/firejail"
 #define RUN_FIREJAIL_SANDBOX_DIR	RUN_FIREJAIL_DIR "/sandbox"
 #define RUN_FIREJAIL_APPIMAGE_DIR	RUN_FIREJAIL_DIR "/appimage"
-#define RUN_FIREJAIL_NAME_DIR		RUN_FIREJAIL_DIR "/name" // also used in src/lib/pid.c - todo: move it in a common place
+#define RUN_FIREJAIL_NAME_DIR		RUN_FIREJAIL_DIR "/name"
 #define RUN_FIREJAIL_LIB_DIR		RUN_FIREJAIL_DIR "/lib"
 #define RUN_FIREJAIL_X11_DIR		RUN_FIREJAIL_DIR "/x11"
 #define RUN_FIREJAIL_NETWORK_DIR	RUN_FIREJAIL_DIR "/network"
diff --git a/src/lib/common.c b/src/lib/common.c
index 7706077ec..e2a5c9871 100644
--- a/src/lib/common.c
+++ b/src/lib/common.c
@@ -33,6 +33,7 @@
 #include <limits.h>
 #include <sched.h>
 #include "../include/common.h"
+#include "../include/rundefs.h"
 
 #include <fcntl.h>
 #ifndef O_PATH
@@ -155,10 +156,9 @@ int name2pid(const char *name, pid_t *pid) {
 			free(comm);
 		}
 
-		// look for the sandbox name in /run/firejail/name/<PID>
-		// todo: use RUN_FIREJAIL_NAME_DIR define from src/firejail/firejail.h
+		// look for the sandbox name
 		char *fname;
-		if (asprintf(&fname, "/run/firejail/name/%d", newpid) == -1)
+		if (asprintf(&fname, "%s/%d", RUN_FIREJAIL_NAME_DIR, newpid) == -1)
 			errExit("asprintf");
 		FILE *fp = fopen(fname, "r");
 		if (fp) {
diff --git a/src/lib/pid.c b/src/lib/pid.c
index 3f40ecdf3..ad1ee1c03 100644
--- a/src/lib/pid.c
+++ b/src/lib/pid.c
@@ -20,6 +20,7 @@
 
 #include "../include/common.h"
 #include "../include/pid.h"
+#include "../include/rundefs.h"
 #include <string.h>
 #include <sys/types.h>
 #include <sys/stat.h>
@@ -173,10 +174,6 @@ doexit:
 	return rv;
 }
 
-// todo: RUN_FIREJAIL_NAME_DIR is borrowed from src/firejail/firejail.h
-// move it in a common place
-#define RUN_FIREJAIL_NAME_DIR	"/run/firejail/name"
-
 static void print_elem(unsigned index, int nowrap) {
 	// get terminal size
 	struct winsize sz;