OSDN Git Service

selinux: policydb - fix byte order and alignment issues
authorOndrej Mosnacek <omosnace@redhat.com>
Tue, 23 Oct 2018 07:02:17 +0000 (09:02 +0200)
committerPaul Moore <paul@paul-moore.com>
Mon, 5 Nov 2018 20:25:50 +0000 (15:25 -0500)
commit5df275cd4cf51c86d49009f1397132f284ba515e
tree7670bfeb837a0449796067ab3e5ef154888fb569
parent651022382c7f8da46cb4872a545ee1da6d097d2a
selinux: policydb - fix byte order and alignment issues

Do the LE conversions before doing the Infiniband-related range checks.
The incorrect checks are otherwise causing a failure to load any policy
with an ibendportcon rule on BE systems. This can be reproduced by
running (on e.g. ppc64):

cat >my_module.cil <<EOF
(type test_ibendport_t)
(roletype object_r test_ibendport_t)
(ibendportcon mlx4_0 1 (system_u object_r test_ibendport_t ((s0) (s0))))
EOF
semodule -i my_module.cil

Also, fix loading/storing the 64-bit subnet prefix for OCON_IBPKEY to
use a correctly aligned buffer.

Finally, do not use the 'nodebuf' (u32) buffer where 'buf' (__le32)
should be used instead.

Tested internally on a ppc64 machine with a RHEL 7 kernel with this
patch applied.

Cc: Daniel Jurgens <danielj@mellanox.com>
Cc: Eli Cohen <eli@mellanox.com>
Cc: James Morris <jmorris@namei.org>
Cc: Doug Ledford <dledford@redhat.com>
Cc: <stable@vger.kernel.org> # 4.13+
Fixes: a806f7a1616f ("selinux: Create policydb version for Infiniband support")
Signed-off-by: Ondrej Mosnacek <omosnace@redhat.com>
Acked-by: Stephen Smalley <sds@tycho.nsa.gov>
Signed-off-by: Paul Moore <paul@paul-moore.com>
security/selinux/ss/policydb.c