Handle edge case when we are at the last byte position, but not all pixels have been written

Author: brianpopow

src/ImageSharp/Formats/Tiff/Compression/Decompressors/T4TiffCompression.cs

@@ -20,6 +20,8 @@ internal class T4TiffCompression : TiffBaseDecompressor
 
         private readonly byte blackValue;
 
+        private readonly int width;
+
         /// <summary>
         /// Initializes a new instance of the <see cref="T4TiffCompression" /> class.
         /// </summary>
@@ -34,7 +36,7 @@ public T4TiffCompression(MemoryAllocator allocator, TiffFillOrder fillOrder, int
         {
             this.faxCompressionOptions = faxOptions;
             this.FillOrder = fillOrder;
-
+            this.width = width;
             bool isWhiteZero = photometricInterpretation == TiffPhotometricInterpretation.WhiteIsZero;
             this.whiteValue = (byte)(isWhiteZero ? 0 : 1);
             this.blackValue = (byte)(isWhiteZero ? 1 : 0);
@@ -58,22 +60,17 @@ protected override void Decompress(BufferedReadStream stream, int byteCount, Spa
 
             buffer.Clear();
             uint bitsWritten = 0;
+            uint pixelWritten = 0;
             while (bitReader.HasMoreData)
             {
                 bitReader.ReadNextRun();
 
                 if (bitReader.RunLength > 0)
                 {
-                    if (bitReader.IsWhiteRun)
-                    {
-                        BitWriterUtils.WriteBits(buffer, (int)bitsWritten, bitReader.RunLength, this.whiteValue);
-                        bitsWritten += bitReader.RunLength;
-                    }
-                    else
-                    {
-                        BitWriterUtils.WriteBits(buffer, (int)bitsWritten, bitReader.RunLength, this.blackValue);
-                        bitsWritten += bitReader.RunLength;
-                    }
+                    this.WritePixelRun(buffer, bitReader, bitsWritten);
+
+                    bitsWritten += bitReader.RunLength;
+                    pixelWritten += bitReader.RunLength;
                 }
 
                 if (bitReader.IsEndOfScanLine)
@@ -85,8 +82,29 @@ protected override void Decompress(BufferedReadStream stream, int byteCount, Spa
                         BitWriterUtils.WriteBits(buffer, (int)bitsWritten, pad, 0);
                         bitsWritten += pad;
                     }
+
+                    pixelWritten = 0;
                 }
             }
+
+            // Edge case for when we are at the last byte, but there are still some unwritten pixels left.
+            if (pixelWritten > 0 && pixelWritten < this.width)
+            {
+                bitReader.ReadNextRun();
+                this.WritePixelRun(buffer, bitReader, bitsWritten);
+            }
+        }
+
+        private void WritePixelRun(Span<byte> buffer, T4BitReader bitReader, uint bitsWritten)
+        {
+            if (bitReader.IsWhiteRun)
+            {
+                BitWriterUtils.WriteBits(buffer, (int)bitsWritten, bitReader.RunLength, this.whiteValue);
+            }
+            else
+            {
+                BitWriterUtils.WriteBits(buffer, (int)bitsWritten, bitReader.RunLength, this.blackValue);
+            }
         }
 
         /// <inheritdoc/>