From: Michel Dänzer Date: Mon, 4 Feb 2019 17:53:52 +0000 (+0100) Subject: loader/dri3: Use strlen instead of sizeof for creating VRR property atom X-Git-Url: https://git.libre-soc.org/?p=mesa.git;a=commitdiff_plain;h=c0a540f32067cc8cb126d9aa1eb12a11cf15373a loader/dri3: Use strlen instead of sizeof for creating VRR property atom sizeof counts the terminating null character as well, so that also contributed to the ID computed for the X11 atom. But the convention is for only the non-null characters to contribute to the atom ID. Fixes: 2e12fe425fe3 "loader/dri3: Enable adaptive_sync via _VARIABLE_REFRESH property" Reviewed-by: Nicholas Kazlauskas Reviewed-by: Eric Anholt --- diff --git a/src/loader/loader_dri3_helper.c b/src/loader/loader_dri3_helper.c index ad9b9d87b05..7d61c1df4fc 100644 --- a/src/loader/loader_dri3_helper.c +++ b/src/loader/loader_dri3_helper.c @@ -111,7 +111,7 @@ set_adaptive_sync_property(xcb_connection_t *conn, xcb_drawable_t drawable, xcb_intern_atom_reply_t* reply; xcb_void_cookie_t check; - cookie = xcb_intern_atom(conn, 0, sizeof(name), name); + cookie = xcb_intern_atom(conn, 0, strlen(name), name); reply = xcb_intern_atom_reply(conn, cookie, NULL); if (reply == NULL) return;