[PATCH] randr: Fix REQUEST vs. REQUEST_SIZE_MATCH mismatch

Aaron Plattner aplattner at nvidia.com
Fri Jun 29 13:31:52 PDT 2012


On 06/29/2012 12:57 PM, Alan Coopersmith wrote:
> On 06/29/12 12:22 PM, Aaron Plattner wrote:
>> ProcRRGetScreenSizeRange uses REQUEST(xRRGetScreenSizeRangeReq) followed by
>> REQUEST_SIZE_MATCH(xRRGetScreenSizeRangeReq).  This happens to work out because

d'oh, this should read REQUEST_SIZE_MATCH(xRRGetScreenInfoReq), of course. 
Stupid copy & paste errors... :P

-- Aaron

>> both requests have the same size, so this is not a functional change, just a
>> cosmetic one.
>>
>> Signed-off-by: Aaron Plattner <aplattner at nvidia.com>
>> ---
>>   randr/rrscreen.c |    2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/randr/rrscreen.c b/randr/rrscreen.c
>> index 55110e0..c564d1f 100644
>> --- a/randr/rrscreen.c
>> +++ b/randr/rrscreen.c
>> @@ -195,7 +195,7 @@ ProcRRGetScreenSizeRange(ClientPtr client)
>>       rrScrPrivPtr pScrPriv;
>>       int rc;
>>
>> -    REQUEST_SIZE_MATCH(xRRGetScreenInfoReq);
>> +    REQUEST_SIZE_MATCH(xRRGetScreenSizeRangeReq);
>>       rc = dixLookupWindow(&pWin, stuff->window, client, DixGetAttrAccess);
>>       if (rc != Success)
>>           return rc;
>
> Reviewed-by: Alan Coopersmith <alan.coopersmith at oracle.com>


More information about the xorg-devel mailing list