Merge pull request #12753 from Snuffleupagus/issue-12752

Ignore, rather than throwing on, Coding style component (COC) markers in JPEG 2000 images (issue 12752)
mozilla · Dec 18, 2020 · b7fc916 · b7fc916
2 parents f2378eb + 48a76ae
commit b7fc916
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 4 deletions.
diff --git a/src/core/jpx.js b/src/core/jpx.js
@@ -392,17 +392,16 @@ var JpxImage = (function JpxImageClosure() {
               length = tile.dataEnd - position;
               parseTilePackets(context, data, position, length);
               break;
+            case 0xff53: // Coding style component (COC)
+              warn("JPX: Codestream code 0xFF53 (COC) is not implemented.");
+            /* falls through */
             case 0xff55: // Tile-part lengths, main header (TLM)
             case 0xff57: // Packet length, main header (PLM)
             case 0xff58: // Packet length, tile-part header (PLT)
             case 0xff64: // Comment (COM)
               length = readUint16(data, position);
               // skipping content
               break;
-            case 0xff53: // Coding style component (COC)
-              throw new Error(
-                "Codestream code 0xFF53 (COC) is not implemented"
-              );
             default:
               throw new Error("Unknown codestream code: " + code.toString(16));
           }

diff --git a/test/pdfs/issue12752.pdf.link b/test/pdfs/issue12752.pdf.link
@@ -0,0 +1 @@
+https://github.com/mozilla/pdf.js/files/5715933/WE2330Ausweis_2.pdf
diff --git a/test/test_manifest.json b/test/test_manifest.json
@@ -2045,6 +2045,14 @@
        "link": true,
        "type": "eq"
     },
+    {  "id": "issue12752",
+       "file": "pdfs/issue12752.pdf",
+       "md5": "9f8ada17a613d18919714baf684e165f",
+       "rounds": 1,
+       "lastPage": 1,
+       "link": true,
+       "type": "eq"
+    },
     {  "id": "issue7872",
        "file": "pdfs/issue7872.pdf",
        "md5": "81781dfecfcb7e9cd9cc7e60f8b747b7",
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		https://github.com/mozilla/pdf.js/files/5715933/WE2330Ausweis_2.pdf