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)
Wed, Mar 12, 12:56 AM
Unknown Object (File)
Mon, Mar 10, 5:08 AM
Unknown Object (File)
Sun, Mar 9, 11:56 PM
Unknown Object (File)
Sun, Mar 9, 10:54 PM
Unknown Object (File)
Sun, Mar 9, 10:16 PM
Unknown Object (File)
Sun, Mar 9, 10:15 PM
Unknown Object (File)
Feb 9 2025, 7:55 PM
Unknown Object (File)
Feb 9 2025, 8:32 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.