OpenVZ Forum


Home » Mailing lists » Devel » [patch 0/1][NETNS49] Make af_unix autobind per namespace
Re: [patch 0/1][NETNS49] Make af_unix autobind per namespace [message #21167 is a reply to message #21157] Tue, 02 October 2007 20:51 Go to previous messageGo to previous message
Daniel Lezcano is currently offline  Daniel Lezcano
Messages: 417
Registered: June 2006
Senior Member
Eric W. Biederman wrote:
> Daniel Lezcano <dlezcano@fr.ibm.com> writes:
> 
>> The following patch change autobind fonction to use the ordernum
>> from the network namespace instead of using the local static variable.
> 
> Why do we care?
> Information leak?
> Some application is expecting a predictable autobind value?
> 
> Just skimming the code it looks like it will work correctly without
> this.

I think my summary is ... too short :)

I don't see any applications taking care of this. If they ask for an 
abstract socket, then they don't care about the bind result. So 
probably, the patchset is totally useless.

But from the POV of the checkpoint/restart, we should check if this 
value is somewhere visible from userspace and so storable by an application.

It appears this is the case with /proc/net/unix, where an abstract 
socket is symbolized by the path pattern "@". Example:

cat /proc/net/unix

Num       RefCount Protocol Flags    Type St Inode Path
c6a27710: 00000002 00000000 00000000 0002 01  4357 @00003

I agree by the fact that can be considered as a detail and the 
probability to have an application storing this informaton is very small 
( eg. checkpointing while doing netstat in the container ). But IMHO, 
the paradigm "never seen from userspace" fails and that justifies to 
have the ordernum variable relative to a namespace.

   -- Daniel
_______________________________________________
Containers mailing list
Containers@lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/containers
 
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Previous Topic: [PATCH] mark read_crX() asm code as volatile
Next Topic: [PATCH 03/33] task containersv11 add tasks file interface
Goto Forum:
  


Current Time: Fri Oct 18 18:08:37 GMT 2024

Total time taken to generate the page: 0.04906 seconds