Skip to content

Commit 1a7256a

Browse files
author
Denys Vlasenko
committed
chpst: fix a bug where -U USER was using wrong USER (one from -u USER)
Signed-off-by: Denys Vlasenko <[email protected]>
1 parent 7274563 commit 1a7256a

File tree

2 files changed

+3
-5
lines changed

2 files changed

+3
-5
lines changed

libbb/inet_common.c

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -175,8 +175,7 @@ int FAST_FUNC INET6_resolve(const char *name, struct sockaddr_in6 *sin6)
175175
return -1;
176176
}
177177
memcpy(sin6, ai->ai_addr, sizeof(*sin6));
178-
if (ai)
179-
freeaddrinfo(ai);
178+
freeaddrinfo(ai);
180179
return 0;
181180
}
182181

runit/chpst.c

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -236,7 +236,6 @@ int chpst_main(int argc UNUSED_PARAM, char **argv)
236236
{
237237
struct bb_uidgid_t ugid;
238238
char *set_user = set_user; /* for compiler */
239-
char *env_user = env_user;
240239
char *env_dir = env_dir;
241240
char *root;
242241
char *nicestr;
@@ -264,7 +263,7 @@ int chpst_main(int argc UNUSED_PARAM, char **argv)
264263
IF_CHPST("/:n:vP012"),
265264
&limita, &limitc, &limitd, &limitf, &limitl,
266265
&limitm, &limito, &limitp, &limitr, &limits, &limitt,
267-
&set_user, &env_user, &env_dir
266+
&set_user, &set_user, &env_dir
268267
IF_CHPST(, &root, &nicestr));
269268
argv += optind;
270269
if (opt & OPT_m) { // -m means -asld
@@ -292,7 +291,7 @@ int chpst_main(int argc UNUSED_PARAM, char **argv)
292291

293292
// envuidgid?
294293
if (ENABLE_ENVUIDGID && applet_name[0] == 'e' && applet_name[3] == 'u') {
295-
env_user = *argv++;
294+
set_user = *argv++;
296295
opt |= OPT_U;
297296
}
298297

0 commit comments

Comments
 (0)