Commit 7173c54e authored by Eric Paris's avatar Eric Paris

audit: use spin_lock in audit_receive_msg to process tty logging

This function is called when we receive a netlink message from
userspace.  We don't need to worry about it coming from irq context or
irqs making it re-entrant.
Signed-off-by: default avatarEric Paris <eparis@redhat.com>
parent bee0a224
...@@ -805,12 +805,11 @@ static int audit_receive_msg(struct sk_buff *skb, struct nlmsghdr *nlh) ...@@ -805,12 +805,11 @@ static int audit_receive_msg(struct sk_buff *skb, struct nlmsghdr *nlh)
case AUDIT_TTY_GET: { case AUDIT_TTY_GET: {
struct audit_tty_status s; struct audit_tty_status s;
struct task_struct *tsk = current; struct task_struct *tsk = current;
unsigned long flags;
spin_lock_irqsave(&tsk->sighand->siglock, flags); spin_lock(&tsk->sighand->siglock);
s.enabled = tsk->signal->audit_tty != 0; s.enabled = tsk->signal->audit_tty != 0;
s.log_passwd = tsk->signal->audit_tty_log_passwd; s.log_passwd = tsk->signal->audit_tty_log_passwd;
spin_unlock_irqrestore(&tsk->sighand->siglock, flags); spin_unlock(&tsk->sighand->siglock);
audit_send_reply(NETLINK_CB(skb).portid, seq, audit_send_reply(NETLINK_CB(skb).portid, seq,
AUDIT_TTY_GET, 0, 0, &s, sizeof(s)); AUDIT_TTY_GET, 0, 0, &s, sizeof(s));
...@@ -819,7 +818,6 @@ static int audit_receive_msg(struct sk_buff *skb, struct nlmsghdr *nlh) ...@@ -819,7 +818,6 @@ static int audit_receive_msg(struct sk_buff *skb, struct nlmsghdr *nlh)
case AUDIT_TTY_SET: { case AUDIT_TTY_SET: {
struct audit_tty_status s; struct audit_tty_status s;
struct task_struct *tsk = current; struct task_struct *tsk = current;
unsigned long flags;
memset(&s, 0, sizeof(s)); memset(&s, 0, sizeof(s));
/* guard against past and future API changes */ /* guard against past and future API changes */
...@@ -828,10 +826,10 @@ static int audit_receive_msg(struct sk_buff *skb, struct nlmsghdr *nlh) ...@@ -828,10 +826,10 @@ static int audit_receive_msg(struct sk_buff *skb, struct nlmsghdr *nlh)
(s.log_passwd != 0 && s.log_passwd != 1)) (s.log_passwd != 0 && s.log_passwd != 1))
return -EINVAL; return -EINVAL;
spin_lock_irqsave(&tsk->sighand->siglock, flags); spin_lock(&tsk->sighand->siglock);
tsk->signal->audit_tty = s.enabled; tsk->signal->audit_tty = s.enabled;
tsk->signal->audit_tty_log_passwd = s.log_passwd; tsk->signal->audit_tty_log_passwd = s.log_passwd;
spin_unlock_irqrestore(&tsk->sighand->siglock, flags); spin_unlock(&tsk->sighand->siglock);
break; break;
} }
default: default:
......
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