All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paul Moore <paul.moore@hp.com>
To: selinux@tycho.nsa.gov
Subject: Re: [RFC PATCH v11 18/19] SELinux: Add network ingress and egress control permission checks
Date: Tue, 8 Jan 2008 18:17:50 -0500	[thread overview]
Message-ID: <200801081817.50884.paul.moore@hp.com> (raw)
In-Reply-To: <20080107175636.13733.35147.stgit@flek.americas.hpqcorp.net>


> +	/* If skb->dst->xfrm is non-NULL then the packet is undergoing an
> IPsec
> +	 * packet transformation so allow the packet to pass without 
> any checks
> +	 * since we'll have another chance to perform access 
> control checks
> +	 * when the packet is on it's final way out. */ 
> +	if (skb->dst->xfrm != NULL)
> +		return NF_ACCEPT;

Just in case anyone is playing with this right now, the if statement 
above is wrong, it should be:

 if (skb->dst != NULL && skb->dst->xfrm != NULL)
	return NF_ACCEPT;

... it turns out some IPv6 multicast code never sets skb->dst which 
causes some problems :)  I'll be delaying posting a new patchset for a 
few days to try and catch some more bugs, however, I have updated the 
git tree for those of you who are interested.

-- 
paul moore
linux security @ hp

--
This message was distributed to subscribers of the selinux mailing list.
If you no longer wish to subscribe, send mail to majordomo@tycho.nsa.gov with
the words "unsubscribe selinux" without quotes as the message.

  reply	other threads:[~2008-01-08 23:17 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-01-07 17:54 [RFC PATCH v11 00/19] Labeled networking changes for 2.6.25 Paul Moore
2008-01-07 17:54 ` [RFC PATCH v11 01/19] NetLabel: Remove unneeded RCU read locks Paul Moore
2008-01-07 17:54 ` [RFC PATCH v11 02/19] NetLabel: Cleanup the LSM domain hash functions Paul Moore
2008-01-07 17:55 ` [RFC PATCH v11 03/19] NetLabel: Consolidate the LSM domain mapping/hashing locks Paul Moore
2008-01-07 17:55 ` [RFC PATCH v11 04/19] NetLabel: Add secid token support to the NetLabel secattr struct Paul Moore
2008-01-07 17:55 ` [RFC PATCH v11 05/19] LSM: Add secctx_to_secid() LSM hook Paul Moore
2008-01-07 17:55 ` [RFC PATCH v11 06/19] NetLabel: Add IP address family information to the netlbl_skbuff_getattr() function Paul Moore
2008-01-07 17:55 ` [RFC PATCH v11 07/19] NET: Clone the sk_buff 'iif' field in __skb_clone() Paul Moore
2008-01-07 17:55 ` [RFC PATCH v11 08/19] SELinux: Convert the netif code to use ifindex values Paul Moore
2008-01-07 17:55 ` [RFC PATCH v11 09/19] SELinux: Only store the network interface's ifindex Paul Moore
2008-01-07 17:55 ` [RFC PATCH v11 10/19] SELinux: Add a network node caching mechanism similar to the sel_netif_*() functions Paul Moore
2008-01-07 17:55 ` [RFC PATCH v11 11/19] SELinux: Add a capabilities bitmap to SELinux policy version 22 Paul Moore
2008-01-07 17:55 ` [RFC PATCH v11 12/19] SELinux: Add a new peer class and permissions to the Flask definitions Paul Moore
2008-01-07 17:56 ` [RFC PATCH v11 13/19] SELinux: Better integration between peer labeling subsystems Paul Moore
2008-01-07 17:56 ` [RFC PATCH v11 14/19] SELinux: Enable dynamic enable/disable of the network access checks Paul Moore
2008-01-07 17:56 ` [RFC PATCH v11 15/19] SELinux: Allow NetLabel to directly cache SIDs Paul Moore
2008-01-07 17:56 ` [RFC PATCH v11 16/19] NetLabel: Introduce static network labels for unlabeled connections Paul Moore
2008-01-07 17:56 ` [RFC PATCH v11 17/19] NetLabel: Add auditing to the static labeling mechanism Paul Moore
2008-01-07 17:56 ` [RFC PATCH v11 18/19] SELinux: Add network ingress and egress control permission checks Paul Moore
2008-01-08 23:17   ` Paul Moore [this message]
2008-01-07 17:56 ` [RFC PATCH v11 19/19] SELinux: Add warning messages on network denial due to error Paul Moore

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200801081817.50884.paul.moore@hp.com \
    --to=paul.moore@hp.com \
    --cc=selinux@tycho.nsa.gov \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.