Skip to contentSkip to conversation

secid reconciliation-v02: Label locally generated IPv4 traffic

PATCH In review Venkat Yekkirala 2 participants 2 messages Sep 8, 2006 – Sep 18, 2006

Subsystems (5)
lore
Share & tools
lore
#0 lore raw

PATCH 6/7 secid reconciliation-v02: Label locally generated IPv4 traffic

This labels the skb(s) for locally generated IPv4 traffic. This will be reconciled with xfrm secid as well as used in pertinent flow control checks on the outbound later in the LSM hook.

This is not as pretty as it is for IPv6, but what to do? Note that skb(s) that derive the secmark from the originating socket do so in the outbound hook.

NOTE: Forwarded traffic is already labeled with the reconciled secmark on the inbound.

---

include/net/ip.h           |   32 ++++++++++++++++++++++++++++++++
include/net/request_sock.h |   17 +++++++++++++++++
net/dccp/ipv4.c            |    5 +++++
net/ipv4/icmp.c            |    4 ++++
net/ipv4/ip_output.c       |    6 ++++++
net/ipv4/tcp_ipv4.c        |    1 +
6 files changed, 65 insertions(+)
include/net/ip.h +0 -0
index 98f9084..4646c13 100644
@@ -48,6 +48,9 @@ struct ipcm_cookie
	u32			addr;
	int			oif;
	struct ip_options	*opt;

+#ifdef CONFIG_SECURITY_NETWORK
+ __u32 secid;
+#endif /* CONFIG_SECURITY_NETWORK */
};

#define IPCB(skb) ((struct inet_skb_parm*)((skb)->cb)) @@ -383,4 +386,33 @@ #endif

extern struct ctl_table ipv4_table[];

+#ifdef CONFIG_SECURITY_NETWORK
+
+static inline void security_skb_classify_ipcm(struct sk_buff *skb,
+ struct ipcm_cookie *ipc)
+{
+ ipc->secid = 0;
+ ipc->secid = skb->secmark;
+}
+
+static inline void security_ipcm_classify_skb(struct ipcm_cookie *ipc,
+ struct sk_buff *skb)
+{
+ skb->secmark = ipc->secid;
+}
+
+#else
+
+static inline void security_skb_classify_ipcm(struct sk_buff *skb,
+ struct ipcm_cookie *ipc)
+{
+}
+
+static inline void security_ipcm_classify_skb(struct ipcm_cookie *ipc,
+ struct sk_buff *skb)
+{
+}
+
+#endif /* CONFIG_SECURITY_NETWORK */
+
#endif /* _IP_H */

include/net/request_sock.h +0 -0
index 8e165ca..bba8dba 100644
@@ -259,4 +259,21 @@ static inline void reqsk_queue_hash_req(
	write_unlock(&queue->syn_wait_lock);
}

+#ifdef CONFIG_SECURITY_NETWORK
+
+static inline void security_req_classify_skb(struct request_sock *req,
+ struct sk_buff *skb)
+{
+ skb->secmark = req->secid;
+}
+
+#else
+
+static inline void security_req_classify_skb(struct request_sock *req,
+ struct sk_buff *skb)
+{
+}
+
+#endif /* CONFIG_SECURITY_NETWORK */
+
#endif /* _REQUEST_SOCK_H */

net/dccp/ipv4.c +0 -0
index 9a1a76a..526835e 100644
@@ -233,6 +233,8 @@ static void dccp_v4_reqsk_send_ack(struc
	dccp_hdr_set_ack(dccp_hdr_ack_bits(skb),
			 DCCP_SKB_CB(rxskb)->dccpd_seq);

+ security_req_classify_skb(req, skb);
+

	bh_lock_sock(dccp_v4_ctl_socket->sk);
	err = ip_build_and_send_pkt(skb, dccp_v4_ctl_socket->sk,
				    rxskb->nh.iph->daddr,
@@ -264,6 +266,7 @@ static int dccp_v4_send_response(struct 
		dh->dccph_checksum = dccp_v4_checksum(skb, ireq->loc_addr,
						      ireq->rmt_addr);
		memset(&(IPCB(skb)->opt), 0, sizeof(IPCB(skb)->opt));
+		security_req_classify_skb(req, skb);
		err = ip_build_and_send_pkt(skb, sk, ireq->loc_addr,
					    ireq->rmt_addr,
					    ireq->opt);
@@ -746,6 +749,8 @@ static void dccp_v4_ctl_send_reset(struc
	dh->dccph_checksum = dccp_v4_checksum(skb, rxskb->nh.iph->saddr,
					      rxskb->nh.iph->daddr);

+ security_skb_classify_skb(rxskb, skb);
+

	bh_lock_sock(dccp_v4_ctl_socket->sk);
	err = ip_build_and_send_pkt(skb, dccp_v4_ctl_socket->sk,
				    rxskb->nh.iph->daddr,
net/ipv4/icmp.c +0 -0
index c2ad07e..956791a 100644
@@ -389,6 +389,8 @@ static void icmp_reply(struct icmp_bxm *
	if (icmp_xmit_lock())
		return;

+ security_skb_classify_ipcm(skb, &ipc);
+

	icmp_param->data.icmph.checksum = 0;
	icmp_out_count(icmp_param->data.icmph.type);

@@ -507,6 +509,8 @@ void icmp_send(struct sk_buff *skb_in, i

	if (icmp_xmit_lock())
		return;

+ security_skb_classify_ipcm(skb_in, &ipc);
+

	/*
	 *	Construct source address and options.
	 */
net/ipv4/ip_output.c +0 -0
index 97aee76..2e0775c 100644
@@ -926,6 +926,8 @@ alloc_new_skb:
			if (skb == NULL)
				goto error;

+ security_ipcm_classify_skb(ipc, skb);
+

			/*
			 *	Fill in the control structures
			 */
@@ -1122,6 +1124,8 @@ ssize_t	ip_append_page(struct sock *sk, 
				goto error;
			}

+ security_skb_classify_skb(skb_prev, skb);
+

			/*
			 *	Fill in the control structures
			 */
@@ -1349,6 +1353,8 @@ void ip_send_reply(struct sock *sk, stru
	daddr = ipc.addr = rt->rt_src;
	ipc.opt = NULL;

+ security_skb_classify_ipcm(skb, &ipc);
+

	if (replyopts.opt.optlen) {
		ipc.opt = &replyopts.opt;
net/ipv4/tcp_ipv4.c +0 -0
index 39b1798..f21509b 100644
@@ -658,6 +658,7 @@ static int tcp_v4_send_synack(struct soc
					 ireq->rmt_addr,
					 csum_partial((char *)th, skb->len,
						      skb->csum));

+ security_req_classify_skb(req, skb);

		err = ip_build_and_send_pkt(skb, sk, ireq->loc_addr,
					    ireq->rmt_addr,

--
This message was distributed to subscribers of the selinux mailing list. If you no longer wish to subscribe, send mail to [email protected] with the words "unsubscribe selinux" without quotes as the message.

#1 lore raw

include/net/ip.h struct ipcm_cookie · line 54

Quoted 12 lines · Venkat Yekkirala
include/net/ip.h +3 -0
index 98f9084..4646c13 100644
@@ -48,6 +48,9 @@ struct ipcm_cookie
 	u32			addr;
 	int			oif;
 	struct ip_options	*opt;
+#ifdef CONFIG_SECURITY_NETWORK
+	__u32			secid;
+#endif /* CONFIG_SECURITY_NETWORK */
 };

This field should be 'u32'.

Signature · 2 lines
James Morris
<[email protected]>