[maxp] Attempt to improve fixup of bad 'maxp' version numbers.

If the version number is less than 0x00010000 (e.g. an example reported at
mozilla/pdf.js#16839 apparently has 0x0100),
it will be treated as version 0.5 and only the num_glyphs field is kept.

However, this can prevent the font working on Windows, which apparently
requires the full maxp table for truetype fonts.

So if the version number was bad, but there is in fact enough data to
parse as version 1.0, let's try correcting it to 1.0 rather than 0.5.

src/cff.cc

@@ -1250,6 +1250,11 @@ bool OpenTypeCFF::Parse(const uint8_t *data, size_t length) {
   if (!maxp) {
     return Error("Required maxp table missing");
   }
+  if (this->maxp->version_1) {
+    Warning("Fixing incorrect maxp version for CFF font");
+    this->maxp->version_1 = false;
+  }
+
   const uint16_t num_glyphs = maxp->num_glyphs;
   const size_t sid_max = string_index.count + kNStdString;
   // string_index.count == 0 is allowed.

src/maxp.cc

@@ -29,6 +29,23 @@ bool OpenTypeMAXP::Parse(const uint8_t *data, size_t length) {
     return Error("numGlyphs is 0");
   }
 
+  // Per https://learn.microsoft.com/en-gb/typography/opentype/spec/maxp,
+  // the only two 'maxp' version numbers are 0.5 (for CFF/CFF2) and 1.0
+  // (for TrueType).
+  if (version == 0x00005000) {
+    this->version_1 = false;
+    return true;
+  }
+
+  if (version != 0x00010000) {
+    Warning("Unrecognized version %08x, attempting to fix", version);
+    // If there's at least enough data to read as a version 1.0 table,
+    // we'll try that.
+    if (length >= 32) {
+      version = 0x00010000;
+    }
+  }
+
   if (version >> 16 == 1) {
     this->version_1 = true;
     if (!table.ReadU16(&this->max_points) ||