Fix sigtimedwait manpage.
ClosedPublic
Actions

Authored by dchagin on Apr 20 2022, 6:53 PM.

Details

Reviewers

imp
jilles

Group Reviewers

manpages

Commits

rGf482354a8636: Fix sigtimedwait manpage.
rG89ecdff2c32c: Fix sigtimedwait manpage.

Summary

Historically, sigtimedwait() blocks indefinitely if timeout is NULL.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 45257
Build 42145: arc lint + arc unit

Event Timeline

dchagin created this revision.Apr 20 2022, 6:53 PM

Herald added a reviewer: manpages. · View Herald TranscriptApr 20 2022, 6:53 PM

Herald added a subscriber: imp. · View Herald Transcript

dchagin requested review of this revision.Apr 20 2022, 6:53 PM

Harbormaster completed remote builds in B45257: Diff 105215.Apr 20 2022, 6:53 PM

dchagin changed the repository for this revision from rS FreeBSD src repository - subversion to rG FreeBSD src repository.Apr 20 2022, 6:53 PM

POSIX indeed leaves this behavior unspecified, but Linux chooses the same as we do. Might want to add something like

POSIX leaves the behavior of
.Fn sigtimedwait
with a
.Dv NULL
.Fa timeout
pointer unspecified.

to the STANDARDS section.

Done, thanks!

Harbormaster completed remote builds in B45287: Diff 105245.Apr 20 2022, 9:15 PM

imp accepted this revision.Apr 20 2022, 9:26 PM

This revision is now accepted and ready to land.Apr 20 2022, 9:26 PM

jilles accepted this revision.Apr 20 2022, 10:11 PM

Closed by commit rG89ecdff2c32c: Fix sigtimedwait manpage. (authored by dchagin). · Explain WhyApr 21 2022, 7:53 AM

This revision was automatically updated to reflect the committed changes.

dchagin added a commit: rG89ecdff2c32c: Fix sigtimedwait manpage..

dchagin added a commit: rGf482354a8636: Fix sigtimedwait manpage..Jun 17 2022, 7:42 PM

Revision Contents
Changeset List

Path

Size

lib/

libc/

sys/

sigwaitinfo.2

6 lines

Diff 105215

View Options

lib/libc/sys/sigwaitinfo.2

Fix sigtimedwait manpage.ClosedPublicActions