Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

xgc/res protocol / data size mismatch #54

Open
mariusbu opened this issue Oct 9, 2020 · 0 comments
Open

xgc/res protocol / data size mismatch #54

mariusbu opened this issue Oct 9, 2020 · 0 comments

Comments

@mariusbu
Copy link

mariusbu commented Oct 9, 2020

Hi,

Calling res.QueryClientIds(conn, 1, specs).Reply() will crash due to an old bug in xcb/proto/src/res.xml.
The problem is located here:

xgb/res/res.go

Line 170 in 27f1227

v.Value[i] = xgb.Get32(buf[b:])

The ClientIdValue.Length we get from the server indicates the length in bytes, but the loop reads CARD32s.

The bug has been fixed in the protocol description (see https://cgit.freedesktop.org/xcb/proto/commit/?id=f7948e355d85624c577d0fa63977b3bed4d6024f), but the X-Resource bindings will have to be re-generated from the updated res.xml (or edited manually) to fix this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant