[PATCH v2 04/27] adduser: replace BB_EXECLP call with BB_EXECVP

Nadav Tasher tashernadav at gmail.com
Tue Jan 21 22:34:43 UTC 2025


Signed-off-by: Nadav Tasher <tashernadav at gmail.com>
---
 loginutils/adduser.c | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/loginutils/adduser.c b/loginutils/adduser.c
index d3c795afa..d9f682389 100644
--- a/loginutils/adduser.c
+++ b/loginutils/adduser.c
@@ -158,8 +158,12 @@ static void passwd_wrapper(const char *login_name) NORETURN;
 
 static void passwd_wrapper(const char *login_name)
 {
-	BB_EXECLP("passwd", "passwd", "--", login_name, NULL);
-	bb_simple_error_msg_and_die("can't execute passwd, you must set password manually");
+	char* passwd_argv[4];
+	passwd_argv[0] = (char *) "passwd";
+	passwd_argv[1] = (char *) "--";
+	passwd_argv[2] = (char *) login_name;
+	passwd_argv[3] = NULL;
+	BB_EXECVP_or_die(passwd_argv);
 }
 
 //FIXME: upstream adduser has no short options! NOT COMPATIBLE!
-- 
2.43.0



More information about the busybox mailing list