[PATCH] randr: Fix REQUEST vs. REQUEST_SIZE_MATCH mismatch

Alan Coopersmith alan.coopersmith at oracle.com
Fri Jun 29 12:57:27 PDT 2012


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
> 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>

-- 
	-Alan Coopersmith-              alan.coopersmith at oracle.com
	 Oracle Solaris Engineering - http://blogs.oracle.com/alanc


More information about the xorg-devel mailing list