From eb7d60b681b2c5ad49de2066e6bc4a9c440458bc Mon Sep 17 00:00:00 2001
From: popow <popow@gmx.de>
Date: Sat, 15 Dec 2018 16:18:55 +0100
Subject: [PATCH] added parsing of the OS22XBITMAPHEADER short variant (16
 bytes)

---
 src/ImageSharp/Formats/Bmp/BmpDecoderCore.cs |  5 ++++
 src/ImageSharp/Formats/Bmp/BmpInfoHeader.cs  | 24 +++++++++++++++++++-
 2 files changed, 28 insertions(+), 1 deletion(-)

diff --git a/src/ImageSharp/Formats/Bmp/BmpDecoderCore.cs b/src/ImageSharp/Formats/Bmp/BmpDecoderCore.cs
index ef3ca24ee..f6eb78ea9 100644
--- a/src/ImageSharp/Formats/Bmp/BmpDecoderCore.cs
+++ b/src/ImageSharp/Formats/Bmp/BmpDecoderCore.cs
@@ -510,6 +510,11 @@ namespace SixLabors.ImageSharp.Formats.Bmp
                 // 12 bytes
                 this.infoHeader = BmpInfoHeader.ParseCore(buffer);
             }
+            else if (headerSize == BmpInfoHeader.Os2Short)
+            {
+                // 16 bytes
+                this.infoHeader = BmpInfoHeader.ParseOs2Short(buffer);
+            }
             else if (headerSize >= BmpInfoHeader.Size)
             {
                 // >= 40 bytes
diff --git a/src/ImageSharp/Formats/Bmp/BmpInfoHeader.cs b/src/ImageSharp/Formats/Bmp/BmpInfoHeader.cs
index 4dd63a962..a32f806a9 100644
--- a/src/ImageSharp/Formats/Bmp/BmpInfoHeader.cs
+++ b/src/ImageSharp/Formats/Bmp/BmpInfoHeader.cs
@@ -26,6 +26,11 @@ namespace SixLabors.ImageSharp.Formats.Bmp
         /// </summary>
         public const int CoreSize = 12;
 
+        /// <summary>
+        /// Defines the size of the short variant of the OS22XBITMAPHEADER data structure in the bitmap file.
+        /// </summary>
+        public const int Os2Short = 16;
+
         /// <summary>
         /// Defines the size of the biggest supported header data structure in the bitmap file.
         /// </summary>
@@ -143,7 +148,7 @@ namespace SixLabors.ImageSharp.Formats.Bmp
         /// <summary>
         /// Parses the BITMAPCOREHEADER consisting of the headerSize, width, height, planes, and bitsPerPixel fields (12 bytes).
         /// </summary>
-        /// <param name="data">The data to parse,</param>
+        /// <param name="data">The data to parse.</param>
         /// <returns>Parsed header</returns>
         /// <seealso href="https://msdn.microsoft.com/en-us/library/windows/desktop/dd183372.aspx"/>
         public static BmpInfoHeader ParseCore(ReadOnlySpan<byte> data)
@@ -156,6 +161,23 @@ namespace SixLabors.ImageSharp.Formats.Bmp
                 bitsPerPixel: BinaryPrimitives.ReadInt16LittleEndian(data.Slice(10, 2)));
         }
 
+        /// <summary>
+        /// Parses a short variant of the OS22XBITMAPHEADER. It is identical to the BITMAPCOREHEADER, except that the width and height
+        /// are 4 bytes instead of 2.
+        /// </summary>
+        /// <param name="data">The data to parse.</param>
+        /// <returns>Parsed header</returns>
+        /// <seealso href="https://www.fileformat.info/format/os2bmp/egff.htm"/>
+        public static BmpInfoHeader ParseOs2Short(ReadOnlySpan<byte> data)
+        {
+            return new BmpInfoHeader(
+                headerSize: BinaryPrimitives.ReadInt32LittleEndian(data.Slice(0, 4)),
+                width: BinaryPrimitives.ReadInt32LittleEndian(data.Slice(4, 4)),
+                height: BinaryPrimitives.ReadInt32LittleEndian(data.Slice(8, 4)),
+                planes: BinaryPrimitives.ReadInt16LittleEndian(data.Slice(12, 2)),
+                bitsPerPixel: BinaryPrimitives.ReadInt16LittleEndian(data.Slice(14, 2)));
+        }
+
         public unsafe void WriteTo(Span<byte> buffer)
         {
             ref BmpInfoHeader dest = ref Unsafe.As<byte, BmpInfoHeader>(ref MemoryMarshal.GetReference(buffer));