dri3: Fix XACE access mode for open and get_supported_modifiers
Neither opening a screen nor querying its modifiers confers the right to attach the buffer for any particular pixmap. GetAttr seems more correct. Fixes: xorg/xserver#550
This commit is contained in:
parent
6c1d720047
commit
086c2e3de5
|
@ -135,7 +135,7 @@ proc_dri3_open(ClientPtr client)
|
||||||
|
|
||||||
REQUEST_SIZE_MATCH(xDRI3OpenReq);
|
REQUEST_SIZE_MATCH(xDRI3OpenReq);
|
||||||
|
|
||||||
status = dixLookupDrawable(&drawable, stuff->drawable, client, 0, DixReadAccess);
|
status = dixLookupDrawable(&drawable, stuff->drawable, client, 0, DixGetAttrAccess);
|
||||||
if (status != Success)
|
if (status != Success)
|
||||||
return status;
|
return status;
|
||||||
|
|
||||||
|
@ -365,7 +365,7 @@ proc_dri3_get_supported_modifiers(ClientPtr client)
|
||||||
|
|
||||||
REQUEST_SIZE_MATCH(xDRI3GetSupportedModifiersReq);
|
REQUEST_SIZE_MATCH(xDRI3GetSupportedModifiersReq);
|
||||||
|
|
||||||
status = dixLookupWindow(&window, stuff->window, client, DixReadAccess);
|
status = dixLookupWindow(&window, stuff->window, client, DixGetAttrAccess);
|
||||||
if (status != Success)
|
if (status != Success)
|
||||||
return status;
|
return status;
|
||||||
pScreen = window->drawable.pScreen;
|
pScreen = window->drawable.pScreen;
|
||||||
|
|
Loading…
Reference in New Issue
Block a user