From cfe0fb4f1776ba22c994dddfd23d8d1e200ee6fc Mon Sep 17 00:00:00 2001
From: Christian Kellner <christian@kellner.me>
Date: Sun, 14 Jul 2019 22:15:06 +0200
Subject: [PATCH] daemon: add requester to GameModeClient

Record the requester process id in the GameModeClient struct and
add a getter for it.
---
 daemon/gamemode-context.c | 17 ++++++++++++++---
 daemon/gamemode.h         |  5 +++++
 2 files changed, 19 insertions(+), 3 deletions(-)

diff --git a/daemon/gamemode-context.c b/daemon/gamemode-context.c
index e0351d6..f2d701a 100644
--- a/daemon/gamemode-context.c
+++ b/daemon/gamemode-context.c
@@ -57,6 +57,7 @@ POSSIBILITY OF SUCH DAMAGE.
 struct GameModeClient {
 	_Atomic int refcount;        /**<Allow outside usage */
 	pid_t pid;                   /**< Process ID */
+	pid_t requester;             /**< Process ID that requested it */
 	struct GameModeClient *next; /**<Next client in the list */
 	char executable[PATH_MAX];   /**<Process executable */
 	time_t timestamp;            /**<When was the client registered */
@@ -90,7 +91,7 @@ static GameModeContext instance = { 0 };
  */
 static volatile bool had_context_init = false;
 
-static GameModeClient *game_mode_client_new(pid_t pid, char *exe);
+static GameModeClient *game_mode_client_new(pid_t pid, char *exe, pid_t req);
 static const GameModeClient *game_mode_context_has_client(GameModeContext *self, pid_t client);
 static void *game_mode_context_reaper(void *userdata);
 static void game_mode_context_enter(GameModeContext *self);
@@ -464,7 +465,7 @@ int game_mode_context_register(GameModeContext *self, pid_t client, pid_t reques
 	}
 
 	/* From now on we depend on the client, initialize it */
-	cl = game_mode_client_new(client, executable);
+	cl = game_mode_client_new(client, executable, requester);
 	if (!cl)
 		goto error_cleanup;
 	free(executable); /* we're now done with memory */
@@ -652,13 +653,14 @@ int game_mode_context_query_status(GameModeContext *self, pid_t client, pid_t re
  *
  * This is deliberately OOM safe
  */
-static GameModeClient *game_mode_client_new(pid_t pid, char *executable)
+static GameModeClient *game_mode_client_new(pid_t pid, char *executable, pid_t requester)
 {
 	/* This bit seems to be formatted differently by different clang-format versions */
 	/* clang-format off */
 	GameModeClient c = {
 		.next = NULL,
 		.pid = pid,
+		.requester = requester,
 		.timestamp = 0,
 	};
 	/* clang-format on */
@@ -726,6 +728,15 @@ const char *game_mode_client_get_executable(GameModeClient *client)
 	return client->executable;
 }
 
+/**
+ * The process identifier of the requester.
+ */
+pid_t game_mode_client_get_requester(GameModeClient *client)
+{
+	assert(client != NULL);
+	return client->requester;
+}
+
 /**
  * The time that game mode was requested for the client.
  */
diff --git a/daemon/gamemode.h b/daemon/gamemode.h
index 78418f0..255aafb 100644
--- a/daemon/gamemode.h
+++ b/daemon/gamemode.h
@@ -70,6 +70,11 @@ pid_t game_mode_client_get_pid(GameModeClient *client);
  */
 const char *game_mode_client_get_executable(GameModeClient *client);
 
+/**
+ * The process identifier of the requester.
+ */
+pid_t game_mode_client_get_requester(GameModeClient *client);
+
 /**
  * The time that game mode was requested for the client.
  */