[PATCH] dmx: Correctly compute DMXGetScreenAttributes reply length
Peter Hutterer
peter.hutterer at who-t.net
Sun Oct 25 17:29:05 PDT 2009
On Fri, Oct 23, 2009 at 07:32:42PM +0100, Jon TURNEY wrote:
> Correctly allow for excess length of DMXGetScreenAttributes reply
> over standard 32 byte reply in addition to the displayName string
> when computing the length of reply
>
> http://bugs.freedesktop.org/show_bug.cgi?id=24685
>
> Signed-off-by: Jon TURNEY <jon.turney at dronecode.org.uk>
> Reviewed-by: Julien Cristau <jcristau at debian.org>
> ---
> hw/dmx/dmx.c | 3 ++-
> 1 files changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/hw/dmx/dmx.c b/hw/dmx/dmx.c
> index b8fbf02..a1afe76 100644
> --- a/hw/dmx/dmx.c
> +++ b/hw/dmx/dmx.c
> @@ -362,7 +362,8 @@ static int ProcDMXGetScreenAttributes(ClientPtr client)
> paddedLength = pad_to_int32(length);
> rep.type = X_Reply;
> rep.sequenceNumber = client->sequence;
> - rep.length = bytes_to_int32(paddedLength);
> + rep.length = bytes_to_int32((sizeof(xDMXGetScreenAttributesReply) - sizeof(xGenericReply))
> + + paddedLength);
> rep.displayNameLength = length;
>
> if (client->swapped) {
> --
> 1.6.4.2
Thanks, merged this into my tree.
Cheers,
Peter
More information about the xorg-devel
mailing list