Page MenuHomeFreeBSD

renice: fix renice_abs_user and renice_rel_user if root user configured with other shell as sh
ClosedPublic

Authored by olivier on Oct 11 2022, 12:04 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Jul 27, 7:30 AM
Unknown Object (File)
Fri, Jul 25, 9:22 PM
Unknown Object (File)
Thu, Jul 24, 11:09 AM
Unknown Object (File)
Jul 2 2025, 4:48 AM
Unknown Object (File)
Jul 1 2025, 3:04 AM
Unknown Object (File)
Jun 28 2025, 5:44 PM
Unknown Object (File)
Jun 27 2025, 9:13 PM
Unknown Object (File)
Jun 25 2025, 8:29 PM
Subscribers

Details

Summary

renice_abs_user and renice_rel_user regression tests imply a root account configured with /bin/sh.
In case of other shell like csh, it will fail to retrieve the pid because csh already display job and process id when send to background.

Test Plan

Without the patch:

root@dev:/usr/tests # chsh -s /bin/csh
chsh: user information updated
root@dev:/usr/tests # kyua test usr.bin/renice/
usr.bin/renice/renice_test:renice_abs_pgid  ->  passed  [0.022s]
usr.bin/renice/renice_test:renice_abs_pid  ->  passed  [0.018s]
usr.bin/renice/renice_test:renice_abs_user  ->  failed: 3 !=  (3 != )  [0.084s]
usr.bin/renice/renice_test:renice_delim  ->  passed  [0.062s]
usr.bin/renice/renice_test:renice_rel_pgid  ->  passed  [0.024s]
usr.bin/renice/renice_test:renice_rel_pid  ->  passed  [0.026s]
usr.bin/renice/renice_test:renice_rel_user  ->  failed: 6 !=  (6 != )  [0.025s]

With the patch:

root@dev:/usr/tests # chsh -s /bin/csh
chsh: user information updated
root@dev:/usr/tests # kyua test usr.bin/renice/
usr.bin/renice/renice_test:renice_abs_pgid  ->  passed  [0.011s]
usr.bin/renice/renice_test:renice_abs_pid  ->  passed  [0.011s]
usr.bin/renice/renice_test:renice_abs_user  ->  passed  [0.018s]
usr.bin/renice/renice_test:renice_delim  ->  passed  [0.036s]
usr.bin/renice/renice_test:renice_rel_pgid  ->  passed  [0.014s]
usr.bin/renice/renice_test:renice_rel_pid  ->  passed  [0.016s]
usr.bin/renice/renice_test:renice_rel_user  ->  passed  [0.020s]

Results file id is usr_tests.20221010-235631-717371
Results saved to /root/.kyua/store/results.usr_tests.20221010-235631-717371.db

7/7 passed (0 failed)

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped