Merge pull request #1849 from Exiv2/mergify/bp/main/pr-1848

Replace assertion with an error message (backport #1848)
Exiv2 · Aug 6, 2021 · a1b65a7 · a1b65a7
2 parents 0dc5cdd + 9e160a4
commit a1b65a7
Show file tree

Hide file tree

Showing 3 changed files with 22 additions and 1 deletion.
diff --git a/src/tiffcomposite_int.cpp b/src/tiffcomposite_int.cpp
@@ -1545,7 +1545,11 @@ namespace Exiv2 {
         for (auto&& component : components_) {
             if (component->tag() == 0x014a) {
                 // Hack: delay writing of sub-IFD image data to get the order correct
-                assert(pSubIfd == 0);
+#ifndef SUPPRESS_WARNINGS
+                if (pSubIfd != 0) {
+                    EXV_ERROR << "Multiple sub-IFD image data tags found\n";
+                }
+#endif
                 pSubIfd = component;
                 continue;
             }

diff --git a/test/data/issue_1847_poc.jpg b/test/data/issue_1847_poc.jpg
diff --git a/tests/bugfixes/github/test_issue_1847.py b/tests/bugfixes/github/test_issue_1847.py
@@ -0,0 +1,17 @@
+# -*- coding: utf-8 -*-
+
+from system_tests import CaseMeta, path, check_no_ASAN_UBSAN_errors
+
+class TiffMnEntryDoCountInvalidTiffType(metaclass=CaseMeta):
+    """
+    Test for the bug described in:
+    https://github.com/Exiv2/exiv2/issues/1847
+    """
+    url = "https://github.com/Exiv2/exiv2/issues/1847"
+
+    filename = path("$data_path/issue_1847_poc.jpg")
+    commands = ["$exiv2 -pS $filename"]
+    stderr = [""]
+    retval = [0]
+
+    compare_stdout = check_no_ASAN_UBSAN_errors