sway

commit 54914260ed38ef4975cdb53ec93ff8a3aab3adf5
parent 889618d3eeb5786df45e1e4885debe610932fc31
Author: Zandr Martin <zandrmartin@gmail.com>
Date:   Sat, 11 Jun 2016 17:10:01 -0500

Merge branch 'master' of git://github.com/SirCmpwn/sway into set-size-command

Diffstat:
M
common/util.c
 | 
20
++++++++++++--------
M
sway/config.c
 | 
2
++

2 files changed, 14 insertions(+), 8 deletions(-)
diff --git a/common/util.c b/common/util.c
@@ -70,7 +70,7 @@ int get_modifier_names(const char **names, uint32_t modifier_masks) {
 }
 
 pid_t get_parent_pid(pid_t child) {
-	pid_t parent;
+	pid_t parent = -1;
 	char file_name[100];
 	char *buffer = NULL;
 	char *token = NULL;
@@ -79,15 +79,19 @@ pid_t get_parent_pid(pid_t child) {
 
 	sprintf(file_name, "/proc/%d/stat", child);
 
-	if ((stat = fopen(file_name, "r")) && (buffer = read_line(stat))) {
-		fclose(stat);
+	if ((stat = fopen(file_name, "r"))) {
+		if ((buffer = read_line(stat))) {
+			token = strtok(buffer, sep); // pid
+			token = strtok(NULL, sep);   // executable name
+			token = strtok(NULL, sep);   // state
+			token = strtok(NULL, sep);   // parent pid
+			parent = strtol(token, NULL, 10);
+		}
 
-		token = strtok(buffer, sep); // pid
-		token = strtok(NULL, sep);   // executable name
-		token = strtok(NULL, sep);   // state
-		token = strtok(NULL, sep);   // parent pid
+		fclose(stat);
+	}
 
-		parent = strtol(token, NULL, 10);
+	if (parent) {
 		return (parent == child) ? -1 : parent;
 	}
 
diff --git a/sway/config.c b/sway/config.c
@@ -101,6 +101,7 @@ static void pid_workspace_cleanup() {
 		pw = config->pid_workspaces->items[i];
 
 		if (difftime(ts.tv_sec, *pw->time_added) >= PID_WORKSPACE_TIMEOUT) {
+			free_pid_workspace(config->pid_workspaces->items[i]);
 			list_del(config->pid_workspaces, i);
 		}
 	}
@@ -126,6 +127,7 @@ void pid_workspace_add(struct pid_workspace *pw) {
 		list_pw = config->pid_workspaces->items[i];
 
 		if (pw->pid == list_pw->pid) {
+			free_pid_workspace(config->pid_workspaces->items[i]);
 			list_del(config->pid_workspaces, i);
 		}
 	}