From f7fdd5897296a7b9389b6c292e278d842a7a1baf Mon Sep 17 00:00:00 2001
From: sin <sin@2f30.org>
Date: Fri, 18 Oct 2013 10:08:10 +0100
Subject: [PATCH] Use a compound literal instead of malloc() in su(1)

---
 su.c | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/su.c b/su.c
index d2e28e8..273728a 100644
--- a/su.c
+++ b/su.c
@@ -20,7 +20,7 @@ int
 main(int argc, char **argv)
 {
 	char *usr, *pass, *cryptpass;
-	char **newargv;
+	char * const *newargv;
 	struct spwd *spw;
 	struct passwd *pw;
 	uid_t uid;
@@ -83,11 +83,7 @@ main(int argc, char **argv)
 	if (setuid(pw->pw_uid) < 0)
 		eprintf("setuid:");
 
-	newargv = malloc(2 * sizeof(char *));
-	if (!newargv)
-		eprintf("malloc:");
-	newargv[0] = pw->pw_shell;
-	newargv[1] = NULL;
+	newargv = (char *const[]){pw->pw_shell, NULL};
 	setenv("HOME", pw->pw_dir, 1);
 	execve(pw->pw_shell, newargv, environ);
 	return (errno == ENOENT) ? 127 : 126;