Commit ddf5836a authored by Chris Wright's avatar Chris Wright Committed by Willy Tarreau

security: add cred argument to security_capable()

commit 6037b715 upstream.

Expand security_capable() to include cred, so that it can be usable in a
wider range of call sites.
Signed-off-by: default avatarChris Wright <chrisw@sous-sol.org>
Acked-by: default avatarSerge Hallyn <serge.hallyn@canonical.com>
Signed-off-by: default avatarJames Morris <jmorris@namei.org>
[wt: needed by next patch only]
Signed-off-by: default avatarWilly Tarreau <w@1wt.eu>
parent 154f5f0c
...@@ -1733,7 +1733,7 @@ int security_capset(struct cred *new, const struct cred *old, ...@@ -1733,7 +1733,7 @@ int security_capset(struct cred *new, const struct cred *old,
const kernel_cap_t *effective, const kernel_cap_t *effective,
const kernel_cap_t *inheritable, const kernel_cap_t *inheritable,
const kernel_cap_t *permitted); const kernel_cap_t *permitted);
int security_capable(int cap); int security_capable(const struct cred *cred, int cap);
int security_real_capable(struct task_struct *tsk, int cap); int security_real_capable(struct task_struct *tsk, int cap);
int security_real_capable_noaudit(struct task_struct *tsk, int cap); int security_real_capable_noaudit(struct task_struct *tsk, int cap);
int security_acct(struct file *file); int security_acct(struct file *file);
...@@ -1938,9 +1938,9 @@ static inline int security_capset(struct cred *new, ...@@ -1938,9 +1938,9 @@ static inline int security_capset(struct cred *new,
return cap_capset(new, old, effective, inheritable, permitted); return cap_capset(new, old, effective, inheritable, permitted);
} }
static inline int security_capable(int cap) static inline int security_capable(const struct cred *cred, int cap)
{ {
return cap_capable(current, current_cred(), cap, SECURITY_CAP_AUDIT); return cap_capable(current, cred, cap, SECURITY_CAP_AUDIT);
} }
static inline int security_real_capable(struct task_struct *tsk, int cap) static inline int security_real_capable(struct task_struct *tsk, int cap)
......
...@@ -305,7 +305,7 @@ int capable(int cap) ...@@ -305,7 +305,7 @@ int capable(int cap)
BUG(); BUG();
} }
if (security_capable(cap) == 0) { if (security_capable(current_cred(), cap) == 0) {
current->flags |= PF_SUPERPRIV; current->flags |= PF_SUPERPRIV;
return 1; return 1;
} }
......
...@@ -151,10 +151,9 @@ int security_capset(struct cred *new, const struct cred *old, ...@@ -151,10 +151,9 @@ int security_capset(struct cred *new, const struct cred *old,
effective, inheritable, permitted); effective, inheritable, permitted);
} }
int security_capable(int cap) int security_capable(const struct cred *cred, int cap)
{ {
return security_ops->capable(current, current_cred(), cap, return security_ops->capable(current, cred, cap, SECURITY_CAP_AUDIT);
SECURITY_CAP_AUDIT);
} }
int security_real_capable(struct task_struct *tsk, int cap) int security_real_capable(struct task_struct *tsk, int cap)
......
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