tree c69d3e3b9e8d1ab7a5e0f394c4d91ec51babb03c
parent bc5f92e5db6f303e73387278e32f8669f0abf0e5
author Markus Armbruster <armbru@redhat.com> 1357824290 +0100
committer Gerd Hoffmann <kraxel@redhat.com> 1358150378 +0100

qxl: Don't drop client capability bits

interface_set_client_capabilities() copies only the first few bits,
because it falls into a Classic C trap: you can declare a parameter
uint8_t caps[58], but the resulting parameter type is uint8_t *, not
uint8_t[58].  In particular, sizeof(caps) is sizeof(uint8_t *), not
the intended sizeof(uint8_t[58]).

Harmless, because the bits aren't used, yet.  Broken in commit
c10018d6.  Spotted by Coverity.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>