From 749452503b6a7f638a1adf31b391a577569c4080 Mon Sep 17 00:00:00 2001 From: Ynse Hoornenborg Date: Fri, 10 Dec 2021 13:25:14 +0100 Subject: [PATCH] Write EOF indicated for plain encoding --- src/ImageSharp/Formats/Pbm/PlainEncoder.cs | 3 +++ tests/ImageSharp.Tests/Formats/Pbm/PbmEncoderTests.cs | 8 +++++++- 2 files changed, 10 insertions(+), 1 deletion(-) diff --git a/src/ImageSharp/Formats/Pbm/PlainEncoder.cs b/src/ImageSharp/Formats/Pbm/PlainEncoder.cs index 75e6f56e9..2868922ea 100644 --- a/src/ImageSharp/Formats/Pbm/PlainEncoder.cs +++ b/src/ImageSharp/Formats/Pbm/PlainEncoder.cs @@ -66,6 +66,9 @@ namespace SixLabors.ImageSharp.Formats.Pbm { WriteBlackAndWhite(configuration, stream, image); } + + // Write EOF indicator, as some encoders expect it. + stream.WriteByte(Space); } private static void WriteGrayscale(Configuration configuration, Stream stream, ImageFrame image) diff --git a/tests/ImageSharp.Tests/Formats/Pbm/PbmEncoderTests.cs b/tests/ImageSharp.Tests/Formats/Pbm/PbmEncoderTests.cs index 2ca49a39d..e9b496ce4 100644 --- a/tests/ImageSharp.Tests/Formats/Pbm/PbmEncoderTests.cs +++ b/tests/ImageSharp.Tests/Formats/Pbm/PbmEncoderTests.cs @@ -72,7 +72,13 @@ namespace SixLabors.ImageSharp.Tests.Formats.Pbm using (var memStream = new MemoryStream()) { input.Save(memStream, options); - memStream.Position = 0; + + // EOF indicator for plain is a Space. + memStream.Seek(-1, SeekOrigin.End); + int lastByte = memStream.ReadByte(); + Assert.Equal(0x20, lastByte); + + memStream.Seek(0, SeekOrigin.Begin); using (var output = Image.Load(memStream)) { PbmMetadata meta = output.Metadata.GetPbmMetadata();