After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 752673 - backends/monitor-manager: Validate EDID strings
backends/monitor-manager: Validate EDID strings
Status: RESOLVED FIXED
Product: mutter
Classification: Core
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: mutter-maint
mutter-maint
Depends on:
Blocks:
 
 
Reported: 2015-07-21 14:42 UTC by Rui Matos
Modified: 2015-07-21 15:47 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
backends/monitor-manager: Validate EDID strings (2.73 KB, patch)
2015-07-21 14:42 UTC, Rui Matos 		committed Details | Review

Description Rui Matos 2015-07-21 14:42:50 UTC 
Some monitors return a bunch of bytes on their display descriptor
which aren't valid utf8 and thus we fail to serialize them later on
for the DisplayConfig DBus API.

Let's fall back to the stringified product code and serial number in
that case.
Comment 1 Rui Matos 2015-07-21 14:42:55 UTC 
Created attachment 307833 [details] [review]
backends/monitor-manager: Validate EDID strings
Comment 2 Jasper St. Pierre (not reading bugmail) 2015-07-21 15:43:10 UTC 
Review of attachment 307833 [details] [review]:

OK.
Comment 3 Rui Matos 2015-07-21 15:46:58 UTC 
Attachment 307833 [details] pushed as 2fb8da0 - backends/monitor-manager: Validate EDID strings