All of lore.kernel.org
 help / color / mirror / Atom feed
* edid-decode: Incorrect decoding of CTA extension if it does not contain padding bytes
@ 2020-09-11 18:27 Pavel Kuzmin
  2020-09-11 21:56 ` Hans Verkuil
  0 siblings, 1 reply; 2+ messages in thread
From: Pavel Kuzmin @ 2020-09-11 18:27 UTC (permalink / raw)
  To: linux-media

Hello


If in the EDID CTA extension the last 18 byte DTD block ends
immediately before the checksum byte (no padding bytes) then the last
DTD block is not processed and a failure is displayed "Padding:
CTA-861 padding contains non-zero bytes."

e.g. https://github.com/linuxhw/EDID/blob/master/Digital/Acer/ACR0549/42BFE4422CEF


Regards,
Pavel

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: edid-decode: Incorrect decoding of CTA extension if it does not contain padding bytes
  2020-09-11 18:27 edid-decode: Incorrect decoding of CTA extension if it does not contain padding bytes Pavel Kuzmin
@ 2020-09-11 21:56 ` Hans Verkuil
  0 siblings, 0 replies; 2+ messages in thread
From: Hans Verkuil @ 2020-09-11 21:56 UTC (permalink / raw)
  To: Pavel Kuzmin, linux-media

Hi Pavel,

On 11/09/2020 20:27, Pavel Kuzmin wrote:
> Hello
> 
> 
> If in the EDID CTA extension the last 18 byte DTD block ends
> immediately before the checksum byte (no padding bytes) then the last
> DTD block is not processed and a failure is displayed "Padding:
> CTA-861 padding contains non-zero bytes."

Thanks for reporting this, nice catch!

It was caused by an off-by-one check and this is now fixed.

Regards,

	Hans

> 
> e.g. https://github.com/linuxhw/EDID/blob/master/Digital/Acer/ACR0549/42BFE4422CEF
> 
> 
> Regards,
> Pavel
> 


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2020-09-11 21:57 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-11 18:27 edid-decode: Incorrect decoding of CTA extension if it does not contain padding bytes Pavel Kuzmin
2020-09-11 21:56 ` Hans Verkuil

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.