From: =?utf-8?b?TWljaGFsIMSMaWhhxZk=?= <michal@cihar.com>
Date: Tue, 16 Sep 2025 15:00:09 +0200
Subject: fix(mo): corrected parsing of version

The real-life MO files have version = 0, but based on the specification
only the major version changes should break parsing.

Fixes #3072
---
 tests/translate/storage/test_mo.py |  8 ++++++++
 translate/storage/mo.py            | 10 ++++------
 2 files changed, 12 insertions(+), 6 deletions(-)

diff --git a/tests/translate/storage/test_mo.py b/tests/translate/storage/test_mo.py
index 3bbcb2e..8d2729e 100644
--- a/tests/translate/storage/test_mo.py
+++ b/tests/translate/storage/test_mo.py
@@ -8,6 +8,9 @@ from translate.tools import pocompile
 
 from . import test_base
 
+MO_BIG_ENDIAN = b"\x95\x04\x12\xde\x00\x00\x00\x2a\x00\x00\x00\x02\x00\x00\x00\x1c\x00\x00\x00,\x00\x00\x00\x05\x00\x00\x00<\x00\x00\x00\x06\x00\x00\x00P\x00\x00\x00\x07\x00\x00\x00W\x00\x00\x00\x03\x00\x00\x00_\x00\x00\x00\x06\x00\x00\x00c\x00\x00\x00\x01\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x02simple\x00unicode\x00Een\x00\xe2\x80\xa0wee\x00"
+MO_LITTLE_ENDIAN = b"\xde\x12\x04\x95\x2a\x00\x00\x00\x02\x00\x00\x00\x1c\x00\x00\x00,\x00\x00\x00\x05\x00\x00\x00<\x00\x00\x00\x06\x00\x00\x00P\x00\x00\x00\x07\x00\x00\x00W\x00\x00\x00\x03\x00\x00\x00_\x00\x00\x00\x06\x00\x00\x00c\x00\x00\x00\x01\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x02\x00\x00\x00simple\x00unicode\x00Een\x00\xe2\x80\xa0wee\x00"
+
 
 class TestMOUnit(test_base.TestTranslationUnit):
     UnitClass = mo.mounit
@@ -575,3 +578,8 @@ class TestMOFile(test_base.TestTranslationStore):
             # Verify parsing of generated files
             self.StoreClass.parsefile(MO_POCOMPILE)
             self.StoreClass.parsefile(MO_MSGFMT)
+
+    def test_endian_version_parsing(self) -> None:
+        store_big = self.StoreClass(MO_BIG_ENDIAN)
+        store_little = self.StoreClass(MO_LITTLE_ENDIAN)
+        assert store_big.units == store_little.units
diff --git a/translate/storage/mo.py b/translate/storage/mo.py
index 22dea26..474361d 100644
--- a/translate/storage/mo.py
+++ b/translate/storage/mo.py
@@ -252,19 +252,17 @@ class mofile(poheader.poheader, base.TranslationStore):
         else:
             raise ValueError("This is not an MO file")
         (
-            _magic,
-            version_maj,
-            version_min,
+            version,
             lenkeys,
             startkey,
             startvalue,
             sizehash,
             offsethash,
-        ) = struct.unpack(f"{endian}LHHiiiii", content[: (7 * 4)])
+        ) = struct.unpack(f"{endian}iiiiii", content[4 : (7 * 4)])
         return (
             endian,
-            version_maj,
-            version_min,
+            version >> 16,
+            version & 0xFF,
             lenkeys,
             startkey,
             startvalue,