selinux/stable-5.6 PR 20200210
-----BEGIN PGP SIGNATURE----- iQJIBAABCAAyFiEES0KozwfymdVUl37v6iDy2pc3iXMFAl5ByJYUHHBhdWxAcGF1 bC1tb29yZS5jb20ACgkQ6iDy2pc3iXMNLg/+Nj3gdWa5z0lsFuvUs87uM4HYHLtp 9/wENEaqy7W92LAAEW5y+6sK8Pm9/CO3xf2d40SAr/xjwBo2Fbnfmx7QwDMUCQS6 vyrLDiikSMyFU5j1AQDFuWjWF36FZlfqix+oXMETJ1tqxjIFxFN+rvbsDICPiQTl VxBG4CWWzNtSllYDDj3KYcVaGzNb3nleb8n436gFoHS++12BaypVA0kBlV3BqzPi mC0+gDuY06hwFmc/tAl8WndRwZpJ71rgsKJ5opgel61Zxf1J39QdTxeap9hhldJl 5FbtoiGpnMXtHzpRGh6BHag/2gGX/0+J+t8ZATuk4GRtt1ueyYw9kNnAMblooJSV TwJlv1LLIKpAmSSJnKoN1AUAsxhS3dAmVyPuYWtmqEq8wq7YEYi5UnK1fH4ziI+b 5LmYD0D/FoNE1Dr1TV78HmM9i0+AteH5FhkS0V6GD3v+wO6vCb3hTYaQpMWhhgnY /SEWbvLUO1fvQoKA8GT3UtmiUFdrqvrQFoL/l3weo9tvfyG3Walxo8s0w0r/DVUH AevIhUVi+snSLhI95fF26wskrZtEE9gwl/BoMjfKse0HK2t77xBE3kZW88NnSXOU Wk+ozfgibCnzk/Do/Y4iRr9GbgmCfscjOqlq1Yzyt/ZkUsG2egWHMzpritnzVEio NSQ6n6kpbkRgkHc= =0cTj -----END PGP SIGNATURE----- Merge tag 'selinux-pr-20200210' of git://git.kernel.org/pub/scm/linux/kernel/git/pcmoore/selinux Pull SELinux fixes from Paul Moore: "Two small fixes: one fixes a locking problem in the recently merged label translation code, the other fixes an embarrassing 'binderfs' / 'binder' filesystem name check" * tag 'selinux-pr-20200210' of git://git.kernel.org/pub/scm/linux/kernel/git/pcmoore/selinux: selinux: fix sidtab string cache locking selinux: fix typo in filesystem name
This commit is contained in:
commit
a5650acb5f
|
@ -698,7 +698,7 @@ static int selinux_set_mnt_opts(struct super_block *sb,
|
|||
|
||||
if (!strcmp(sb->s_type->name, "debugfs") ||
|
||||
!strcmp(sb->s_type->name, "tracefs") ||
|
||||
!strcmp(sb->s_type->name, "binderfs") ||
|
||||
!strcmp(sb->s_type->name, "binder") ||
|
||||
!strcmp(sb->s_type->name, "pstore"))
|
||||
sbsec->flags |= SE_SBGENFS;
|
||||
|
||||
|
|
|
@ -518,19 +518,13 @@ void sidtab_sid2str_put(struct sidtab *s, struct sidtab_entry *entry,
|
|||
const char *str, u32 str_len)
|
||||
{
|
||||
struct sidtab_str_cache *cache, *victim = NULL;
|
||||
unsigned long flags;
|
||||
|
||||
/* do not cache invalid contexts */
|
||||
if (entry->context.len)
|
||||
return;
|
||||
|
||||
/*
|
||||
* Skip the put operation when in non-task context to avoid the need
|
||||
* to disable interrupts while holding s->cache_lock.
|
||||
*/
|
||||
if (!in_task())
|
||||
return;
|
||||
|
||||
spin_lock(&s->cache_lock);
|
||||
spin_lock_irqsave(&s->cache_lock, flags);
|
||||
|
||||
cache = rcu_dereference_protected(entry->cache,
|
||||
lockdep_is_held(&s->cache_lock));
|
||||
|
@ -561,7 +555,7 @@ void sidtab_sid2str_put(struct sidtab *s, struct sidtab_entry *entry,
|
|||
rcu_assign_pointer(entry->cache, cache);
|
||||
|
||||
out_unlock:
|
||||
spin_unlock(&s->cache_lock);
|
||||
spin_unlock_irqrestore(&s->cache_lock, flags);
|
||||
kfree_rcu(victim, rcu_member);
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in New Issue
Block a user