netdev - Re: [RFC PATCH v6 12/16] selinux: Set socket NetLabel based on connection endpoint

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 1 Oct 2008 11:09:25 -0400
From:	Paul Moore <paul.moore@...com>
To:	Joe Nall <joe@...l.com>
Cc:	James Morris <jmorris@...ei.org>, selinux@...ho.nsa.gov,
	linux-security-module@...r.kernel.org, netdev@...r.kernel.org
Subject: Re: [RFC PATCH v6 12/16] selinux: Set socket NetLabel based on connection endpoint

On Wednesday 01 October 2008 10:51:31 am Joe Nall wrote:
> On Oct 1, 2008, at 5:00 AM, James Morris wrote:
> > On Tue, 16 Sep 2008, Paul Moore wrote:
> >> /**
> >> + * cipso_v4_sock_delattr - Delete the CIPSO option from a socket
> >> + * @sk: the socket
> >> + *
> >> + * Description:
> >> + * Removes the CIPSO option from a socket, if present.
> >> + *
> >> + */
> >> +void cipso_v4_sock_delattr(struct sock *sk)
> >
> > Reviewed-by: James Morris <jmorris@...ei.org>
> >
> >
> > Btw, what kind of testing has this patch had?
>
> Paul generated a patch for us against 2.6.26-5 that we have on about
> 20 developer and test boxes with good results in MLS enforcing. We
> are adding IPSec between machines and static fallback labels  to the
> test mix and will report any issues. We have a couple of 8 core
> machines I'll stress test on to look for SMP issues.

Thanks Joe, however, I think James was talking specifically about the 
this patch, not the entire aggregate patchset, and the 
cipso_v4_sock_delattr() function in particular as it is not likely to 
be called often in normal use.

To answer that question, I was going to write a small little test 
program that would connect/disconnect UDP sockets and do writes in 
various states to ensure the wire traffic was labeled (or not) 
correctly.  However, I had a bit of luck and Casey ended up using this 
function in Smack[1] in a manner that saw it in a relatively common 
code path so it got a decent test; in fact it was Casey who pointed out 
the bugs in the initial revision.

[1] The Smack bits are still a work in progress and not ready for 2.6.28 
so they have been left out of this patchset, expect something in the 
2.6.29 timeframe.

-- 
paul moore
linux @ hp
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html