Post Go back to editing

Advantiv EDID Editor (EEditGold) misses one DTD

Hi  ,

we found a bug in EEditGold 1.01.0276.

EEditGold misses the last DTD of the Dell AW3821DW EDID.

Probably a fence post error in buffer size calculation. We had the same error in our own EDID parser. This was the fix:

 	// Get detailed timings from CTA
        int ceaStartPos = 128;
        int ceaEndPos = 255;
BUG:	while ((nextDtdPos + DTD_SIZE) < ceaEndPos)
->FIX: while ((nextDtdPos + DTD_SIZE) <= ceaEndPos)



Here are the EDID data of the Dell AW3821DW EDID.

00,FF,FF,FF,FF,FF,FF,00,10,AC,7D,A1,4C,35,42,30,
17,1F,01,03,80,58,25,78,2A,F1,F1,AE,4F,45,AB,26,
0E,50,54,21,08,00,01,01,01,01,01,01,01,01,01,01,
01,01,01,01,01,01,3C,9B,00,A0,F0,40,38,60,30,20,
35,00,70,6F,31,00,00,1A,00,00,00,FF,00,23,48,44,
45,59,4D,78,67,77,41,41,34,71,00,00,00,FD,00,18,
78,1E,8D,3C,01,0A,20,20,20,20,20,20,00,00,00,FC,
00,44,65,6C,6C,20,41,57,33,38,32,31,44,57,01,9E,
02,03,37,F1,48,10,1F,04,13,03,12,01,3F,23,09,07,
01,83,01,00,00,68,03,0C,00,20,00,38,3C,00,6A,D8,
5D,C4,01,78,80,00,00,02,78,E3,05,C0,00,E2,00,6A,
E6,06,05,01,78,60,1F,31,DD,00,A0,F0,40,42,60,30,
20,35,00,70,6F,31,00,00,1A,D1,4C,00,A0,F0,40,27,
60,30,20,35,00,70,6F,31,00,00,1A,05,5F,00,A0,A0,
A0,33,50,30,20,35,00,70,6F,31,00,00,1A,6F,C2,00,
A0,A0,A0,55,50,30,20,35,00,70,6F,31,00,00,1A,4D


This is what EEdit exports

//Creator:Advantiv® EEditGold 1.01.0276 by Analog Devices   04/01/2022 11:27:38
//Source:
00,FF,FF,FF,FF,FF,FF,00,10,AC,7D,A1,4C,35,42,30,
17,1F,01,03,80,58,25,78,2A,F1,F1,AE,4F,45,AB,26,
0E,50,54,21,08,00,01,01,01,01,01,01,01,01,01,01,
01,01,01,01,01,01,3C,9B,00,A0,F0,40,38,60,30,20,
35,00,70,6F,31,00,00,1A,00,00,00,FF,00,23,48,44,
45,59,4D,78,67,77,41,41,34,71,00,00,00,FD,00,18,
78,1E,8D,3C,01,0A,20,20,20,20,20,20,00,00,00,FC,
00,44,65,6C,6C,20,41,57,33,38,32,31,44,57,01,9E,
02,03,37,F4,48,10,1F,04,13,03,12,01,3F,23,09,07,
01,83,01,00,00,68,03,0C,00,20,00,38,3C,00,6A,D8,
5D,C4,01,78,80,00,00,02,78,E3,05,C0,00,E2,00,6A,
E6,06,05,01,78,60,1F,31,DD,00,A0,F0,40,42,60,30,
20,35,00,70,6F,31,00,00,1A,D1,4C,00,A0,F0,40,27,
60,30,20,35,00,70,6F,31,00,00,1A,05,5F,00,A0,A0,
A0,33,50,30,20,35,00,70,6F,31,00,00,1A,00,00,00,
00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,AF

Thanks,

Friedo

Edit Notes

Added original binary data.
[edited by: FriedoB at 10:49 AM (GMT -5) on 4 Jan 2022]
Parents
  • FormerMember
    FormerMember
on Jan 4, 2022 10:46 AM
0

Thanks for finding this bug.  I'll investigate it however I do not have a timeline when I can get it fixed

  • FormerMember
    FormerMember on Mar 1, 2022 9:39 AM in reply to FriedoB
    0

    Hi Friedo,

    Sorry for the very late response.  I have not forgotten about this issue, I just been tasked with getting another critical task done before I touched anything else.  I'm now wrapping up this task and will begin  looking at this problem.  I also plan to review the new CTA861H spec. and add any  needed changes.  

  • FormerMember
    FormerMember on Mar 7, 2022 10:54 AM in reply to FormerMember
    +1

    Please check out the latest EEditGold version and let me know if there are any problems.  You’ll have to uninstall the existing one and replace it with this

    https://ez.analog.com/video/w/documents/750/advantiv-edid-editor

     Also let me know if you have any other changes you would like to see.

    I'm reviewing CTA-861-H currently to see if anything needs to change.

    Also planning to add Display Information block processing.

    Thanks

  • I can confirm that the bug is fixed with EEditGold_1p02p0007.

    Thanks a lot for your help.

  • Reply Children
    No Data