--- v14
+++ v26
@@ -1,469 +1,263 @@
-Create a new entry "display" in the procfs attr directory for
-controlling which LSM security information is displayed for a
-process. A process can only read or write its own display value.
+The IMA interfaces ima_get_action() and ima_match_policy()
+call LSM functions that use lsmblobs. Change the IMA functions
+to pass the lsmblob to be compatible with the LSM functions.
-The name of an active LSM that supplies hooks for
-human readable data may be written to "display" to set the
-value. The name of the LSM currently in use can be read from
-"display". At this point there can only be one LSM capable
-of display active. A helper function lsm_task_display() is
-provided to get the display slot for a task_struct.
-
-Setting the "display" requires that all security modules using
-setprocattr hooks allow the action. Each security module is
-responsible for defining its policy.
-
-AppArmor hook provided by John Johansen <john.johansen@canonical.com>
-SELinux hook provided by Stephen Smalley <sds@tycho.nsa.gov>
-
+Reviewed-by: Kees Cook <keescook@chromium.org>
+Reviewed-by: John Johansen <john.johansen@canonical.com>
Acked-by: Stephen Smalley <sds@tycho.nsa.gov>
Signed-off-by: Casey Schaufler <casey@schaufler-ca.com>
+Cc: linux-integrity@vger.kernel.org
+To: Mimi Zohar <zohar@linux.ibm.com>
---
- fs/proc/base.c | 1 +
- include/linux/lsm_hooks.h | 15 +++
- security/apparmor/include/apparmor.h | 3 +-
- security/apparmor/lsm.c | 32 +++++
- security/security.c | 167 ++++++++++++++++++++++++---
- security/selinux/hooks.c | 11 ++
- security/selinux/include/classmap.h | 2 +-
- security/smack/smack_lsm.c | 7 ++
- 8 files changed, 219 insertions(+), 19 deletions(-)
+ security/integrity/ima/ima.h | 6 ++---
+ security/integrity/ima/ima_api.c | 6 ++---
+ security/integrity/ima/ima_appraise.c | 5 ++--
+ security/integrity/ima/ima_main.c | 36 +++++++++++----------------
+ security/integrity/ima/ima_policy.c | 17 ++++++-------
+ 5 files changed, 31 insertions(+), 39 deletions(-)
-diff --git a/fs/proc/base.c b/fs/proc/base.c
-index ebea9501afb8..950c200cb9ad 100644
---- a/fs/proc/base.c
-+++ b/fs/proc/base.c
-@@ -2652,6 +2652,7 @@ static const struct pid_entry attr_dir_stuff[] = {
- ATTR(NULL, "fscreate", 0666),
- ATTR(NULL, "keycreate", 0666),
- ATTR(NULL, "sockcreate", 0666),
-+ ATTR(NULL, "display", 0666),
- #ifdef CONFIG_SECURITY_SMACK
- DIR("smack", 0555,
- proc_smack_attr_dir_inode_ops, proc_smack_attr_dir_ops),
-diff --git a/include/linux/lsm_hooks.h b/include/linux/lsm_hooks.h
-index 7eb808cde051..2bf82e1cf347 100644
---- a/include/linux/lsm_hooks.h
-+++ b/include/linux/lsm_hooks.h
-@@ -2186,4 +2186,19 @@ static inline void security_delete_hooks(struct security_hook_list *hooks,
-
- extern int lsm_inode_alloc(struct inode *inode);
-
-+/**
-+ * lsm_task_display - the "display" LSM for this task
-+ * @task: The task to report on
-+ *
-+ * Returns the task's display LSM slot.
-+ */
-+static inline int lsm_task_display(struct task_struct *task)
-+{
-+ int *display = task->security;
-+
-+ if (display)
-+ return *display;
-+ return LSMBLOB_INVALID;
-+}
-+
- #endif /* ! __LINUX_LSM_HOOKS_H */
-diff --git a/security/apparmor/include/apparmor.h b/security/apparmor/include/apparmor.h
-index 1fbabdb565a8..b1622fcb4394 100644
---- a/security/apparmor/include/apparmor.h
-+++ b/security/apparmor/include/apparmor.h
-@@ -28,8 +28,9 @@
- #define AA_CLASS_SIGNAL 10
- #define AA_CLASS_NET 14
- #define AA_CLASS_LABEL 16
-+#define AA_CLASS_DISPLAY_LSM 17
-
--#define AA_CLASS_LAST AA_CLASS_LABEL
-+#define AA_CLASS_LAST AA_CLASS_DISPLAY_LSM
-
- /* Control parameters settable through module/boot flags */
- extern enum audit_mode aa_g_audit;
-diff --git a/security/apparmor/lsm.c b/security/apparmor/lsm.c
-index 146d75e5e021..16b992235c11 100644
---- a/security/apparmor/lsm.c
-+++ b/security/apparmor/lsm.c
-@@ -612,6 +612,25 @@ static int apparmor_getprocattr(struct task_struct *task, char *name,
- return error;
- }
-
-+
-+static int profile_display_lsm(struct aa_profile *profile,
-+ struct common_audit_data *sa)
-+{
-+ struct aa_perms perms = { };
-+ unsigned int state;
-+
-+ state = PROFILE_MEDIATES(profile, AA_CLASS_DISPLAY_LSM);
-+ if (state) {
-+ aa_compute_perms(profile->policy.dfa, state, &perms);
-+ aa_apply_modes_to_perms(profile, &perms);
-+ aad(sa)->label = &profile->label;
-+
-+ return aa_check_perms(profile, &perms, AA_MAY_WRITE, sa, NULL);
-+ }
-+
-+ return 0;
-+}
-+
- static int apparmor_setprocattr(const char *name, void *value,
- size_t size)
+diff --git a/security/integrity/ima/ima.h b/security/integrity/ima/ima.h
+index 55f3bd4f0b01..a6b59fcaf62a 100644
+--- a/security/integrity/ima/ima.h
++++ b/security/integrity/ima/ima.h
+@@ -251,7 +251,7 @@ static inline void ima_process_queued_keys(void) {}
+
+ /* LIM API function definitions */
+ int ima_get_action(struct user_namespace *mnt_userns, struct inode *inode,
+- const struct cred *cred, u32 secid, int mask,
++ const struct cred *cred, struct lsmblob *blob, int mask,
+ enum ima_hooks func, int *pcr,
+ struct ima_template_desc **template_desc,
+ const char *func_data);
+@@ -282,8 +282,8 @@ const char *ima_d_path(const struct path *path, char **pathbuf, char *filename);
+
+ /* IMA policy related functions */
+ int ima_match_policy(struct user_namespace *mnt_userns, struct inode *inode,
+- const struct cred *cred, u32 secid, enum ima_hooks func,
+- int mask, int flags, int *pcr,
++ const struct cred *cred, struct lsmblob *blob,
++ enum ima_hooks func, int mask, int flags, int *pcr,
+ struct ima_template_desc **template_desc,
+ const char *func_data);
+ void ima_init_policy(void);
+diff --git a/security/integrity/ima/ima_api.c b/security/integrity/ima/ima_api.c
+index d8e321cc6936..691f68d478f1 100644
+--- a/security/integrity/ima/ima_api.c
++++ b/security/integrity/ima/ima_api.c
+@@ -165,7 +165,7 @@ void ima_add_violation(struct file *file, const unsigned char *filename,
+ * @mnt_userns: user namespace of the mount the inode was found from
+ * @inode: pointer to the inode associated with the object being validated
+ * @cred: pointer to credentials structure to validate
+- * @secid: secid of the task being validated
++ * @blob: LSM data of the task being validated
+ * @mask: contains the permission mask (MAY_READ, MAY_WRITE, MAY_EXEC,
+ * MAY_APPEND)
+ * @func: caller identifier
+@@ -185,7 +185,7 @@ void ima_add_violation(struct file *file, const unsigned char *filename,
+ *
+ */
+ int ima_get_action(struct user_namespace *mnt_userns, struct inode *inode,
+- const struct cred *cred, u32 secid, int mask,
++ const struct cred *cred, struct lsmblob *blob, int mask,
+ enum ima_hooks func, int *pcr,
+ struct ima_template_desc **template_desc,
+ const char *func_data)
+@@ -194,7 +194,7 @@ int ima_get_action(struct user_namespace *mnt_userns, struct inode *inode,
+
+ flags &= ima_policy_flag;
+
+- return ima_match_policy(mnt_userns, inode, cred, secid, func, mask,
++ return ima_match_policy(mnt_userns, inode, cred, blob, func, mask,
+ flags, pcr, template_desc, func_data);
+ }
+
+diff --git a/security/integrity/ima/ima_appraise.c b/security/integrity/ima/ima_appraise.c
+index f8c7b593175f..b2af72289f00 100644
+--- a/security/integrity/ima/ima_appraise.c
++++ b/security/integrity/ima/ima_appraise.c
+@@ -77,10 +77,9 @@ int ima_must_appraise(struct user_namespace *mnt_userns, struct inode *inode,
+ return 0;
+
+ security_task_getsecid_subj(current, &blob);
+- /* scaffolding the .secid[0] */
+ return ima_match_policy(mnt_userns, inode, current_cred(),
+- blob.secid[0], func, mask,
+- IMA_APPRAISE | IMA_HASH, NULL, NULL, NULL);
++ &blob, func, mask, IMA_APPRAISE | IMA_HASH,
++ NULL, NULL, NULL);
+ }
+
+ static int ima_fix_xattr(struct dentry *dentry,
+diff --git a/security/integrity/ima/ima_main.c b/security/integrity/ima/ima_main.c
+index b3e00340a97c..b63f73d43bd2 100644
+--- a/security/integrity/ima/ima_main.c
++++ b/security/integrity/ima/ima_main.c
+@@ -194,8 +194,8 @@ void ima_file_free(struct file *file)
+ }
+
+ static int process_measurement(struct file *file, const struct cred *cred,
+- u32 secid, char *buf, loff_t size, int mask,
+- enum ima_hooks func)
++ struct lsmblob *blob, char *buf, loff_t size,
++ int mask, enum ima_hooks func)
{
-@@ -623,6 +642,19 @@ static int apparmor_setprocattr(const char *name, void *value,
- if (size == 0)
- return -EINVAL;
-
-+ /* LSM infrastructure does actual setting of display if allowed */
-+ if (!strcmp(name, "display")) {
-+ struct aa_profile *profile;
-+ struct aa_label *label;
-+
-+ aad(&sa)->info = "set display lsm";
-+ label = begin_current_label_crit_section();
-+ error = fn_for_each_confined(label, profile,
-+ profile_display_lsm(profile, &sa));
-+ end_current_label_crit_section(label);
-+ return error;
-+ }
-+
- /* AppArmor requires that the buffer must be null terminated atm */
- if (args[size - 1] != '\0') {
- /* null terminate */
-diff --git a/security/security.c b/security/security.c
-index 100e55777426..80ff19ba62e5 100644
---- a/security/security.c
-+++ b/security/security.c
-@@ -43,7 +43,14 @@ static struct kmem_cache *lsm_file_cache;
- static struct kmem_cache *lsm_inode_cache;
-
- char *lsm_names;
--static struct lsm_blob_sizes blob_sizes __lsm_ro_after_init;
-+
-+/*
-+ * The task blob includes the "display" slot used for
-+ * chosing which module presents contexts.
-+ */
-+static struct lsm_blob_sizes blob_sizes __lsm_ro_after_init = {
-+ .lbs_task = sizeof(int),
-+};
-
- /* Boot-time LSM user choice */
- static __initdata const char *chosen_lsm_order;
-@@ -438,8 +445,10 @@ static int lsm_append(const char *new, char **result)
-
- /*
- * Current index to use while initializing the lsmblob secid list.
-+ * Pointers to the LSM id structures for local use.
+ struct inode *inode = file_inode(file);
+ struct integrity_iint_cache *iint = NULL;
+@@ -218,7 +218,7 @@ static int process_measurement(struct file *file, const struct cred *cred,
+ * bitmask based on the appraise/audit/measurement policy.
+ * Included is the appraise submask.
+ */
+- action = ima_get_action(file_mnt_user_ns(file), inode, cred, secid,
++ action = ima_get_action(file_mnt_user_ns(file), inode, cred, blob,
+ mask, func, &pcr, &template_desc, NULL);
+ violation_check = ((func == FILE_CHECK || func == MMAP_CHECK) &&
+ (ima_policy_flag & IMA_MEASURE));
+@@ -392,8 +392,7 @@ int ima_file_mmap(struct file *file, unsigned long prot)
+
+ if (file && (prot & PROT_EXEC)) {
+ security_task_getsecid_subj(current, &blob);
+- /* scaffolding - until process_measurement changes */
+- return process_measurement(file, current_cred(), blob.secid[0],
++ return process_measurement(file, current_cred(), &blob,
+ NULL, 0, MAY_EXEC, MMAP_CHECK);
+ }
+
+@@ -434,7 +433,7 @@ int ima_file_mprotect(struct vm_area_struct *vma, unsigned long prot)
+ inode = file_inode(vma->vm_file);
+ /* scaffolding */
+ action = ima_get_action(file_mnt_user_ns(vma->vm_file), inode,
+- current_cred(), blob.secid[0], MAY_EXEC,
++ current_cred(), &blob, MAY_EXEC,
+ MMAP_CHECK, &pcr, &template, 0);
+
+ /* Is the mmap'ed file in policy? */
+@@ -473,16 +472,14 @@ int ima_bprm_check(struct linux_binprm *bprm)
+ struct lsmblob blob;
+
+ security_task_getsecid_subj(current, &blob);
+- /* scaffolding until process_measurement changes */
+- ret = process_measurement(bprm->file, current_cred(), blob.secid[0],
+- NULL, 0, MAY_EXEC, BPRM_CHECK);
++ ret = process_measurement(bprm->file, current_cred(), &blob, NULL, 0,
++ MAY_EXEC, BPRM_CHECK);
+ if (ret)
+ return ret;
+
+ security_cred_getsecid(bprm->cred, &blob);
+- /* scaffolding until process_measurement changes */
+- return process_measurement(bprm->file, bprm->cred, blob.secid[0],
+- NULL, 0, MAY_EXEC, CREDS_CHECK);
++ return process_measurement(bprm->file, bprm->cred, &blob, NULL, 0,
++ MAY_EXEC, CREDS_CHECK);
+ }
+
+ /**
+@@ -500,8 +497,7 @@ int ima_file_check(struct file *file, int mask)
+ struct lsmblob blob;
+
+ security_task_getsecid_subj(current, &blob);
+- /* scaffolding until process_measurement changes */
+- return process_measurement(file, current_cred(), blob.secid[0], NULL, 0,
++ return process_measurement(file, current_cred(), &blob, NULL, 0,
+ mask & (MAY_READ | MAY_WRITE | MAY_EXEC |
+ MAY_APPEND), FILE_CHECK);
+ }
+@@ -698,9 +694,8 @@ int ima_read_file(struct file *file, enum kernel_read_file_id read_id,
+ /* Read entire file for all partial reads. */
+ func = read_idmap[read_id] ?: FILE_CHECK;
+ security_task_getsecid_subj(current, &blob);
+- /* scaffolding - until process_measurement changes */
+- return process_measurement(file, current_cred(), blob.secid[0], NULL,
+- 0, MAY_READ, func);
++ return process_measurement(file, current_cred(), &blob, NULL, 0,
++ MAY_READ, func);
+ }
+
+ const int read_idmap[READING_MAX_ID] = {
+@@ -742,9 +737,8 @@ int ima_post_read_file(struct file *file, void *buf, loff_t size,
+
+ func = read_idmap[read_id] ?: FILE_CHECK;
+ security_task_getsecid_subj(current, &blob);
+- /* scaffolding until process_measurement changes */
+- return process_measurement(file, current_cred(), blob.secid[0], buf,
+- size, MAY_READ, func);
++ return process_measurement(file, current_cred(), &blob, buf, size,
++ MAY_READ, func);
+ }
+
+ /**
+@@ -889,7 +883,7 @@ void process_buffer_measurement(struct user_namespace *mnt_userns,
+ security_task_getsecid_subj(current, &blob);
+ /* scaffolding */
+ action = ima_get_action(mnt_userns, inode, current_cred(),
+- blob.secid[0], 0, func, &pcr, &template,
++ &blob, 0, func, &pcr, &template,
+ func_data);
+ if (!(action & IMA_MEASURE))
+ return;
+diff --git a/security/integrity/ima/ima_policy.c b/security/integrity/ima/ima_policy.c
+index 5ee7629fd782..caacd8bf0462 100644
+--- a/security/integrity/ima/ima_policy.c
++++ b/security/integrity/ima/ima_policy.c
+@@ -546,7 +546,7 @@ static bool ima_match_rule_data(struct ima_rule_entry *rule,
+ * @mnt_userns: user namespace of the mount the inode was found from
+ * @inode: a pointer to an inode
+ * @cred: a pointer to a credentials structure for user validation
+- * @secid: the secid of the task to be validated
++ * @blob: the lsm data of the task to be validated
+ * @func: LIM hook identifier
+ * @mask: requested action (MAY_READ | MAY_WRITE | MAY_APPEND | MAY_EXEC)
+ * @func_data: func specific data, may be NULL
+@@ -556,8 +556,8 @@ static bool ima_match_rule_data(struct ima_rule_entry *rule,
+ static bool ima_match_rules(struct ima_rule_entry *rule,
+ struct user_namespace *mnt_userns,
+ struct inode *inode, const struct cred *cred,
+- u32 secid, enum ima_hooks func, int mask,
+- const char *func_data)
++ struct lsmblob *blob, enum ima_hooks func,
++ int mask, const char *func_data)
+ {
+ int i;
+
+@@ -626,8 +626,7 @@ static bool ima_match_rules(struct ima_rule_entry *rule,
+ case LSM_SUBJ_USER:
+ case LSM_SUBJ_ROLE:
+ case LSM_SUBJ_TYPE:
+- lsmblob_init(&lsmdata, secid);
+- rc = ima_filter_rule_match(&lsmdata, rule->lsm[i].type,
++ rc = ima_filter_rule_match(blob, rule->lsm[i].type,
+ Audit_equal,
+ rule->lsm[i].rule);
+ break;
+@@ -671,7 +670,7 @@ static int get_subaction(struct ima_rule_entry *rule, enum ima_hooks func)
+ * @inode: pointer to an inode for which the policy decision is being made
+ * @cred: pointer to a credentials structure for which the policy decision is
+ * being made
+- * @secid: LSM secid of the task to be validated
++ * @blob: LSM data of the task to be validated
+ * @func: IMA hook identifier
+ * @mask: requested action (MAY_READ | MAY_WRITE | MAY_APPEND | MAY_EXEC)
+ * @pcr: set the pcr to extend
+@@ -686,8 +685,8 @@ static int get_subaction(struct ima_rule_entry *rule, enum ima_hooks func)
+ * than writes so ima_match_policy() is classical RCU candidate.
*/
- static int lsm_slot __lsm_ro_after_init;
-+static struct lsm_id *lsm_slotlist[LSMBLOB_ENTRIES];
-
- /**
- * security_add_hooks - Add a modules hooks to the hook lists.
-@@ -459,6 +468,7 @@ void __init security_add_hooks(struct security_hook_list *hooks, int count,
- if (lsmid->slot == LSMBLOB_NEEDED) {
- if (lsm_slot >= LSMBLOB_ENTRIES)
- panic("%s Too many LSMs registered.\n", __func__);
-+ lsm_slotlist[lsm_slot] = lsmid;
- lsmid->slot = lsm_slot++;
- init_debug("%s assigned lsmblob slot %d\n", lsmid->lsm,
- lsmid->slot);
-@@ -588,6 +598,8 @@ int lsm_inode_alloc(struct inode *inode)
- */
- static int lsm_task_alloc(struct task_struct *task)
+ int ima_match_policy(struct user_namespace *mnt_userns, struct inode *inode,
+- const struct cred *cred, u32 secid, enum ima_hooks func,
+- int mask, int flags, int *pcr,
++ const struct cred *cred, struct lsmblob *blob,
++ enum ima_hooks func, int mask, int flags, int *pcr,
+ struct ima_template_desc **template_desc,
+ const char *func_data)
{
-+ int *display;
-+
- if (blob_sizes.lbs_task == 0) {
- task->security = NULL;
- return 0;
-@@ -596,6 +608,15 @@ static int lsm_task_alloc(struct task_struct *task)
- task->security = kzalloc(blob_sizes.lbs_task, GFP_KERNEL);
- if (task->security == NULL)
- return -ENOMEM;
-+
-+ /*
-+ * The start of the task blob contains the "display" LSM slot number.
-+ * Start with it set to the invalid slot number, indicating that the
-+ * default first registered LSM be displayed.
-+ */
-+ display = task->security;
-+ *display = LSMBLOB_INVALID;
-+
- return 0;
- }
-
-@@ -1551,14 +1572,26 @@ int security_file_open(struct file *file)
-
- int security_task_alloc(struct task_struct *task, unsigned long clone_flags)
- {
-+ int *odisplay = current->security;
-+ int *ndisplay;
- int rc = lsm_task_alloc(task);
-
-- if (rc)
-+ if (unlikely(rc))
- return rc;
-+
- rc = call_int_hook(task_alloc, 0, task, clone_flags);
-- if (unlikely(rc))
-+ if (unlikely(rc)) {
- security_task_free(task);
-- return rc;
-+ return rc;
-+ }
-+
-+ if (odisplay) {
-+ ndisplay = task->security;
-+ if (ndisplay)
-+ *ndisplay = *odisplay;
-+ }
-+
-+ return 0;
- }
-
- void security_task_free(struct task_struct *task)
-@@ -1955,23 +1988,110 @@ int security_getprocattr(struct task_struct *p, const char *lsm, char *name,
- char **value)
- {
- struct security_hook_list *hp;
-+ int display = lsm_task_display(current);
-+ int slot = 0;
-+
-+ if (!strcmp(name, "display")) {
-+ /*
-+ * lsm_slot will be 0 if there are no displaying modules.
-+ */
-+ if (lsm_slot == 0)
-+ return -EINVAL;
-+
-+ /*
-+ * Only allow getting the current process' display.
-+ * There are too few reasons to get another process'
-+ * display and too many LSM policy issues.
-+ */
-+ if (current != p)
-+ return -EINVAL;
-+
-+ display = lsm_task_display(p);
-+ if (display != LSMBLOB_INVALID)
-+ slot = display;
-+ *value = kstrdup(lsm_slotlist[slot]->lsm, GFP_KERNEL);
-+ if (*value)
-+ return strlen(*value);
-+ return -ENOMEM;
-+ }
-
- hlist_for_each_entry(hp, &security_hook_heads.getprocattr, list) {
- if (lsm != NULL && strcmp(lsm, hp->lsmid->lsm))
+@@ -703,7 +702,7 @@ int ima_match_policy(struct user_namespace *mnt_userns, struct inode *inode,
+ if (!(entry->action & actmask))
continue;
-+ if (lsm == NULL && display != LSMBLOB_INVALID &&
-+ display != hp->lsmid->slot)
-+ continue;
- return hp->hook.getprocattr(p, name, value);
- }
- return -EINVAL;
- }
-
-+/**
-+ * security_setprocattr - Set process attributes via /proc
-+ * @lsm: name of module involved, or NULL
-+ * @name: name of the attribute
-+ * @value: value to set the attribute to
-+ * @size: size of the value
-+ *
-+ * Set the process attribute for the specified security module
-+ * to the specified value. Note that this can only be used to set
-+ * the process attributes for the current, or "self" process.
-+ * The /proc code has already done this check.
-+ *
-+ * Returns 0 on success, an appropriate code otherwise.
-+ */
- int security_setprocattr(const char *lsm, const char *name, void *value,
- size_t size)
- {
- struct security_hook_list *hp;
-+ char *termed;
-+ char *copy;
-+ int *display = current->security;
-+ int rc = -EINVAL;
-+ int slot = 0;
-+
-+ if (!strcmp(name, "display")) {
-+ /*
-+ * Change the "display" value only if all the security
-+ * modules that support setting a procattr allow it.
-+ * It is assumed that all such security modules will be
-+ * cooperative.
-+ */
-+ if (size == 0)
-+ return -EINVAL;
-+
-+ hlist_for_each_entry(hp, &security_hook_heads.setprocattr,
-+ list) {
-+ rc = hp->hook.setprocattr(name, value, size);
-+ if (rc < 0)
-+ return rc;
-+ }
-+
-+ rc = -EINVAL;
-+
-+ copy = kmemdup_nul(value, size, GFP_KERNEL);
-+ if (copy == NULL)
-+ return -ENOMEM;
-+
-+ termed = strsep(©, " \n");
-+
-+ for (slot = 0; slot < lsm_slot; slot++)
-+ if (!strcmp(termed, lsm_slotlist[slot]->lsm)) {
-+ *display = lsm_slotlist[slot]->slot;
-+ rc = size;
-+ break;
-+ }
-+
-+ kfree(copy);
-+ return rc;
-+ }
-
- hlist_for_each_entry(hp, &security_hook_heads.setprocattr, list) {
- if (lsm != NULL && strcmp(lsm, hp->lsmid->lsm))
+
+- if (!ima_match_rules(entry, mnt_userns, inode, cred, secid,
++ if (!ima_match_rules(entry, mnt_userns, inode, cred, blob,
+ func, mask, func_data))
continue;
-+ if (lsm == NULL && *display != LSMBLOB_INVALID &&
-+ *display != hp->lsmid->slot)
-+ continue;
- return hp->hook.setprocattr(name, value, size);
- }
- return -EINVAL;
-@@ -1991,15 +2111,15 @@ EXPORT_SYMBOL(security_ismaclabel);
- int security_secid_to_secctx(struct lsmblob *blob, char **secdata, u32 *seclen)
- {
- struct security_hook_list *hp;
-- int rc;
-+ int display = lsm_task_display(current);
-
- hlist_for_each_entry(hp, &security_hook_heads.secid_to_secctx, list) {
- if (WARN_ON(hp->lsmid->slot < 0 || hp->lsmid->slot >= lsm_slot))
- continue;
-- rc = hp->hook.secid_to_secctx(blob->secid[hp->lsmid->slot],
-- secdata, seclen);
-- if (rc != 0)
-- return rc;
-+ if (display == LSMBLOB_INVALID || display == hp->lsmid->slot)
-+ return hp->hook.secid_to_secctx(
-+ blob->secid[hp->lsmid->slot],
-+ secdata, seclen);
- }
- return 0;
- }
-@@ -2009,16 +2129,15 @@ int security_secctx_to_secid(const char *secdata, u32 seclen,
- struct lsmblob *blob)
- {
- struct security_hook_list *hp;
-- int rc;
-+ int display = lsm_task_display(current);
-
- lsmblob_init(blob, 0);
- hlist_for_each_entry(hp, &security_hook_heads.secctx_to_secid, list) {
- if (WARN_ON(hp->lsmid->slot < 0 || hp->lsmid->slot >= lsm_slot))
- continue;
-- rc = hp->hook.secctx_to_secid(secdata, seclen,
-- &blob->secid[hp->lsmid->slot]);
-- if (rc != 0)
-- return rc;
-+ if (display == LSMBLOB_INVALID || display == hp->lsmid->slot)
-+ return hp->hook.secctx_to_secid(secdata, seclen,
-+ &blob->secid[hp->lsmid->slot]);
- }
- return 0;
- }
-@@ -2026,7 +2145,14 @@ EXPORT_SYMBOL(security_secctx_to_secid);
-
- void security_release_secctx(char *secdata, u32 seclen)
- {
-- call_void_hook(release_secctx, secdata, seclen);
-+ struct security_hook_list *hp;
-+ int display = lsm_task_display(current);
-+
-+ hlist_for_each_entry(hp, &security_hook_heads.release_secctx, list)
-+ if (display == LSMBLOB_INVALID || display == hp->lsmid->slot) {
-+ hp->hook.release_secctx(secdata, seclen);
-+ return;
-+ }
- }
- EXPORT_SYMBOL(security_release_secctx);
-
-@@ -2151,8 +2277,15 @@ EXPORT_SYMBOL(security_sock_rcv_skb);
- int security_socket_getpeersec_stream(struct socket *sock, char __user *optval,
- int __user *optlen, unsigned len)
- {
-- return call_int_hook(socket_getpeersec_stream, -ENOPROTOOPT, sock,
-- optval, optlen, len);
-+ int display = lsm_task_display(current);
-+ struct security_hook_list *hp;
-+
-+ hlist_for_each_entry(hp, &security_hook_heads.socket_getpeersec_stream,
-+ list)
-+ if (display == LSMBLOB_INVALID || display == hp->lsmid->slot)
-+ return hp->hook.socket_getpeersec_stream(sock, optval,
-+ optlen, len);
-+ return -ENOPROTOOPT;
- }
-
- int security_socket_getpeersec_dgram(struct socket *sock, struct sk_buff *skb,
-diff --git a/security/selinux/hooks.c b/security/selinux/hooks.c
-index 97f2ee6e4080..b8501ca3c8f3 100644
---- a/security/selinux/hooks.c
-+++ b/security/selinux/hooks.c
-@@ -6323,6 +6323,17 @@ static int selinux_setprocattr(const char *name, void *value, size_t size)
- /*
- * Basic control over ability to set these attributes at all.
- */
-+
-+ /*
-+ * For setting display, we only perform a permission check;
-+ * the actual update to the display value is handled by the
-+ * LSM framework.
-+ */
-+ if (!strcmp(name, "display"))
-+ return avc_has_perm(&selinux_state,
-+ mysid, mysid, SECCLASS_PROCESS2,
-+ PROCESS2__SETDISPLAY, NULL);
-+
- if (!strcmp(name, "exec"))
- error = avc_has_perm(&selinux_state,
- mysid, mysid, SECCLASS_PROCESS,
-diff --git a/security/selinux/include/classmap.h b/security/selinux/include/classmap.h
-index 7db24855e12d..323da8a38c43 100644
---- a/security/selinux/include/classmap.h
-+++ b/security/selinux/include/classmap.h
-@@ -52,7 +52,7 @@ struct security_class_mapping secclass_map[] = {
- "execmem", "execstack", "execheap", "setkeycreate",
- "setsockcreate", "getrlimit", NULL } },
- { "process2",
-- { "nnp_transition", "nosuid_transition", NULL } },
-+ { "nnp_transition", "nosuid_transition", "setdisplay", NULL } },
- { "system",
- { "ipc_info", "syslog_read", "syslog_mod",
- "syslog_console", "module_request", "module_load", NULL } },
-diff --git a/security/smack/smack_lsm.c b/security/smack/smack_lsm.c
-index 82cbb3eeec76..9737ead06b39 100644
---- a/security/smack/smack_lsm.c
-+++ b/security/smack/smack_lsm.c
-@@ -3518,6 +3518,13 @@ static int smack_setprocattr(const char *name, void *value, size_t size)
- struct smack_known_list_elem *sklep;
- int rc;
-
-+ /*
-+ * Allow the /proc/.../attr/current and SO_PEERSEC "display"
-+ * to be reset at will.
-+ */
-+ if (strcmp(name, "display") == 0)
-+ return 0;
-+
- if (!smack_privileged(CAP_MAC_ADMIN) && list_empty(&tsp->smk_relabel))
- return -EPERM;
--
-2.24.1
+2.29.2