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, Jul 4, 7:07 AM
Unknown Object (File)
Thu, Jun 25, 5:35 AM
Unknown Object (File)
Mon, Jun 22, 12:46 PM
Unknown Object (File)
Mon, Jun 22, 7:37 AM
Unknown Object (File)
Sat, Jun 20, 10:52 PM
Unknown Object (File)
May 18 2026, 10:53 PM
Unknown Object (File)
May 12 2026, 1:33 AM
Unknown Object (File)
May 12 2026, 12:30 AM

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.