Page MenuHomeFreeBSD
Differential D20415

Linuxulator: complete LOCAL_PEERCRED support.
ClosedPublic
Actions

Authored by dchagin on May 26 2019, 7:06 PM.
Tags
None
Referenced Files
F136958110: D20415.id.diff
Thu, Nov 20, 8:20 PM
F136957527: D20415.id58073.diff
Thu, Nov 20, 8:17 PM
F136957285: D20415.id57920.diff
Thu, Nov 20, 8:16 PM
F136957161: D20415.id57919.diff
Thu, Nov 20, 8:15 PM
F136955317: D20415.diff
Thu, Nov 20, 8:05 PM
Unknown Object (File)
Tue, Nov 18, 2:48 PM
Unknown Object (File)
Fri, Nov 14, 2:35 PM
Unknown Object (File)
Fri, Nov 7, 5:53 PM
View All Files
Subscribers
imp

Details

Reviewers
tijl
Group Reviewers
Linux Emulation
network
manpages
Commits
rG925f44f33862: Complete LOCAL_PEERCRED support. Cache pid of the remote process in the
rS348419: Complete LOCAL_PEERCRED support. Cache pid of the remote process in the
Summary

Cache pid of the remote process in the struct xucred. Do not bump XUCRED_VERSION as struct layout is not changed.

PR: 215202

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dchagin created this revision.May 26 2019, 7:06 PM
dchagin updated this revision to Diff 57920.May 26 2019, 8:50 PM

Also cache client process pid

dchagin added a reviewer: tijl.May 27 2019, 10:46 AM
tijl accepted this revision.May 27 2019, 1:07 PM

I assume you used a union in struct xucred because a pointer has a different size on different architectures? You could use something like this:

pid_t cr_pid;
u_char _cr_unused1[sizeof(void *)-sizeof(pid_t)];

But perhaps that's not really an improvement.

This revision is now accepted and ready to land.May 27 2019, 1:07 PM
dchagin added a comment.May 27 2019, 2:51 PM
In D20415#440959, @tijl wrote:

I assume you used a union in struct xucred because a pointer has a different size on different architectures? You could use something like this:

sure

pid_t cr_pid;
u_char _cr_unused1[sizeof(void *)-sizeof(pid_t)];

But perhaps that's not really an improvement.

my variant is better for static struct xucred initialization
thanks

Closed by commit rS348419: Complete LOCAL_PEERCRED support. Cache pid of the remote process in the (authored by dchagin). · Explain WhyMay 30 2019, 2:24 PM
This revision was automatically updated to reflect the committed changes.
Herald added a reviewer: manpages. · View Herald TranscriptMay 30 2019, 2:24 PM
Herald added a subscriber: imp. · View Herald Transcript
dchagin added a commit: rG925f44f33862: Complete LOCAL_PEERCRED support. Cache pid of the remote process in the.Mar 21 2021, 5:35 AM

Revision Contents
Changeset List

PathSize
head/
crypto/
heimdal/
lib/
ipc/
2 lines
share/
man/
man4/
1 line
sys/
compat/
linux/
5 lines
kern/
8 lines
8 lines
sys/
7 lines
usr.sbin/
mountd/
2 lines

Diff 58073

View Options

head/crypto/heimdal/lib/ipc/server.c

Loading...
View Options

head/share/man/man4/unix.4

Loading...
View Options

head/sys/compat/linux/linux_socket.c

Loading...
View Options

head/sys/kern/kern_prot.c

Loading...
View Options

head/sys/kern/uipc_usrreq.c

Loading...
View Options

head/sys/sys/ucred.h

Loading...
View Options

head/usr.sbin/mountd/mountd.c

Loading...