- monitor-get-edid: skip /proc/acpi/video/**/EDID files which can't be valid

  (#34417)

(drop the monitor-parse-edid from NEWS since it's not that useful anymore
since monitor-edid will not give it EDID containing "<not supported>")

2 files changed, 11 insertions, 4 deletions

diff --git a/NEWS b/NEWS
index 378d633..d611931 100644
--- a/NEWS
+++ b/NEWS
@@ -1,3 +1,2 @@
-- monitor-parse-edid: "<file name>: bad edid" so that monitor-edid trying to
-  parse "<not supported>" /proc/acpi/video/**/EDID files will give a more
-  understandable error message
+- monitor-get-edid: skip /proc/acpi/video/**/EDID files which can't be valid
+  (#34417)
diff --git a/monitor-edid b/monitor-edid
index 1ba4d1c..8cbfb8d 100755
--- a/monitor-edid
+++ b/monitor-edid
@@ -61,7 +61,10 @@ sub parse_edid() {
 
 sub get_edids {
     my ($b_get_first) = @_;
-    my @l = map { my $s = slurp($_); $s ? [ $_ => $s ] : () } get_edid_files();
+    my @l = map { 
+	my $s = slurp($_); 
+	is_edid_possibly_valid($s) ? [ $_ => $s ] : ();
+    } get_edid_files();
     
     if (!@l || !$b_get_first && $< == 0) {
 	if (my $cmd = get_using_vbe()) {
@@ -95,6 +98,11 @@ sub find_EDID {
     @l;
 }
 
+sub is_edid_possibly_valid {
+    my ($edid) = @_;
+    length($edid) >= 128;
+}
+
 sub slurp { 
     my ($f) = @_; 
     open(my $F, '<', $f) or return;