diff options
| author | Denys Vlasenko <vda.linux@googlemail.com> | 2018-01-14 19:40:00 +0100 |
|---|---|---|
| committer | Denys Vlasenko <vda.linux@googlemail.com> | 2018-01-14 19:40:00 +0100 |
| commit | c4144ca3a51cf5bfe7dfa5a796ae53e9ae33126c (patch) | |
| tree | efa32d529cad7224312aeb08f88a2194b86f5ac3 /util-linux | |
| parent | 996797855170685eccf0c3d9e56f819e94045938 (diff) | |
| download | busybox-w32-c4144ca3a51cf5bfe7dfa5a796ae53e9ae33126c.tar.gz busybox-w32-c4144ca3a51cf5bfe7dfa5a796ae53e9ae33126c.tar.bz2 busybox-w32-c4144ca3a51cf5bfe7dfa5a796ae53e9ae33126c.zip | |
renice: add comment about getpwnam/NOFORK interaction
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'util-linux')
| -rw-r--r-- | util-linux/renice.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/util-linux/renice.c b/util-linux/renice.c index 65a80001b..46704591f 100644 --- a/util-linux/renice.c +++ b/util-linux/renice.c | |||
| @@ -6,7 +6,6 @@ | |||
| 6 | * | 6 | * |
| 7 | * Licensed under GPLv2 or later, see file LICENSE in this source tree. | 7 | * Licensed under GPLv2 or later, see file LICENSE in this source tree. |
| 8 | */ | 8 | */ |
| 9 | |||
| 10 | /* Notes: | 9 | /* Notes: |
| 11 | * Setting an absolute priority was obsoleted in SUSv2 and removed | 10 | * Setting an absolute priority was obsoleted in SUSv2 and removed |
| 12 | * in SUSv3. However, the common linux version of renice does | 11 | * in SUSv3. However, the common linux version of renice does |
| @@ -95,6 +94,7 @@ int renice_main(int argc UNUSED_PARAM, char **argv) | |||
| 95 | /* Process an ID arg. */ | 94 | /* Process an ID arg. */ |
| 96 | if (which == PRIO_USER) { | 95 | if (which == PRIO_USER) { |
| 97 | struct passwd *p; | 96 | struct passwd *p; |
| 97 | /* NB: use of getpwnam makes it risky to be NOFORK, switch to getpwnam_r? */ | ||
| 98 | p = getpwnam(arg); | 98 | p = getpwnam(arg); |
| 99 | if (!p) { | 99 | if (!p) { |
| 100 | bb_error_msg("unknown user %s", arg); | 100 | bb_error_msg("unknown user %s", arg); |
