Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Tiff: ccitt compression, fix for issue #2451 #2483

Merged
merged 3 commits into from
Jun 25, 2023
Merged

Tiff: ccitt compression, fix for issue #2451 #2483

Changes from 2 commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
6c43909
If we read a extension code, read the next 5 bits because some encode…
brianpopow Jun 24, 2023
450279b
Merge branch 'main' into bp/Issue2451
JimBobSquarePants Jun 25, 2023
63aece9
Update src/ImageSharp/Formats/Tiff/Compression/Decompressors/T6BitRea…
JimBobSquarePants Jun 25, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
20 changes: 18 additions & 2 deletions src/ImageSharp/Formats/Tiff/Compression/Decompressors/T6BitReader.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -125,13 +125,29 @@ public bool ReadNextCodeWord()
if (value == Len7Code0000000.Code)
{
this.Code = Len7Code0000000;
return false;

// We do not support Extensions1D codes, but some encoders (scanner from epson) write a premature EOL code,
// which at this point cannot be distinguished from a distinguish, because we read the data bit by bit.
JimBobSquarePants marked this conversation as resolved.
Show resolved Hide resolved
// Read the next 5 bit, if its a EOL code return true, indicating its the end of the image.
if (this.ReadValue(5) == 1)
{
return true;
}

throw new NotSupportedException("ccitt extensions 1D codes are not supported.");
}

if (value == Len7Code0000001.Code)
{
this.Code = Len7Code0000001;
return false;

// Same as above, we do not support Extensions2D codes, but it could be a EOL instead.
if (this.ReadValue(5) == 1)
{
return true;
}

throw new NotSupportedException("ccitt extensions 2D codes are not supported.");
}

if (value == Len7Code0000011.Code)
Expand Down