Bug 51375

Summary: Xorg doesn't set status for RRGetOutputInfo
Product: xorg Reporter: Jaroslav Šmíd <jardasmid>
Component: Server/Ext/RandRAssignee: Alan Coopersmith <alan.coopersmith>
Status: RESOLVED FIXED QA Contact: Xorg Project Team <xorg-team>
Severity: normal    
Priority: medium    
Version: git   
Hardware: All   
OS: All   
Whiteboard:
i915 platform: i915 features:
Bug Depends on:    
Bug Blocks: 44202    
Attachments:
Description Flags
Set status field in reply to RRGetOutputInfo none

Description Jaroslav Šmíd 2012-06-24 03:41:51 UTC 
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.
Comment 1 Alan Coopersmith 2012-07-04 16:24:25 UTC 
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:
http://patchwork.freedesktop.org/patch/11032/
Comment 2 Jaroslav Šmíd 2012-07-10 16:07:59 UTC 
Seems fixed in commit bd6f948c41865c2c9d3fba1000bf5f7458d3afc1

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.