Add missing header to shmget(2)
ClosedPublic

Authored by sevan on Sat, Aug 12, 4:25 PM.

Details

Reviewers
bcr
Group Reviewers
Src Committers
manpages
Commits
rS322432: Add missing header.
Summary

S_IRUSR is defined in sys/stat.h

Bug 209229
mt@markoturk.info

Diff Detail

Repository
rS FreeBSD src repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
sevan created this revision.Sat, Aug 12, 4:25 PM
bcr accepted this revision.Sat, Aug 12, 4:39 PM

OK, go ahead with this.

This revision has a positive review.Sat, Aug 12, 4:39 PM
This revision was automatically updated to reflect the committed changes.
jilles added a subscriber: jilles.Sat, Aug 12, 6:25 PM

This is internally consistent, but does not match what POSIX says. POSIX says only <sys/shm.h> is required and the permissions should be coded using octal constants (yuck). However, if a programmer wants to use S_IRUSR and the like, that is fine. POSIX defines the exact values these constants should have. I don't think POSIX needs to be followed exactly in this.

Note that <sys/ipc.h> can safely be removed from this list. POSIX says <sys/shm.h> should include <sys/ipc.h> and that is what we implement.

Also, it seems like <sys/types.h> can be removed. A shmget() invocation will compile and run with only #include <sys/shm.h>.