From 46a8485db45d3ca4520fc06e68519b21fe428681 Mon Sep 17 00:00:00 2001
From: Jonas Jenwald <jonas.jenwald@gmail.com>
Date: Mon, 22 Jun 2015 13:33:15 +0200
Subject: [PATCH] Ignore paint form XObject when the name is missing (issue
 4558)

Fixes 4558 (since the font issues already appear to be fixed).
---
 src/core/evaluator.js       |  4 ++++
 test/unit/evaluator_spec.js | 11 +++++++++++
 2 files changed, 15 insertions(+)

diff --git a/src/core/evaluator.js b/src/core/evaluator.js
index be26159ae..ed137f805 100644
--- a/src/core/evaluator.js
+++ b/src/core/evaluator.js
@@ -658,6 +658,10 @@ var PartialEvaluator = (function PartialEvaluatorClosure() {
               }
               // eagerly compile XForm objects
               var name = args[0].name;
+              if (!name) {
+                warn('XObject must be referred to by name.');
+                continue;
+              }
               if (imageCache[name] !== undefined) {
                 operatorList.addOp(imageCache[name].fn, imageCache[name].args);
                 args = null;
diff --git a/test/unit/evaluator_spec.js b/test/unit/evaluator_spec.js
index bc4ba12ae..412a835dd 100644
--- a/test/unit/evaluator_spec.js
+++ b/test/unit/evaluator_spec.js
@@ -227,5 +227,16 @@ describe('evaluator', function() {
         expect(result.fnArray[3]).toEqual(OPS.restore);
       });
     });
+    it('should skip paint form XObject if name is missing', function() {
+      var evaluator = new PartialEvaluator(new PdfManagerMock(),
+                                           new XrefMock(), new HandlerMock(),
+                                           'prefix');
+      var stream = new StringStream('/ Do');
+      runOperatorListCheck(evaluator, stream, new ResourcesMock(),
+          function (result) {
+        expect(result.argsArray).toEqual([]);
+        expect(result.fnArray).toEqual([]);
+      });
+    });
   });
 });