From 0ff28319906eeb3f236acd72201c416ce01f2c6e Mon Sep 17 00:00:00 2001 From: Eamon Walsh Date: Tue, 15 Sep 2009 19:41:04 -0400 Subject: [PATCH] xselinux: Stop special-casing QueryPointer access checks. XACE has been changed to not return BadAccess on device read failures. Thus, no need for this workaround code. Signed-off-by: Eamon Walsh --- Xext/xselinux_hooks.c | 11 ----------- 1 file changed, 11 deletions(-) diff --git a/Xext/xselinux_hooks.c b/Xext/xselinux_hooks.c index a27067319..fe534880b 100644 --- a/Xext/xselinux_hooks.c +++ b/Xext/xselinux_hooks.c @@ -372,17 +372,6 @@ SELinuxDevice(CallbackListPtr *pcbl, pointer unused, pointer calldata) } } - /* XXX only check read permission on XQueryKeymap */ - /* This is to allow the numerous apps that call XQueryPointer to work */ - if (rec->access_mode & DixReadAccess) { - ClientPtr client = rec->client; - REQUEST(xReq); - if (stuff && stuff->reqType != X_QueryKeymap) { - rec->access_mode &= ~DixReadAccess; - rec->access_mode |= DixGetAttrAccess; - } - } - rc = SELinuxDoCheck(subj, obj, SECCLASS_X_DEVICE, rec->access_mode, &auditdata); if (rc != Success)