OpenVZ Forum: Devel » [RFC PATCH 0/5] net: socket bind to file descriptor introduced

Home » Mailing lists » Devel » [RFC PATCH 0/5] net: socket bind to file descriptor introduced

Show: Today's Messages :: Show Polls :: Message Navigator
E-mail to friend

Re: [RFC PATCH 0/5] net: socket bind to file descriptor introduced [message #47556 is a reply to message #47535]

Wed, 15 August 2012 19:49

ebiederm
Messages: 1354
Registered: February 2006

Senior Member

Stanislav Kinsbursky <skinsbursky@parallels.com> writes:

> This patch set introduces new socket operation and new system call:
> sys_fbind(), which allows to bind socket to opened file.
> File to bind to can be created by sys_mknod(S_IFSOCK) and opened by
> open(O_PATH).
>
> This system call is especially required for UNIX sockets, which has name
> lenght limitation.
>
> The following series implements...

Thinking about this a little more I have serious reservations about this
approach.

Today you are not allowed to bind to an address unless mknod for that
file succeeds. Your patch totally changes those semantics.

Name length limitation does not seeme to justify this at all.

It is possible today to trivially change into a directory and
bind or connect to what would be a long absolute path.

There is also the trick of getting a shorter directory name using
/proc/self/fd if you are threaded and can't change the directory.

The obvious choices at this point are
- Teach bind and connect and af_unix sockets to take longer AF_UNIX
socket path names.

- introduce sockaddr_fd that can be applied to AF_UNIX sockets,
and teach unix_bind and unix_connect how to deal with a second type of sockaddr.
struct sockaddr_fd { short fd_family; short pad; int fd; };

- introduce sockaddr_unix_at that takes a directory file descriptor
as well as a unix path, and teach unix_bind and unix_connect to deal with a
second sockaddr type.
struct sockaddr_unix_at { short family; short pad; int dfd; char path[102]; }
AF_UNIX_AT

I don't know what the implications of for breaking connect up into 3
system calls and changing the semantics are and I would really rather
not have to think about it.

But it certainly does not look to me like you introduce new systems
calls to do what you want.

Eric

Report message to a moderator

[Message index]

		[RFC PATCH 0/5] net: socket bind to file descriptor introduced By: Stanislav Kinsbursky on Wed, 15 August 2012 16:21
		[RFC PATCH 2/5] net: split unix_bind() By: Stanislav Kinsbursky on Wed, 15 August 2012 16:22
		[RFC PATCH 1/5] net: cleanup unix_bind() a little By: Stanislav Kinsbursky on Wed, 15 August 2012 16:22
		[RFC PATCH 3/5] net: new protocol operation fbind() introduced By: Stanislav Kinsbursky on Wed, 15 August 2012 16:22
		[RFC PATCH 4/5] net: fbind() for unix sockets protocol operations introduced By: Stanislav Kinsbursky on Wed, 15 August 2012 16:22
		[RFC PATCH 5/5] syscall: sys_fbind() introduced By: Stanislav Kinsbursky on Wed, 15 August 2012 16:22
		Re: [RFC PATCH 5/5] syscall: sys_fbind() introduced By: hpa on Wed, 15 August 2012 16:30
		Re: [RFC PATCH 5/5] syscall: sys_fbind() introduced By: Stanislav Kinsbursky on Wed, 15 August 2012 16:43
		Re: [RFC PATCH 0/5] net: socket bind to file descriptor introduced By: hpa on Wed, 15 August 2012 17:54
		Re: [RFC PATCH 0/5] net: socket bind to file descriptor introduced By: ebiederm on Wed, 15 August 2012 19:49
		Re: [RFC PATCH 0/5] net: socket bind to file descriptor introduced By: hpa on Wed, 15 August 2012 20:58
		Re: [RFC PATCH 0/5] net: socket bind to file descriptor introduced By: ebiederm on Wed, 15 August 2012 21:25
		Re: [RFC PATCH 0/5] net: socket bind to file descriptor introduced By: ebiederm on Thu, 16 August 2012 03:03
		Re: [RFC PATCH 0/5] net: socket bind to file descriptor introduced By: bfields on Thu, 16 August 2012 13:54
		Re: [RFC PATCH 0/5] net: socket bind to file descriptor introduced By: Stanislav Kinsbursky on Mon, 20 August 2012 10:18
		Re: [RFC PATCH 0/5] net: socket bind to file descriptor introduced By: bfields on Tue, 04 September 2012 19:00
		Re: [RFC PATCH 0/5] net: socket bind to file descriptor introduced By: bfields on Fri, 05 October 2012 20:00
		Re: [RFC PATCH 0/5] net: socket bind to file descriptor introduced By: Stanislav Kinsbursky on Mon, 08 October 2012 08:37
		Re: [RFC PATCH 0/5] net: socket bind to file descriptor introduced By: Ben Pfaff on Wed, 15 August 2012 16:52

Previous Topic:	[PATCH v3 04/13] kmem accounting basic infrastructure
Next Topic:	[PATCH v3 00/13] kmem controller for memcg.

Goto Forum:

-=] Back to Top [=-

[ Syndicate this forum (XML) ] [

] [

]

Current Time: Sun Aug 03 23:12:25 GMT 2025

Total time taken to generate the page: 1.26070 seconds