Created attachment 63397 [details] [review]
Set status field in reply to RRGetOutputInfo
I was writing app for screen configuration using randr extension using xcb library and found out that "status" field for RRGetOutputInfo reply is set to a random value every time I run it. That prevents my application to work properly as it does check that field to make sure the request was successfull (server could e.g. return InvalidConfigTime if the configuration was changed in between RRGetScreenResources and RRGetOutputInfo requests).
I did some search and found out it is issue in xorg server, it doesn't set "status" field in reply to RRGetOutputInfo and garbage is returned in the field. Attaching patch to fix the problem.
I think xorg should also check the timestamps and return InvalidConfigTime in the status field if they are wrong, but it doesn't seem to do so. I will report is as different bug.
Thanks - I've included this patch in my current patch series since I'd noticed
the missing field as well while reworking the way the values are set:
Seems fixed in commit bd6f948c41865c2c9d3fba1000bf5f7458d3afc1