From bbda42110b1265bfacf4871b4ecc5c8ec8941499 Mon Sep 17 00:00:00 2001
From: Yury Delendik <ydelendik@mozilla.com>
Date: Fri, 1 Nov 2013 17:13:31 -0500
Subject: [PATCH] Skipping empty font tables

---
 src/core/fonts.js       |   3 +++
 test/pdfs/.gitignore    |   1 +
 test/pdfs/issue1171.pdf | Bin 0 -> 1939 bytes
 test/test_manifest.json |   6 ++++++
 4 files changed, 10 insertions(+)
 create mode 100644 test/pdfs/issue1171.pdf

diff --git a/src/core/fonts.js b/src/core/fonts.js
index 5b107f8e5..4a623911b 100644
--- a/src/core/fonts.js
+++ b/src/core/fonts.js
@@ -3672,6 +3672,9 @@ var Font = (function FontClosure() {
         if (VALID_TABLES.indexOf(table.tag) < 0) {
           continue; // skipping table if it's not a required or optional table
         }
+        if (table.length === 0) {
+          continue; // skipping empty tables
+        }
         tables[table.tag] = table;
       }
 
diff --git a/test/pdfs/.gitignore b/test/pdfs/.gitignore
index 8463dee42..ccdc06242 100644
--- a/test/pdfs/.gitignore
+++ b/test/pdfs/.gitignore
@@ -31,6 +31,7 @@
 !issue1905.pdf
 !rotated.pdf
 !issue1249.pdf
+!issue1171.pdf
 !smaskdim.pdf
 !endchar.pdf
 !type4psfunc.pdf
diff --git a/test/pdfs/issue1171.pdf b/test/pdfs/issue1171.pdf
new file mode 100644
index 0000000000000000000000000000000000000000..c2c5edd48e6306a9108beb0bb5fd31444670328a
GIT binary patch
literal 1939
zcma)7Yfuwc7)7h7J4M7|QEG9kQeO@GNH!rwv0y~BJYoofSS(AjgkZ=*vS?7Mh@!0>
zM6GoYkOES*jA9+sih?gtim%LQs{uvSsYS4W$N;v2_TCMFSpRfqGPASidz^c|lka-U
zWdQ=hOM(zMh2kh1n}E!ni(tysBnHJ6&>XE}H3%lBHH;A@a4`r`Aill`qgTUF>z6l^
z_*euBX4G2RpEaQ?Kn{+QIF3>{8jWBJ*c3g75(xIWR&4|*flFXj0NK}t-`=hZnHU(D
zK0pOiFh(}TpaSe5?En^F{0jj>ykT8fmH&apdJg2mb4YE-;5lHdt)D+_WcW&0K%_j%
zPd+D@W{jMH1@*NCBex))HlRYGwVRAFstnpBjy0eXlCO#d(Y6>7f`z5Ta=eyGLka^`
z1=_OVdaa68GpJBx&9JHsTOwG5R?Wp5QSl)8L5h&Hjbad^*KqNukc3AA+!#6$@mGR*
z0HIN(3~(d(w>V_xN3&360<usEXEqQyP?9HXtLSG`@xqEI378R`6v${b@jz%&N+B3L
zB}4^MZyeOt(HbKP4hI?gTW2f~p#loD7m`wxq)0euhz4kN42cS%wG`GbA#@^e)_;Xv
z42-5#`RO$}1_d30X%nvlN>DfnMC6#nW$^ouP(!*SSmXd#sf4ie5HGInbbtrP7(ibg
z%`q|ts*hj;B9ge+PDN8gwESv9H9oHL>71S%kBt2MtfG<;Y4M8IKi_fH>PB=gIsACj
z>$AJu6;Eil)p44hqR{>oX5XE!cEpUD?mo}w!uVy!3Vl8{w<b+*PF$;`?$*nGEH_#9
zY_4b~<acLJ?cdwtSr>37!mZsj%Mw1hrb~9_uJ+KcKVN>4H7Wnl(&$BFRVUX;I%DH@
zl->NQgY39<(p-9O`%yv0_7lE+W$I1U<Fhg+78EudXldxG2#e{>o*h`<C#@@+ni@Fo
z=xMhv&8GF}mPjSF-r@CWRqb0d=Y0iLS%C@(DR7_=jUKQAKc)X8m<U?JDjtD~rS={)
zH6-pduX@kVwYDj*tzBK-*f>!VY}ws#LaT6}usOH8Fl*$I(MVNz)Y98IV$0#>4f4{A
z(H@nqZRZ|2KP!u1@5^fPM!I0zw$|vH+C$_e)z>Qf)0)=#2s?%iKR<J>;}|bd8s^}1
z<Gbetdm1h6A^v2~{zO4!@9W3iJ^d%x>=$Pr{BY#Jn&zj~m&Q5FcexWay=0WdG3Nc`
zpB}GSW9fb7+*$wRx77z9ZXD+PH8%cW8au;b(g!sW-^}PpUEfuEuhlEr=TTw#?@mqW
zOM=|5jmW!>R*YRIXwOVP_Tk9PSMNS>n3g%l?0CCq5tiAKZ*+XR>6faiD(^XeIZs@<
zF*|9}`6I!&Z?@O<%8O2agi(?UxvK)dkX|Ydb(vOS?3#9P>+@T&b-Mz{leAhSoWC-1
zW&|hL_kH&#&D%a*TB>fll5^PSlJ||{PSo{^YvI8Y?my2d_L*`jyfioH*71y;QT3-f
zU;e#Y*d7<M@`{_Fz$am9RINs{JsHWPmM;lyx$NT9W=J`GUU2oZJ>TA2Og&=GmFk*E
zbN}%0Z7+NDy>;b96R-}K(&T-zg1+Jn89nBQjn5ABEmU@Q?1<TWu{HhF<Q|o0tIE~$
zVWLa_hLB<=)1z0@`OnQO7Zu&PckZ?7S8r+FitYE9LYCA_cBrr<q`qY@lYxmtaIoF+
z!3Pyn6tGsD69WE`gp&=C2*I@gA5;$zY6u~4Xha(V{5qIJ;Bcke5U{-l5uCC|;8ivR
z;c(mAlHk2z2Z_PT9)cux5Li6+kR1!0d^1aGmn9<YsB;DyY(E2kpN47IGN=^66f6rb
YFY7+!S2u0oc+m(kK_Q-=3qu3`1vo{lssI20

literal 0
HcmV?d00001

diff --git a/test/test_manifest.json b/test/test_manifest.json
index b66c4cd98..dec722231 100644
--- a/test/test_manifest.json
+++ b/test/test_manifest.json
@@ -1343,6 +1343,12 @@
       "rounds": 1,
       "type": "eq"
     },
+    {  "id": "issue1171.pdf",
+       "file": "pdfs/issue1171.pdf",
+       "md5": "2a6188a42a5874c7874b88eebd4acaf0",
+       "rounds": 1,
+       "type": "eq"
+    },
     {  "id": "issue3521.pdf",
        "file": "pdfs/issue3521.pdf",
        "md5": "df95d31443e20a38efa29c3a635a045b",