Browse Source

main: exec_self: make clang analyzer happy

Prevent a theoretical leak of the args memory when the executable path
cannot be found.

Signed-off-by: Yousong Zhou <yszhou4tech@gmail.com>
[fix whitespace, commit description]
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Yousong Zhou 4 years ago
parent
commit
ee26d83e98
1 changed files with 8 additions and 3 deletions
  1. 8 3
      main.c

+ 8 - 3
main.c

@@ -47,10 +47,15 @@ static void
 exec_self(int argc, char **argv)
 {
 	int i;
-	const char *cmd = rpc_exec_lookup(argv[0]);
-	char **args = calloc(argc + 1, sizeof(char *));
+	const char *cmd;
+	char **args;
 
-	if (!cmd || !args)
+	cmd = rpc_exec_lookup(argv[0]);
+	if (!cmd)
+		return;
+
+	args = calloc(argc + 1, sizeof(char *));
+	if (!args)
 		return;
 
 	for (i = 0; i < argc; i++)