diff options
author | Eamon Walsh <ewalsh@tycho.nsa.gov> | 2009-06-18 21:02:28 -0400 |
---|---|---|
committer | Eamon Walsh <ewalsh@tycho.nsa.gov> | 2009-06-23 20:50:29 -0400 |
commit | 119b96667778391436998c76a68bf64e746c9e08 (patch) | |
tree | 867e894a7e37cc232ce356c60f17a1beba78f891 | |
parent | 07c36e4fdcd93df3d33bdab6cca4780ebc9c1f54 (diff) |
Xi: fix up access modes for calls to dixLookupDevice().
New access modes are being passed to the device access hook for XI2:
DixCreateAccess for creating a new master device;
DixAdd/RemoveAccess for attaching/removing slave devices to a master; and
DixListProp/GetProp/SetPropAccess for device properties.
Refer to the XACE-Spec document in xorg-docs, section "Device Access."
Signed-off-by: Eamon Walsh <ewalsh@tycho.nsa.gov>
-rw-r--r-- | Xi/exevents.c | 2 | ||||
-rw-r--r-- | Xi/grabdev.c | 2 | ||||
-rw-r--r-- | Xi/grabdevb.c | 4 | ||||
-rw-r--r-- | Xi/grabdevk.c | 4 | ||||
-rw-r--r-- | Xi/stubs.c | 2 | ||||
-rw-r--r-- | Xi/xichangecursor.c | 2 | ||||
-rw-r--r-- | Xi/xichangehierarchy.c | 10 | ||||
-rw-r--r-- | Xi/xigetclientpointer.c | 2 | ||||
-rw-r--r-- | Xi/xiproperty.c | 20 | ||||
-rw-r--r-- | Xi/xiquerypointer.c | 2 | ||||
-rw-r--r-- | Xi/xiselectev.c | 8 | ||||
-rw-r--r-- | Xi/xisetclientpointer.c | 4 | ||||
-rw-r--r-- | Xi/xiwarppointer.c | 4 |
13 files changed, 33 insertions, 33 deletions
diff --git a/Xi/exevents.c b/Xi/exevents.c index e54af094e..34fdf5007 100644 --- a/Xi/exevents.c +++ b/Xi/exevents.c @@ -2000,7 +2000,7 @@ CheckDeviceGrabAndHintWindow(WindowPtr pWin, int type, DeviceIntPtr dev; dixLookupDevice(&dev, xE->deviceid & DEVICE_BITS, serverClient, - DixReadAccess); + DixGrabAccess); if (!dev) return; diff --git a/Xi/grabdev.c b/Xi/grabdev.c index adcd7e79f..e1d430a1f 100644 --- a/Xi/grabdev.c +++ b/Xi/grabdev.c @@ -184,7 +184,7 @@ CreateMaskFromList(ClientPtr client, XEventClass * list, int count, if (device > 255) /* FIXME: we only use 7 bit for devices? */ return BadClass; - rc = dixLookupDevice(&tdev, device, client, DixReadAccess); + rc = dixLookupDevice(&tdev, device, client, DixUseAccess); if (rc != BadDevice && rc != Success) return rc; if (rc == BadDevice || (dev != NULL && tdev != dev)) diff --git a/Xi/grabdevb.c b/Xi/grabdevb.c index 7cd5cc474..58fb73bca 100644 --- a/Xi/grabdevb.c +++ b/Xi/grabdevb.c @@ -119,14 +119,14 @@ ProcXGrabDeviceButton(ClientPtr client) if (stuff->modifier_device != UseXKeyboard) { ret = dixLookupDevice(&mdev, stuff->modifier_device, client, - DixReadAccess); + DixUseAccess); if (ret != Success) return ret; if (mdev->key == NULL) return BadMatch; } else { mdev = PickKeyboard(client); - ret = XaceHook(XACE_DEVICE_ACCESS, client, mdev, DixReadAccess); + ret = XaceHook(XACE_DEVICE_ACCESS, client, mdev, DixUseAccess); if (ret != Success) return ret; } diff --git a/Xi/grabdevk.c b/Xi/grabdevk.c index 5ffecd25d..9ae38f0f1 100644 --- a/Xi/grabdevk.c +++ b/Xi/grabdevk.c @@ -116,14 +116,14 @@ ProcXGrabDeviceKey(ClientPtr client) if (stuff->modifier_device != UseXKeyboard) { ret = dixLookupDevice(&mdev, stuff->modifier_device, client, - DixReadAccess); + DixUseAccess); if (ret != Success) return ret; if (mdev->key == NULL) return BadMatch; } else { mdev = PickKeyboard(client); - ret = XaceHook(XACE_DEVICE_ACCESS, client, mdev, DixReadAccess); + ret = XaceHook(XACE_DEVICE_ACCESS, client, mdev, DixUseAccess); if (ret != Success) return ret; } diff --git a/Xi/stubs.c b/Xi/stubs.c index 229394b93..400e937d1 100644 --- a/Xi/stubs.c +++ b/Xi/stubs.c @@ -153,7 +153,7 @@ AddOtherInputDevices(void) void OpenInputDevice(DeviceIntPtr dev, ClientPtr client, int *status) { - *status = XaceHook(XACE_DEVICE_ACCESS, client, dev, DixReadAccess); + *status = XaceHook(XACE_DEVICE_ACCESS, client, dev, DixUseAccess); } /**************************************************************************** diff --git a/Xi/xichangecursor.c b/Xi/xichangecursor.c index 7517d2c2e..dc33d70a1 100644 --- a/Xi/xichangecursor.c +++ b/Xi/xichangecursor.c @@ -95,7 +95,7 @@ int ProcXIChangeCursor(ClientPtr client) else { rc = dixLookupResourceByType((pointer *)&pCursor, stuff->cursor, - RT_CURSOR, client, DixReadAccess); + RT_CURSOR, client, DixUseAccess); if (rc != Success) return (rc == BadValue) ? BadCursor : rc; } diff --git a/Xi/xichangehierarchy.c b/Xi/xichangehierarchy.c index 5401554ac..917a0d7d9 100644 --- a/Xi/xichangehierarchy.c +++ b/Xi/xichangehierarchy.c @@ -340,7 +340,7 @@ ProcXIChangeHierarchy(ClientPtr client) newkeybd; rc = dixLookupDevice(&newptr, r->return_pointer, - client, DixWriteAccess); + client, DixAddAccess); if (rc != Success) goto unwind; @@ -352,7 +352,7 @@ ProcXIChangeHierarchy(ClientPtr client) } rc = dixLookupDevice(&newkeybd, r->return_keyboard, - client, DixWriteAccess); + client, DixAddAccess); if (rc != Success) goto unwind; @@ -415,7 +415,7 @@ ProcXIChangeHierarchy(ClientPtr client) DeviceIntPtr *xtstdevice; rc = dixLookupDevice(&ptr, c->deviceid, client, - DixWriteAccess); + DixManageAccess); if (rc != Success) goto unwind; @@ -448,7 +448,7 @@ ProcXIChangeHierarchy(ClientPtr client) DeviceIntPtr *xtstdevice; rc = dixLookupDevice(&ptr, c->deviceid, client, - DixWriteAccess); + DixManageAccess); if (rc != Success) goto unwind; @@ -471,7 +471,7 @@ ProcXIChangeHierarchy(ClientPtr client) } rc = dixLookupDevice(&newmaster, c->new_master, - client, DixWriteAccess); + client, DixAddAccess); if (rc != Success) goto unwind; if (!IsMaster(newmaster)) diff --git a/Xi/xigetclientpointer.c b/Xi/xigetclientpointer.c index 859d3fd00..401e89fb4 100644 --- a/Xi/xigetclientpointer.c +++ b/Xi/xigetclientpointer.c @@ -68,7 +68,7 @@ int ProcXIGetClientPointer(ClientPtr client) if (stuff->win != None) { rc = dixLookupClient(&winclient, stuff->win, client, - DixWriteAccess); + DixGetAttrAccess); if (rc != Success) return BadWindow; diff --git a/Xi/xiproperty.c b/Xi/xiproperty.c index cd4946026..396061f80 100644 --- a/Xi/xiproperty.c +++ b/Xi/xiproperty.c @@ -858,7 +858,7 @@ ProcXListDeviceProperties (ClientPtr client) REQUEST(xListDevicePropertiesReq); REQUEST_SIZE_MATCH(xListDevicePropertiesReq); - rc = dixLookupDevice (&dev, stuff->deviceid, client, DixReadAccess); + rc = dixLookupDevice (&dev, stuff->deviceid, client, DixListPropAccess); if (rc != Success) return rc; @@ -894,7 +894,7 @@ ProcXChangeDeviceProperty (ClientPtr client) REQUEST_AT_LEAST_SIZE(xChangeDevicePropertyReq); UpdateCurrentTime(); - rc = dixLookupDevice (&dev, stuff->deviceid, client, DixWriteAccess); + rc = dixLookupDevice (&dev, stuff->deviceid, client, DixSetPropAccess); if (rc != Success) return rc; @@ -922,7 +922,7 @@ ProcXDeleteDeviceProperty (ClientPtr client) REQUEST_SIZE_MATCH(xDeleteDevicePropertyReq); UpdateCurrentTime(); - rc = dixLookupDevice (&dev, stuff->deviceid, client, DixWriteAccess); + rc = dixLookupDevice (&dev, stuff->deviceid, client, DixSetPropAccess); if (rc != Success) return rc; @@ -951,8 +951,8 @@ ProcXGetDeviceProperty (ClientPtr client) if (stuff->delete) UpdateCurrentTime(); rc = dixLookupDevice (&dev, stuff->deviceid, client, - stuff->delete ? DixWriteAccess : - DixReadAccess); + stuff->delete ? DixSetPropAccess : + DixGetPropAccess); if (rc != Success) return rc; @@ -1102,7 +1102,7 @@ ProcXIListProperties(ClientPtr client) REQUEST(xXIListPropertiesReq); REQUEST_SIZE_MATCH(xXIListPropertiesReq); - rc = dixLookupDevice (&dev, stuff->deviceid, client, DixReadAccess); + rc = dixLookupDevice (&dev, stuff->deviceid, client, DixListPropAccess); if (rc != Success) return rc; @@ -1138,7 +1138,7 @@ ProcXIChangeProperty(ClientPtr client) REQUEST_AT_LEAST_SIZE(xXIChangePropertyReq); UpdateCurrentTime(); - rc = dixLookupDevice (&dev, stuff->deviceid, client, DixWriteAccess); + rc = dixLookupDevice (&dev, stuff->deviceid, client, DixSetPropAccess); if (rc != Success) return rc; @@ -1165,7 +1165,7 @@ ProcXIDeleteProperty(ClientPtr client) REQUEST_SIZE_MATCH(xXIDeletePropertyReq); UpdateCurrentTime(); - rc = dixLookupDevice (&dev, stuff->deviceid, client, DixWriteAccess); + rc = dixLookupDevice (&dev, stuff->deviceid, client, DixSetPropAccess); if (rc != Success) return rc; @@ -1195,8 +1195,8 @@ ProcXIGetProperty(ClientPtr client) if (stuff->delete) UpdateCurrentTime(); rc = dixLookupDevice (&dev, stuff->deviceid, client, - stuff->delete ? DixWriteAccess : - DixReadAccess); + stuff->delete ? DixSetPropAccess : + DixGetPropAccess); if (rc != Success) return rc; diff --git a/Xi/xiquerypointer.c b/Xi/xiquerypointer.c index e770e84fc..2222873aa 100644 --- a/Xi/xiquerypointer.c +++ b/Xi/xiquerypointer.c @@ -91,7 +91,7 @@ ProcXIQueryPointer(ClientPtr client) return BadDevice; } - rc = dixLookupWindow(&pWin, stuff->win, client, DixReadAccess); + rc = dixLookupWindow(&pWin, stuff->win, client, DixGetAttrAccess); if (rc != Success) { SendErrorToClient(client, IReqCode, X_XIQueryPointer, diff --git a/Xi/xiselectev.c b/Xi/xiselectev.c index a8763a2a9..750ae61cc 100644 --- a/Xi/xiselectev.c +++ b/Xi/xiselectev.c @@ -87,7 +87,7 @@ ProcXISelectEvents(ClientPtr client) { if (evmask->deviceid != XIAllDevices && evmask->deviceid != XIAllMasterDevices) - rc = dixLookupDevice(&dev, evmask->deviceid, client, DixReadAccess); + rc = dixLookupDevice(&dev, evmask->deviceid, client, DixUseAccess); else { /* XXX: XACE here? */ } @@ -127,7 +127,7 @@ ProcXISelectEvents(ClientPtr client) dummy.id = evmask->deviceid; dev = &dummy; } else - dixLookupDevice(&dev, evmask->deviceid, client, DixReadAccess); + dixLookupDevice(&dev, evmask->deviceid, client, DixUseAccess); XISetEventMask(dev, win, client, evmask->mask_len * 4, (unsigned char*)&evmask[1]); evmask = (xXIEventMask*)(((unsigned char*)evmask) + evmask->mask_len * 4); evmask++; @@ -169,7 +169,7 @@ ProcXIGetSelectedEvents(ClientPtr client) REQUEST(xXIGetSelectedEventsReq); REQUEST_SIZE_MATCH(xXIGetSelectedEventsReq); - rc = dixLookupWindow(&win, stuff->win, client, DixReceiveAccess); + rc = dixLookupWindow(&win, stuff->win, client, DixGetAttrAccess); if (rc != Success) return rc; @@ -208,7 +208,7 @@ ProcXIGetSelectedEvents(ClientPtr client) if (i > 2) { - rc = dixLookupDevice(&dev, i, client, DixReadAccess); + rc = dixLookupDevice(&dev, i, client, DixGetAttrAccess); if (rc != Success) continue; } diff --git a/Xi/xisetclientpointer.c b/Xi/xisetclientpointer.c index afc13d676..273618503 100644 --- a/Xi/xisetclientpointer.c +++ b/Xi/xisetclientpointer.c @@ -72,7 +72,7 @@ ProcXISetClientPointer(ClientPtr client) REQUEST_SIZE_MATCH(xXISetClientPointerReq); - rc = dixLookupDevice(&pDev, stuff->deviceid, client, DixWriteAccess); + rc = dixLookupDevice(&pDev, stuff->deviceid, client, DixManageAccess); if (rc != Success) { client->errorValue = stuff->deviceid; @@ -90,7 +90,7 @@ ProcXISetClientPointer(ClientPtr client) if (stuff->win != None) { rc = dixLookupClient(&targetClient, stuff->win, client, - DixWriteAccess); + DixManageAccess); if (rc != Success) return BadWindow; diff --git a/Xi/xiwarppointer.c b/Xi/xiwarppointer.c index 105c87a0d..bf361dbe4 100644 --- a/Xi/xiwarppointer.c +++ b/Xi/xiwarppointer.c @@ -85,7 +85,7 @@ ProcXIWarpPointer(ClientPtr client) if (stuff->dst_win != None) { - rc = dixLookupWindow(&dest, stuff->dst_win, client, DixReadAccess); + rc = dixLookupWindow(&dest, stuff->dst_win, client, DixGetAttrAccess); if (rc != Success) { return rc; @@ -101,7 +101,7 @@ ProcXIWarpPointer(ClientPtr client) int winX, winY; WindowPtr src; - rc = dixLookupWindow(&src, stuff->src_win, client, DixReadAccess); + rc = dixLookupWindow(&src, stuff->src_win, client, DixGetAttrAccess); if (rc != Success) { return rc; |