Page MenuHomeFreeBSD

daemon(8): handle case of waitpid() returning without exited child
ClosedPublic

Authored by kib on Mar 18 2024, 8:49 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, May 4, 10:42 AM
Unknown Object (File)
Tue, Apr 30, 5:33 PM
Unknown Object (File)
Tue, Apr 30, 5:33 PM
Unknown Object (File)
Tue, Apr 30, 5:33 PM
Unknown Object (File)
Sun, Apr 28, 1:26 AM
Unknown Object (File)
Sat, Apr 27, 9:49 PM
Unknown Object (File)
Fri, Apr 26, 4:18 AM
Unknown Object (File)
Wed, Apr 24, 5:38 PM

Details

Summary

Not checking for either WIFEXITED(status) or zero result results in never finishing the loop.

PR: 277764

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kib requested review of this revision.Mar 18 2024, 8:49 AM
This revision is now accepted and ready to land.Mar 18 2024, 2:41 PM

Oh, return () style differs here

tamelingdaniel_gmail.com added inline comments.
usr.sbin/daemon/daemon.c
767

I think you also need to check WIFSIGNALED(status). Otherwise daemon will not finish when the child is killed by a signal.

Check for WIFSIGNALLED.
Un-style return.

This revision now requires review to proceed.Mar 18 2024, 6:28 PM
This revision was not accepted when it landed; it landed in state Needs Review.Mar 20 2024, 1:07 AM
This revision was automatically updated to reflect the committed changes.