edid-decode does not allow spaces, it is incorrectly reporting the name and throwing an error. edid-decode: -- descriptor 4: 00 00 00 fc 00 53 4d 41 52 54 20 49 46 50 0a 20 20 20 ... Monitor name: SMART ... Detailed block string not properly terminated -- parse-edid: -- Section "Monitor" Identifier "SMART IFP" ModelName "SMART IFP" -- After attached patch edid-decode behaves correctly. Simon.
Created attachment 130390 [details] [review] patch to fix
Created attachment 130391 [details] sample edid with spaces
*** Bug 100349 has been marked as a duplicate of this bug. ***
Related Bug 93777.
I found and fixed this independently in Ubuntu before checking upstream and finding this report. It has been fixed in commit 57c73067385e0bd29d0a67fd73db4ebecc8fb084 Author: Adam Jackson <ajax@redhat.com> Date: Tue Mar 28 11:21:36 2017 -0400 Fix valid termination check
Hi Adam, Don't know what I could have done to make my patch easier to find... ;-) Do you have a link to your patch, so that it can be validated? Simon.
https://git.linuxtv.org/edid-decode.git/commit/edid-decode.c?id=57c73067385e0bd29d0a67fd73db4ebecc8fb084 is the fix that was applied. Thanks for the report and sample fix, sorry that no one was watching bugzilla at the time.
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.