From 0b08485708df38d353df07368a9d9b5456c5c764 Mon Sep 17 00:00:00 2001
From: James Jackson-South <james_south@hotmail.com>
Date: Wed, 8 Mar 2017 22:07:23 +1100
Subject: [PATCH] Fix #127

Stop processing once we hit the end marker.
---
 src/ImageSharp.Formats.Png/PngDecoderCore.cs | 16 +++++++---------
 1 file changed, 7 insertions(+), 9 deletions(-)

diff --git a/src/ImageSharp.Formats.Png/PngDecoderCore.cs b/src/ImageSharp.Formats.Png/PngDecoderCore.cs
index a7765342e9..fd03ed39b8 100644
--- a/src/ImageSharp.Formats.Png/PngDecoderCore.cs
+++ b/src/ImageSharp.Formats.Png/PngDecoderCore.cs
@@ -109,6 +109,11 @@ namespace ImageSharp.Formats
         /// </summary>
         private byte[] paletteAlpha;
 
+        /// <summary>
+        /// A value indicating whether the end chunk has been reached.
+        /// </summary>
+        private bool isEndChunkReached;
+
         /// <summary>
         /// Initializes static members of the <see cref="PngDecoderCore"/> class.
         /// </summary>
@@ -158,18 +163,11 @@ namespace ImageSharp.Formats
             this.currentStream = stream;
             this.currentStream.Skip(8);
 
-            bool isEndChunkReached = false;
-
             using (MemoryStream dataStream = new MemoryStream())
             {
                 PngChunk currentChunk;
-                while ((currentChunk = this.ReadChunk()) != null)
+                while (!this.isEndChunkReached && (currentChunk = this.ReadChunk()) != null)
                 {
-                    if (isEndChunkReached)
-                    {
-                        throw new ImageFormatException("Image does not end with end chunk.");
-                    }
-
                     try
                     {
                         switch (currentChunk.Type)
@@ -199,7 +197,7 @@ namespace ImageSharp.Formats
                                 this.ReadTextChunk(currentImage, currentChunk.Data, currentChunk.Length);
                                 break;
                             case PngChunkTypes.End:
-                                isEndChunkReached = true;
+                                this.isEndChunkReached = true;
                                 break;
                         }
                     }