selinux: revert 102aefdda4d8275ce7d7100bc16c88c74272b260
authorPaul Moore <pmoore@redhat.com>
Fri, 13 Dec 2013 19:49:53 +0000 (14:49 -0500)
committerPaul Moore <pmoore@redhat.com>
Fri, 13 Dec 2013 19:52:25 +0000 (14:52 -0500)
commit4d546f81717d253ab67643bf072c6d8821a9249c
tree2bfae32c5e42b7b57cab82efbe71f522db79af60
parent598cdbcf861825692fe7905e0fd662c7d06bae58
selinux: revert 102aefdda4d8275ce7d7100bc16c88c74272b260

Revert "selinux: consider filesystem subtype in policies"

This reverts commit 102aefdda4d8275ce7d7100bc16c88c74272b260.

Explanation from Eric Paris:

SELinux policy can specify if it should use a filesystem's
xattrs or not.  In current policy we have a specification that
fuse should not use xattrs but fuse.glusterfs should use
xattrs.  This patch has a bug in which non-glusterfs
filesystems would match the rule saying fuse.glusterfs should
use xattrs.  If both fuse and the particular filesystem in
question are not written to handle xattr calls during the mount
command, they will deadlock.

I have fixed the bug to do proper matching, however I believe a
revert is still the correct solution.  The reason I believe
that is because the code still does not work.  The s_subtype is
not set until after the SELinux hook which attempts to match on
the ".gluster" portion of the rule.  So we cannot match on the
rule in question.  The code is useless.

Signed-off-by: Paul Moore <pmoore@redhat.com>
security/selinux/hooks.c
security/selinux/ss/services.c