Commit fc7eadf7 authored by Richard Guy Briggs's avatar Richard Guy Briggs Committed by James Morris

capabilities: rename has_cap to has_fcap

Rename has_cap to has_fcap to clarify it applies to file capabilities
since the entire source file is about capabilities.
Signed-off-by: default avatarRichard Guy Briggs <rgb@redhat.com>
Reviewed-by: default avatarSerge Hallyn <serge@hallyn.com>
Acked-by: default avatarJames Morris <james.l.morris@oracle.com>
Acked-by: default avatarKees Cook <keescook@chromium.org>
Okay-ished-by: default avatarPaul Moore <paul@paul-moore.com>
Signed-off-by: default avatarJames Morris <james.l.morris@oracle.com>
parent 4c7e715f
...@@ -536,7 +536,7 @@ int cap_convert_nscap(struct dentry *dentry, void **ivalue, size_t size) ...@@ -536,7 +536,7 @@ int cap_convert_nscap(struct dentry *dentry, void **ivalue, size_t size)
static inline int bprm_caps_from_vfs_caps(struct cpu_vfs_cap_data *caps, static inline int bprm_caps_from_vfs_caps(struct cpu_vfs_cap_data *caps,
struct linux_binprm *bprm, struct linux_binprm *bprm,
bool *effective, bool *effective,
bool *has_cap) bool *has_fcap)
{ {
struct cred *new = bprm->cred; struct cred *new = bprm->cred;
unsigned i; unsigned i;
...@@ -546,7 +546,7 @@ static inline int bprm_caps_from_vfs_caps(struct cpu_vfs_cap_data *caps, ...@@ -546,7 +546,7 @@ static inline int bprm_caps_from_vfs_caps(struct cpu_vfs_cap_data *caps,
*effective = true; *effective = true;
if (caps->magic_etc & VFS_CAP_REVISION_MASK) if (caps->magic_etc & VFS_CAP_REVISION_MASK)
*has_cap = true; *has_fcap = true;
CAP_FOR_EACH_U32(i) { CAP_FOR_EACH_U32(i) {
__u32 permitted = caps->permitted.cap[i]; __u32 permitted = caps->permitted.cap[i];
...@@ -652,7 +652,7 @@ int get_vfs_caps_from_disk(const struct dentry *dentry, struct cpu_vfs_cap_data ...@@ -652,7 +652,7 @@ int get_vfs_caps_from_disk(const struct dentry *dentry, struct cpu_vfs_cap_data
* its xattrs and, if present, apply them to the proposed credentials being * its xattrs and, if present, apply them to the proposed credentials being
* constructed by execve(). * constructed by execve().
*/ */
static int get_file_caps(struct linux_binprm *bprm, bool *effective, bool *has_cap) static int get_file_caps(struct linux_binprm *bprm, bool *effective, bool *has_fcap)
{ {
int rc = 0; int rc = 0;
struct cpu_vfs_cap_data vcaps; struct cpu_vfs_cap_data vcaps;
...@@ -683,7 +683,7 @@ static int get_file_caps(struct linux_binprm *bprm, bool *effective, bool *has_c ...@@ -683,7 +683,7 @@ static int get_file_caps(struct linux_binprm *bprm, bool *effective, bool *has_c
goto out; goto out;
} }
rc = bprm_caps_from_vfs_caps(&vcaps, bprm, effective, has_cap); rc = bprm_caps_from_vfs_caps(&vcaps, bprm, effective, has_fcap);
if (rc == -EINVAL) if (rc == -EINVAL)
printk(KERN_NOTICE "%s: cap_from_disk returned %d for %s\n", printk(KERN_NOTICE "%s: cap_from_disk returned %d for %s\n",
__func__, rc, bprm->filename); __func__, rc, bprm->filename);
...@@ -707,7 +707,7 @@ static int get_file_caps(struct linux_binprm *bprm, bool *effective, bool *has_c ...@@ -707,7 +707,7 @@ static int get_file_caps(struct linux_binprm *bprm, bool *effective, bool *has_c
* set UID root and nothing is changed. If we are root, cap_permitted is * set UID root and nothing is changed. If we are root, cap_permitted is
* updated. If we have become set UID root, the effective bit is set. * updated. If we have become set UID root, the effective bit is set.
*/ */
static void handle_privileged_root(struct linux_binprm *bprm, bool has_cap, static void handle_privileged_root(struct linux_binprm *bprm, bool has_fcap,
bool *effective, kuid_t root_uid) bool *effective, kuid_t root_uid)
{ {
const struct cred *old = current_cred(); const struct cred *old = current_cred();
...@@ -720,7 +720,7 @@ static void handle_privileged_root(struct linux_binprm *bprm, bool has_cap, ...@@ -720,7 +720,7 @@ static void handle_privileged_root(struct linux_binprm *bprm, bool has_cap,
* for a setuid root binary run by a non-root user. Do set it * for a setuid root binary run by a non-root user. Do set it
* for a root user just to cause least surprise to an admin. * for a root user just to cause least surprise to an admin.
*/ */
if (has_cap && !uid_eq(new->uid, root_uid) && uid_eq(new->euid, root_uid)) { if (has_fcap && !uid_eq(new->uid, root_uid) && uid_eq(new->euid, root_uid)) {
warn_setuid_and_fcaps_mixed(bprm->filename); warn_setuid_and_fcaps_mixed(bprm->filename);
return; return;
} }
...@@ -759,20 +759,20 @@ int cap_bprm_set_creds(struct linux_binprm *bprm) ...@@ -759,20 +759,20 @@ int cap_bprm_set_creds(struct linux_binprm *bprm)
{ {
const struct cred *old = current_cred(); const struct cred *old = current_cred();
struct cred *new = bprm->cred; struct cred *new = bprm->cred;
bool effective = false, has_cap = false, is_setid; bool effective = false, has_fcap = false, is_setid;
int ret; int ret;
kuid_t root_uid; kuid_t root_uid;
if (WARN_ON(!cap_ambient_invariant_ok(old))) if (WARN_ON(!cap_ambient_invariant_ok(old)))
return -EPERM; return -EPERM;
ret = get_file_caps(bprm, &effective, &has_cap); ret = get_file_caps(bprm, &effective, &has_fcap);
if (ret < 0) if (ret < 0)
return ret; return ret;
root_uid = make_kuid(new->user_ns, 0); root_uid = make_kuid(new->user_ns, 0);
handle_privileged_root(bprm, has_cap, &effective, root_uid); handle_privileged_root(bprm, has_fcap, &effective, root_uid);
/* if we have fs caps, clear dangerous personality flags */ /* if we have fs caps, clear dangerous personality flags */
if (__cap_gained(permitted, new, old)) if (__cap_gained(permitted, new, old))
...@@ -802,7 +802,7 @@ int cap_bprm_set_creds(struct linux_binprm *bprm) ...@@ -802,7 +802,7 @@ int cap_bprm_set_creds(struct linux_binprm *bprm)
new->sgid = new->fsgid = new->egid; new->sgid = new->fsgid = new->egid;
/* File caps or setid cancels ambient. */ /* File caps or setid cancels ambient. */
if (has_cap || is_setid) if (has_fcap || is_setid)
cap_clear(new->cap_ambient); cap_clear(new->cap_ambient);
/* /*
......
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