[PATCH 06/11] glx: Implement GLX SetClientInfoARB protocol

Christopher James Halse Rogers christopher.halse.rogers at canonical.com
Tue Apr 17 03:06:10 PDT 2012


On Fri, 2012-04-13 at 18:11 -0700, Ian Romanick wrote:
> From: Ian Romanick <ian.d.romanick at intel.com>
> 
> Signed-off-by: Ian Romanick <ian.d.romanick at intel.com>
> Reviewed-by: Dave Airlie <airlied at redhat.com>
> ---
>  glx/clientinfo.c |   49 +++++++++++++++++++++++++++++++++++++++++++++++--
>  1 files changed, 47 insertions(+), 2 deletions(-)
> 
> diff --git a/glx/clientinfo.c b/glx/clientinfo.c
> index 15bbf15..b26ac1a 100644
> --- a/glx/clientinfo.c
> +++ b/glx/clientinfo.c
> @@ -26,17 +26,62 @@
>  
>  #include "glxserver.h"
>  #include "indirect_dispatch.h"
> +#include "glxbyteorder.h"
> +#include "unpack.h"
>  
>  int
>  __glXDisp_SetClientInfoARB(__GLXclientState * cl, GLbyte * pc)
>  {
> -    return BadRequest;
> +    xGLXSetClientInfoARBReq *req = (xGLXSetClientInfoARBReq *) pc;
> +    char *gl_extensions;
> +    char *glx_extensions;
> +
> +    /* Verify that the size of the packet matches the size inferred from the
> +     * sizes specified for the various fields.
> +     */
> +    const unsigned expected_size = sz_xGLXSetClientInfoARBReq
> +        + (req->numVersions * 8)
> +        + __GLX_PAD(req->numGLExtensionBytes)
> +        + __GLX_PAD(req->numGLXExtensionBytes);
> +
> +    if (req->length != (expected_size / 4))
> +        return BadLength;
> +
> +    /* Verify that the actual length of the GL extension string matches what's
> +     * encoded in protocol packet.
> +     */
> +    gl_extensions = (char *) (req + 1) + (req->numVersions * 8);
> +    if (req->numGLExtensionBytes != 0
> +        && memchr(gl_extensions, 0,
> +                  __GLX_PAD(req->numGLExtensionBytes)) == NULL)
> +        return BadLength;
> +
> +    /* Verify that the actual length of the GLX extension string matches
> +     * what's encoded in protocol packet.
> +     */
> +    glx_extensions = gl_extensions + __GLX_PAD(req->numGLExtensionBytes);
> +    if (req->numGLXExtensionBytes != 0
> +        && memchr(glx_extensions, 0,
> +                  __GLX_PAD(req->numGLXExtensionBytes)) == NULL)
> +        return BadLength;
> +
> +    free(cl->GLClientextensions);
> +    cl->GLClientextensions = strdup(gl_extensions);
> +
> +    return 0;
>  }
>  
>  int
>  __glXDispSwap_SetClientInfoARB(__GLXclientState * cl, GLbyte * pc)
>  {
> -    return BadRequest;
> +    xGLXSetClientInfoARBReq *req = (xGLXSetClientInfoARBReq *) pc;
> +
> +    req->length = bswap_16(req->length);
> +    req->numVersions = bswap_32(req->numVersions);
> +    req->numGLExtensionBytes = bswap_32(req->numGLExtensionBytes);
> +    req->numGLXExtensionBytes = bswap_32(req->numGLXExtensionBytes);
> +
> +    return __glXDisp_SetClientInfoARB(cl, pc);
>  }
>  
>  int

This should probably also bump the minimum required glproto in
configure.ac
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part
URL: <http://lists.x.org/archives/xorg-devel/attachments/20120417/6e399e2b/attachment.pgp>


More information about the xorg-devel mailing list