Commit a86219af authored by James Morris's avatar James Morris Committed by David S. Miller

[SELINUX]: Fine-grained Netlink support - add sk to netlink_send hook

Modifies the LSM netlink_send() hook so that it takes a struct sock
parameter.  SELinux will use this parameter to lookup the class of socket,
which was assigned during socket security initialization.
Signed-off-by: default avatarJames Morris <jmorris@redhat.com>
Signed-off-by: default avatarDavid S. Miller <davem@redhat.com>
parent e74974d7
...@@ -53,7 +53,7 @@ extern void cap_task_reparent_to_init (struct task_struct *p); ...@@ -53,7 +53,7 @@ extern void cap_task_reparent_to_init (struct task_struct *p);
extern int cap_syslog (int type); extern int cap_syslog (int type);
extern int cap_vm_enough_memory (long pages); extern int cap_vm_enough_memory (long pages);
static inline int cap_netlink_send (struct sk_buff *skb) static inline int cap_netlink_send (struct sock *sk, struct sk_buff *skb)
{ {
NETLINK_CB (skb).eff_cap = current->cap_effective; NETLINK_CB (skb).eff_cap = current->cap_effective;
return 0; return 0;
...@@ -639,9 +639,12 @@ struct swap_info_struct; ...@@ -639,9 +639,12 @@ struct swap_info_struct;
* Save security information for a netlink message so that permission * Save security information for a netlink message so that permission
* checking can be performed when the message is processed. The security * checking can be performed when the message is processed. The security
* information can be saved using the eff_cap field of the * information can be saved using the eff_cap field of the
* netlink_skb_parms structure. * netlink_skb_parms structure. Also may be used to provide fine
* grained control over message transmission.
* @sk associated sock of task sending the message.,
* @skb contains the sk_buff structure for the netlink message. * @skb contains the sk_buff structure for the netlink message.
* Return 0 if the information was successfully saved. * Return 0 if the information was successfully saved and message
* is allowed to be transmitted.
* @netlink_recv: * @netlink_recv:
* Check permission before processing the received netlink message in * Check permission before processing the received netlink message in
* @skb. * @skb.
...@@ -1181,7 +1184,7 @@ struct security_operations { ...@@ -1181,7 +1184,7 @@ struct security_operations {
int (*sem_semop) (struct sem_array * sma, int (*sem_semop) (struct sem_array * sma,
struct sembuf * sops, unsigned nsops, int alter); struct sembuf * sops, unsigned nsops, int alter);
int (*netlink_send) (struct sk_buff * skb); int (*netlink_send) (struct sock * sk, struct sk_buff * skb);
int (*netlink_recv) (struct sk_buff * skb); int (*netlink_recv) (struct sk_buff * skb);
/* allow module stacking */ /* allow module stacking */
...@@ -1873,9 +1876,9 @@ static inline int security_setprocattr(struct task_struct *p, char *name, void * ...@@ -1873,9 +1876,9 @@ static inline int security_setprocattr(struct task_struct *p, char *name, void *
return security_ops->setprocattr(p, name, value, size); return security_ops->setprocattr(p, name, value, size);
} }
static inline int security_netlink_send(struct sk_buff * skb) static inline int security_netlink_send(struct sock *sk, struct sk_buff * skb)
{ {
return security_ops->netlink_send(skb); return security_ops->netlink_send(sk, skb);
} }
static inline int security_netlink_recv(struct sk_buff * skb) static inline int security_netlink_recv(struct sk_buff * skb)
...@@ -2501,9 +2504,9 @@ static inline int security_setprocattr(struct task_struct *p, char *name, void * ...@@ -2501,9 +2504,9 @@ static inline int security_setprocattr(struct task_struct *p, char *name, void *
* (rather than hooking into the capability module) to reduce overhead * (rather than hooking into the capability module) to reduce overhead
* in the networking code. * in the networking code.
*/ */
static inline int security_netlink_send (struct sk_buff *skb) static inline int security_netlink_send (struct sock *sk, struct sk_buff *skb)
{ {
return cap_netlink_send (skb); return cap_netlink_send (sk, skb);
} }
static inline int security_netlink_recv (struct sk_buff *skb) static inline int security_netlink_recv (struct sk_buff *skb)
......
...@@ -734,7 +734,7 @@ static int netlink_sendmsg(struct kiocb *kiocb, struct socket *sock, ...@@ -734,7 +734,7 @@ static int netlink_sendmsg(struct kiocb *kiocb, struct socket *sock,
goto out; goto out;
} }
err = security_netlink_send(skb); err = security_netlink_send(sk, skb);
if (err) { if (err) {
kfree_skb(skb); kfree_skb(skb);
goto out; goto out;
......
...@@ -720,7 +720,7 @@ static int dummy_sem_semop (struct sem_array *sma, ...@@ -720,7 +720,7 @@ static int dummy_sem_semop (struct sem_array *sma,
return 0; return 0;
} }
static int dummy_netlink_send (struct sk_buff *skb) static int dummy_netlink_send (struct sock *sk, struct sk_buff *skb)
{ {
if (current->euid == 0) if (current->euid == 0)
cap_raise (NETLINK_CB (skb).eff_cap, CAP_NET_ADMIN); cap_raise (NETLINK_CB (skb).eff_cap, CAP_NET_ADMIN);
......
...@@ -1567,7 +1567,7 @@ static int selinux_vm_enough_memory(long pages) ...@@ -1567,7 +1567,7 @@ static int selinux_vm_enough_memory(long pages)
return -ENOMEM; return -ENOMEM;
} }
static int selinux_netlink_send(struct sk_buff *skb) static int selinux_netlink_send(struct sock *sk, struct sk_buff *skb)
{ {
if (capable(CAP_NET_ADMIN)) if (capable(CAP_NET_ADMIN))
cap_raise (NETLINK_CB (skb).eff_cap, CAP_NET_ADMIN); cap_raise (NETLINK_CB (skb).eff_cap, CAP_NET_ADMIN);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment