From aceffbe29d34e0efeafeb1f7b56eba1bf5b600a4 Mon Sep 17 00:00:00 2001 From: Nick Burch Date: Mon, 10 Nov 2008 12:20:49 +0000 Subject: [PATCH] Fix for bug #46137 - continue records after eofrecord git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@712652 13f79535-47bb-0310-9956-ffa450edef68 --- src/documentation/content/xdocs/changes.xml | 1 + src/documentation/content/xdocs/status.xml | 1 + .../apache/poi/hssf/record/RecordFactory.java | 4 ++++ .../org/apache/poi/hssf/data/46137.xls | Bin 0 -> 51712 bytes .../org/apache/poi/hssf/usermodel/TestBugs.java | 10 ++++++++++ 5 files changed, 16 insertions(+) create mode 100644 src/testcases/org/apache/poi/hssf/data/46137.xls diff --git a/src/documentation/content/xdocs/changes.xml b/src/documentation/content/xdocs/changes.xml index 0d04e7ef44..259caeae30 100644 --- a/src/documentation/content/xdocs/changes.xml +++ b/src/documentation/content/xdocs/changes.xml @@ -37,6 +37,7 @@ + 46137 - Handle odd files with a ContinueRecord after EOFRecord Fixed problem with linking shared formulas when ranges overlap 45784 - More fixes to SeriesTextRecord 46033 - fixed TableCell to correctly set text type diff --git a/src/documentation/content/xdocs/status.xml b/src/documentation/content/xdocs/status.xml index 430a136bfb..3cef6f652c 100644 --- a/src/documentation/content/xdocs/status.xml +++ b/src/documentation/content/xdocs/status.xml @@ -34,6 +34,7 @@ + 46137 - Handle odd files with a ContinueRecord after EOFRecord Fixed problem with linking shared formulas when ranges overlap 45784 - More fixes to SeriesTextRecord 46033 - fixed TableCell to correctly set text type diff --git a/src/java/org/apache/poi/hssf/record/RecordFactory.java b/src/java/org/apache/poi/hssf/record/RecordFactory.java index bae867e512..da8424e466 100644 --- a/src/java/org/apache/poi/hssf/record/RecordFactory.java +++ b/src/java/org/apache/poi/hssf/record/RecordFactory.java @@ -383,6 +383,10 @@ public final class RecordFactory { //Gracefully handle records that we don't know about, //that happen to be continued records.add(record); + } else if (lastRecord instanceof EOFRecord) { + // This is really odd, but excel still sometimes + // outputs a file like this all the same + records.add(record); } else { throw new RecordFormatException("Unhandled Continue Record"); } diff --git a/src/testcases/org/apache/poi/hssf/data/46137.xls b/src/testcases/org/apache/poi/hssf/data/46137.xls new file mode 100644 index 0000000000000000000000000000000000000000..5fc399dd6214458fb97055f4bba3ab0ca5f469ec GIT binary patch literal 51712 zcmeHw2V4`&^Y|qMh=3A66blfrA<_f{1O%igND;waX#r816vYbmu88#^Dk}Dd*179D!9{2{}8-i~HzA<=x@CM)w!Slfz zfj0)<1bkEQ&A>MY-vYb|_?F;J!M6f$2HqUJ1$ayFR^VHMw+3$m-WGft@OI$s!8?F& z3*HgD6L@Fv?ZCT$cLnbTzCCz%@E+hj!Fz%40KOx5Z}2|gJAwBF?+4x=d;s{)-~+*T z0UrcD7<^ao-N1JT-vfM4@V&tI2HyvKU-13F_Xj@!JhmJ3VT}8~TXcemc=#nlU3d|3 za7`qJLvE_?>Vd@2d1ga0J{SK9;vo4?vXZ)kf4;}?+!IzOSfOx#5Cslg51^h%3?{?5HTc&i*esM=IuF%JiaE62L&a2vnoA*K;TKAl-3Pc3N#+S5$wV~V$0Wj#US8#{r#G z3-pmX=%Q?h!2|t*F+d97Ux)nPLZ;bA{b2QgTn zWA`De66=Lw*bcPGjf^pw~BUYTU%ENfWC9 zEuH$B(1E@!4LC+%S2UJ*-X};9(N8jxF&G|TBGBn9tR(S37*Vx=j2KJDlENm8Y7f=} z*arhJE2expXTo4#6B5w3rIQd~Y;Fws9$?}&0H|1i@d4(?025|9Q$911DPJOmE0`!? zY%qyYC6J^?W{pA8)m2Q?4lEt8X+%Wn^dh=+Vv@8tWf^IG5yZPnKGZes4)PI>_8bX+ z`y2i%_<&qw5qz4Npbe$?yhHeivrLlzBKRgb@bz-wo8`b4%Yko^1K%hIzC;dux*WK? z{K&(0*e}o96qvf-u9JrYr zxT#Dy*n$*XN}u5PP%{6NJmZ*=gkL0wOfdAL@`K|@1{}~*a2#6-;e5D-pcGt+E{;RV z_#*yL@{eOvA)XMpZAjz6@hS-y$>9be--PgNEvx+V1t8tmwVdIrbVBpd|>oe+-lK*9&|0@FF^ z974s#knR_Ra2^FWt0x=2Oj8O@02UY@Q1Qk1u^wgN$0-4XOXH{E()a;4I*31tTbCF` zmxly;6W_FK7LVqd6Qd>I_>c|r5Ov8h7aHei6hL=sS_do;cGU!w{a%9vl!04=1UUGs zQI?RYHA;ZfGBrwoeSVD+sMa(AR!WU1pc<=D0+hEJB|ybgqXek2YLoy~S&b5)j;m1u zRDKE*w3liRsQr}Ww+9Z{Py~@Ea+GDu=a1$}BSBSJ4J0Ur@{yoktp*YlP5DSr*;WGy zimQAisDZ121Vvar5>(gKK!Rc|9|`LAA0j#Jr?_@d;a3BR;@Z)x9umd1gVVXy$cf_G zfsVL3NEFu&PAFF+CyHwa#-j`*5RmLksJ3Hmqquf(N?V*0xkwb(4$g^J1Bv3=0gIwK zNEFu&zj{a%*N##3kSMMl7~=o9IMlL*{07*hdPo%4PSffkQCvICs)s~z?KH0*62-OC zqIyUa*N%w@3EE59gi57^9foZOA5m;c{Cv#$?MmzMMzYI zh||*2D)~akl>w{8fB|mOsmXwi=1OPE6yj2u5-w6y)e4x~RjXE2$&{n<|s}vVO6czRuEmGT_9~Io7FS{F0Ivwf<;XeSc(!L|F8ljM5d})Vg=DCt%wO= z`du;sEn`J5nKCAA+rV@hLycgSN>D}FL7-wK4|9k=MpEgj7zr*?ij};aoJe;!()DT! zNW|4t1xZzL#wAg48*_CXDw~Owr~jxT&cCjWNmmN4(TZ!fO2elcbY(hAl&PxXVK0Fs zTOda~DxYODkVv<72N8RamsQ4-w2W0&o)#gYRnWF=+p4s7CX8 z*_v5ruCnV)mR@A48=8$w7@aAmx}h`0gjH2RFtyE)yegckngAEw<%=q7G`E{nci;>O zIP%9b$am?c%51)LcThDaxa?2m1QiKXh9p0$Ccr%c;;dJrtgb&wFj}gtZjeAlR8|N` zswK+ylvGQ~jPlcErOS^9Xc$%}@`>q2bEPxT9v+J`P^E}e1|)|jR0b#*P3-40;9i{! zNRC{o3{bF|XCPXt3OpbK%AzXxX$~;iO=oH}SDKio1ThHwy85n4%u9mUPnS$q-Bu;m zp*qBVt{$tZh>k!^fr_ZE)XrsV!=xlk=U7`cxWjbFNqI(kNB`(C>(y&P7TViV8LDlRgNI?`h!=M%6O051_jd$XCsk}mNXgbs$`56 zkfBTul8mw-^7YLdvdL&mld-8vrnv$#lyO3mQ4vHweRWed869adwpGcPDjKwNOBY zDQTzuuH4@zn@j^~GBPBsl>#zMNjvR#`e3PSG7Y84$dEKk1!S0#cG@rh=407p8cCCp zA!#-W$S@`CwBOfDJ7kk-jAZN-OBx@@*i}Q)tZn9Rze15gg$7Qf)AXgu$dI(A3dk@e z&D!SL!uhhv7)X#zMNwc<@dq$|CrR&B>nv4udvsFNbDQVU=pATlpDvbtgz=4@6QNP(XpuBfQ7CLZK$%!44}}wetv# z2~UX$O^%L_%jAxgM98jJsSew-aqA^+lZ}Bp@wQx5(vm==I25J{;B*2>Kn3=y<3pUs zU;!JLA+AKImPBYSTNCz;!TO`eKtcFmzK0+tCX<^cm9P4hYLKsiU<`)C24+6o3m|tf zP;7CoaE_2P3Ym?R&J~WfRYbX>LUgXE5S=S3G?&dGTBR#hW^(zGSy5I3A|9}hnA*n- zv8g;zLMnx7rACz$lC*ST4?nD3dZi}BgvNy?$0rWwcNL^2XL37AqGGqH=! zbstC8KW-Wy4m|G)*+~U?z`;6Q`&ijR@u~%7gDJU=@$n$2PLg<84J$dY%^$bFONuJa zCepZo7>%nA?8bl@ac@CrL^SMxhOvTFMwNKLS#QV?%6>FtGzv0EjS)x@fCgjeGnzt& zqNoZvqZHv>b{!aGFpz`plSZy3139Jt5r6aU(vg82^&wSbs!X=bWvjsn4p@uviAzWUQAm-jGIm`k5AM)U09nLJ z7dJGVy`(D*r#Sy&^XGwq1bWFYI&QEaLfHO&r81)jddZKh=)q9w5fU0eNhFOTT|{di zoYGVQ#%aJA73e_&1S!dhp`c!*6f>(aDCPh_j3rG3nL@=vjc7=H3Uiutd9z{SQz-Bt zQt1RVM;pn!vIrm<0Fs_a9&|udLhiHRRyMAAd?#=iG!^M?FDfpL#A-v z1V=q&R$yk7R%tk^DOR*ph*oJ-h*oJ-Xf9hFj_AO;S?82wvc`u=5!D6Mok<>4t#hWz zM-3@a=LitzgbL9)p+a;{s1T|$$8=@jT~o;{C@Vun1VWh;MFW3i8H%bCifY&B{LY0;d zp@?b0aVHHpV-uPfhboIwudb3AWvhdx?g11MpAR$??R;V(!IjR;K+od_pB=50fm z8sQ1yq=5oKa%+S(+#vPJK`sAVNaN1|h*V4jK{SK|6$J%5aJf*V%}=xVcH_JmO;$ zVxmLi!UgWYnW521z!fK?jaw1ekOI{!(U6gX;-buSNKFSk3)10?EMZbLU1D@{ba-eC zRD_sd`?v@(J}1mMjRhAk2*WB^n04qr+(V4h35!Z4E@W|uLMTbpLl*Ij7{`L$P!%Ic zuBueH!Iia$IvR+(q;Rzz9X>|yg#mD6^lDxaM@{0_6#w5HADNEjfmTmmVKRWc;x;sd z+Yss%+Hfs^Cc`Dhvf4x2ImmK@e>cM56>cSi6kJHng*`{y4{*lwD=}LMH>aRzL3ia5 zACnRr=ODUk3ldJjT3f*mBW!Z~;P{lpZ~@;WNRSYpn9T1M zfFBSVD*!5iiGqa0_;5i|QhXv14vbGq7KKR?v9T~M3z@VG9vGSkJu!tc$uf9&QnDb{ zk`ik^Q?e|@Ldg#s4v(nj5n3Z0Lc9za<(ouC$0Q395od%DCqFbUf*&qwi!HnS6b(^& zCJ8#*EqnY7PESyV(J@JwR|YpQw~P|`{}g719wyO?TX9_!gsOVU1R5r(!WpFHC=u8f zCX6ZrPYe2?uyCM3M6pm6i8Ep-ipj{Vgu(&sL18iDCx?c?z%x-7jyI;n#)c*iH{)A- z@)O{kV>n1$GVC(z8lS)qFykjBhlZmq7)~|->M1i76E12UwxYW>qB}d$ojtTxp`>AZ zfYv8265mfSR1gD7F<20u0%aS{4~kA2%nuHaPZU(b+zXd5hmkrq^IW)&hwD06$esdf zI2zQiHz@URZawhuKYH7J-O7GQJ%z^ zQFS)dKR9#nxY+6pi$X!2P@Uk(N)7mrfQ|fkuLb{jcAQImUH9sOe^{XF01|FW!dsF5 z<48z4#Gzd?FwfuwIBP6CDdZq6U_O)^aZMoqn?vkK(miy=17h^Oi3rPM*OrWXf8qrc z#vSY)w6Jg(pGfdw{N@K`z}e9@;5}iC>Ip_KpGYE~N5g*-P!u~Bht0l0@YD>0v0rBq zPu1eN5pY$7bEX)^e>^!Z02*jGCyJB)&+qO-?-m&3(;c$^%N^)?aJ&}y!z)eThE{D{ zUI&MDjsG=w29Ofdl1_NT$s&kH#9N}4C;fnNCF3C1_p+y;nEb$Dr~eLSvt>jhF|~(U ziWb`3GU8MVM53n(FHsax0Or$wgh&-TTyT8wn2t;fr@i6!xBJb3zsvz#(V^c#@hh)1 zUTqkv((;ZbB<)bDDY#caM?arnZ~6gf;~;n+Lm=o&D=<|(VdUclZ_%I)d(_rJ@C zGpd=v^b*3XWWZX>fLY6c*~oy|%7C?z0ke|2Fy_g%t;2!Sqkz_13=1&kGi0@F~`ydzD`Y;Zv+j`Dw9M<)_8^m7f-CT7Fur zbNOkp_T{HVIgpMhZnMBTGGonv zHzdjbl@nfEXz>+CT9UH?6{vD>cEJ;FGzq1OjR4mqI29%UgSQG*;N+yVV-sp*I#ee( zkB;%*fkn1LgMvE?k0kd0{Uh7b;bH&RC7y%5_l5bLbr95!Huk+&&^LGY*GAiV2K4>7d3)AL zyTrxe@wUr07M~w}(5=C@0bjplIu$Qd} z{m(A1UcTRHweF8?sk2lr`>Rh|A$W8zr0m(Ko}<$<$DO}(WRAuSpOD-83!VqqhOX;q z{kiG-pcQ(0zCO!(PuFjmW1DSq+5c?Ykagh($?hWy$G%-1yVJ2U(rstR`js{rv*VPW zj_Q2r>ty|XXUuLaEJ{3gbK$Dv5ocH48;YDszo>LIBdo;5W{qkA(Zck#3ak4fuRWk5D^rUeck+%O09VN zhpnz`zj9<)ICqsnxu1I{%O!RtO~P3#WSl_y_0FV{P+p?dD)W!ndjjTh&xyw`B|p>1vUCT4oM zUQ>;7m^(A-+R-8A*|o|#&sgzc<5P7P{n;&6#vJPPr{%}g%+ZVdhie!1J=S~e#+D9I z1|M3jOUu!(|L*tHd>}5~d~QmiR!5KN(ff~myRtg!o|4{+VKCWs_PyGbRijdim4Y zL)ndWo8*k^)8!M-D5k)n^Saa(e8;GH8y5zxyK;Qo_Gg#ed*v&4tFYZ4urT`gcblx` z^;%b6oLKyFevbFN?-o{5Lyi0QRWGk?=Fu>&e7#ZQY>UPL*`2;_e%z>p$DO*n@L56Y z4{ko?E28r{#Gg)UTKj|j=XW3btedI-dUjCrS>MXOI80mg^7ycU6R#2%)or)ETk78X zj%9Jhx|@01&*|RyHj2~AVS9gbjg;n5UDDJIq7J9@n0?FaftBH$>_dUYx4WdSQPH-| zanhY=Y%Eavv#9=>*LH)ZH!JWeV2>Cv!DZyG!)qSZKY8AErupn?JJ>A(dvs~-QR(sG zROeFPf++h#F##;?9*RGD- z?H=)|8T0Cv-cDC-82I|@yLrk3%=&!-%b=<-Kla+sMk8e%cocUO=7{)VafyPVD9e2V z&j%cGyHTB4fsh@|w-@N0(-X_CZ)GfHRs-cz3i_^vB zr;F}&xD?T3(}o$5372Z6doO9!*gkJjuxWyE{5yWT##-qu%rRPWt?+w@_R(0s?a zizg1ywmlzrq`|yfTRP6$vv1zCy|a4x&g>hy@vcF~y}5^8n`dpvn$-T>nrS^6TFx6~ z9`F0dmU_#k73N->I=yi6x#;KJE#FK&cJ2P9(|S&>gD*7V_qY4L>+9vWB~R|V<#!J* z%u&tVb$kEyP7ybTM>hzukC|p}RQS<<$FNNxZ>u!ylUdJfo#zl=i zd8!Y+?&Kc(GtWt5?7KeI*e{>sWDWOQnrKh(jp;F z{ee?OKmvW0agcf2(xS9wNmJ>Hr(E_=_ut3_$u|U$Tr7aNHXC1DSUhXyaX?%O?g585lYSrF5bF2BAiYsw*OSiRi zi0@_{P}1wJW2>Iehq{04c4K+BirNExmtTxcpR&3r%Wm_oF5M1zXDC@5R_^(AV3z6G zhxf{dtDA>j$r{}zV5R<*7EKKF&U7p(-&g+y=XR7^>(oP&GXq-8^>I0CatH2K-y80- z(kJ<$;|1$Df1j4G68igmQ@=bU_``>rOJ)xKzQ16*=jCQgx~%A>zTV~Fr?Zj#YbP3H z7pfo844Jemr(NKPC&!f+Zq#dd-C+E|{#xS}_|&%T6`D1P=xKN6;U~RYQ#ya0x6+P# zc3S0>4A17N+qM^{={^kW&e5NLQSF72$K|vngH;Y5{A`;}mj|!A*k>&-v4`3GX6=HT z39{9nw=g0sGiKeHVsy|s`Fjp0v&-i-guaLWKF7ux2E3`Z&ZR^4-sBsuwKVCx*7svv zMWgxL^eH#e*)L~|p5U2$-j=s|SW%iv?U(kSpI3kd((b9|neuGjm?@XIBRcL2Ha)^Q z*!ST2O`6`(N(F&M3nLHT+O|3O#Rr}3qzt2&9n>W zOiKK2@T6a-raQ*I2(moRUECN ztwz1`8u>&sA{s0XUi*wQV>d=v4w)4?vSPVUx^Il`G47H>+6~t!4PIo}$$j*eMNvKd z2Y!lr&FOP-_h%RpR5dm3_d0aP6gsZ;a!iekKAqXEc@uB@f7Lv*=wtJn(+`fTyUj?g z$a8MLQ59R`_LfhI^(|#jFSvDQg{R@8)1#suo*sB%{iK*xDSw{VI21s5K0P$0w`GHU zmP=n~^NW+rQ^R*eKRo}mm(^od%ST^2_nAJ!$3>dJF5%E&#?+Y?ZVVq=&A;o=tf9Um6JcPXq>Fj#McEBZVXtV=E1AFkC>D4hb^vSUu8XB+)CNwk{ zJ3Sb?l$^|3xI$M>+$XJy`wXkXMUGWL*!&f%0ydJ6Rz)CbRdo7oRs6Ope$uKK z+3m%ESzuMT%C#y^#a*}J?W_OTmA9`&=90;=MnM^l{+3=kwkK3KwmjLq(;U+iCW!?H z_U1NL-CuXg_`dhwW!`8|?27lsS`PqaAJHtGVmPpj<~pY9z#b3WU~@ZiOq zO>>&v?sV32!`WpPlS*|%jubggGMZs|uFGqGZofH#{xK2i->(k6P&O#MpE|2@PfB;E z)S|LY#jA&pS^BBiXYIERCli)*Pt^@?QW`X9w60O+h9UX>3-b1yw6wnK7U`T`6kF1@ zWc#&37bcx>9#CFf*4tTU{7bC|yhVDQ#*Emu=zfV!@sa%d zE!LD=`W8jnCe=$Q=>D*NY4rKGU|9I(e>{J??dqqQXB}+Lt=Oo!t8%*j zRbQ(c9;!X{&dr=^l;Qm(g>UHL-r9Dv{&ph%czgAee1DUe5-X0cc47znzGmJH-6!95 zZ5T0?`@B3`^LB2#MRixYH5(Q*!JGH_rAvnPNba~d-8X#h5dF=-_(vq+Y|7 z9XR3XhFYdE*DkEf;SAi?qtDPyBfK;loDAB&D%NRJAm8Vn@sZ_YdA5#4GoD|l`)GZ~ z_)(pA92!$FiMZJB{-;OHrX9!~`3{yXs@fL8Dz!J^G=^N;VnF9ZR&|epZBge;n^*ph z&8_UN%$b;@H?z4?_^6G2j@7$d@4DLI&MzMFmq5ON&-^w~H{JvFXpSbI()V zCY(&^vO#(L&QIUY@TR2n)!lc#lhsu<^_BNu8$ax_+}>2@*&ls)8eQz}YM4IiuC>Z; z%Y^&+D|@)#*?h4>;^dXR&K;_#J?Ye>&r9udGOwr&So^Wlj8=0jqi$!mcQlwV_tW9v z)dN;vFF(9`=W_et8T|(Q`9z=BqW3VnQzzbB`SLn`_v{g7dK|ZtPu`bY^eWnBLGyc*W?1Xwe=m$v-EZ4ld!@a`C{E(d^7eK6pSqwnIe&S5m2cmkHvRN1 zXS(*y2acn2zcslzeo4Z%p(DRn80t^xU+GrzrkBr*!=I+l9RMB4Kg+t1R!G>fQm#7I zg|tY}X$j)tkF5(=DeHnD@>q3=H!#iN!>UId1YFYAg$r00*ps`>s*ZJGF0wA*oB(ZI zhH$jsdmc^C_)7rp&o9N1kN2Im!h+%EG%CR=q+Dcm+aGb5$a~SqO4d*a|No&LZ zx3%%x+W1Lp!zlIW)P-PeWXZKQu7kCqG1@RMjraDJGK<8d}jIYJC6r>`#|{)_(19GSAzff6>(`b_;vo_M8&2eo5?wPCfg*k8Cg`N8p@3 zeLzaTI;&TncznM}aI(|hR$rcQ&1c)a{-ApJNyyvs=^ew$b#lttI{gZ_Bs@#Cy6ac2 z?y|3;-|FzCt3nP4YICF1i+dgX*87j!y8;5X_sbtswa>^8pF=!4&G z#~f>ZEM(W@I(g@oH+bg$-usCFZH-%(Hn*+Iy47Wz%Dnbj8Q1qjau--1Z*Vg&t<90^ z#t!%6bNRl`1EZde`e^a|TDG3D>8`oPo@4U9-BD_Gm(xDs)}U@TO*gq3t9IF)xLN=D zI12}^k-ztow6V!19ZZRJ=O*u8;MrmE$f><%=)ZaqmgA6DXMbw6<&6?mysD4Mz!lLO-_Im7988ISvcWU3eC2#6azP)Q*Ghjw?9eb@O$LHnj`#3)O?%;k-`MdJ3cDMWb?3*5c>g-{+Z@$~Sa>#~3 z+cGbA>tM6%v9eok@yxYv#}!_G_I4C;WcX&~Ei)HrT)t4~(AK5ltL0WRm!9hOus+9Q zN25fQRUb|nt+KFOb$4^rf}newthx`Gea>QS-oC01?;&~Tj+U-IGm{Rc0vdY$1Yne}(x=o-`P zA+DX2+4Y^)%Bb*jEVXu$^klu^jr<3=cCz|fNao@_Z;Zy6sW52f95m^G~bh{yA0@EU)&{{)Nw|15(;S|?%OO1bJ79MZx- zrzMD&KQ=hrs~8+*EEcSraEN*=C7J6cuDEUjoq0~N5$>WOo&0r^dZKj`u%E@&1k=!< zS4j|)nlL(j{=}ImB`u6nhK1oK&%&s(N>URG!vUzHh0%euFx-Ay7{4uypR_RCyN-33 z4i<)uTnpn2S{Mg3uR3w}3>$jm>!r2*lAr83m8U!S3o*^C#i583D=%KWRx*2DcK%g? zTkzYvR=G!B>ZF~uoK&2*guP|SrjPf7i~rbmp-pL=>6o-(qqp5Zts2@WG`pMb?N(qxKTNVm*X%#q zpMIs@^^^Kz_N(hz9Y~3E-0tL*wPSVjMh`q5m`#qG%<7V#ux(qbI;VC%d1s!%n>c^P z(C$O7t|#Wc`gF}(W7XjDzOnU&cv&5pw7e`Qysu`<+7r8Xm@unPtLeR;?9GR0_oW~Y+d&cjs9Q@+$z1dFwC20>` zvUKuoN=kx8COB4ngvS2QGAg9i5ZO_af7LN6q-8;z`reO?iZ)V41sA>y$|7dM8c7IS ziTuK?^el!O8Wqrgmz@+F70z-uV;GBOF`9^GF__1?r4Ac2r!I_z1#yOVOW^j4I0$D7 zH-NSm8W!QO0n`@YX#|J$fJL8v4}hJX)TYn$!?^v9+U|+(bi&tR;74)ER~ZO*pdcAO zytrW|9Lvijv|$QIdg?|Qw$p04VZ0p1)D3l9mM?K!HV|GM5eRwH(BQzeH#Zc1gSdma zOSr4yLr8!9a4T;0$IqftTm7MWAxHEUfBKpR8(DEzP4zeYQ=eHaCjx=b>hxrT~D1!xVfdH6^DkB1{;I zs$(kZOA#;Vq;y@P1fc5<_0t`A69K&1HV-~9&NF~JB_SuNIO2a~BE@ebC=1joLwYzM zJyj*aDlg^9B{RezT?+;Ks9IRcH6WHUY6my66La9y9Nz-JXKq8lQxl08*vc^mhu?*) za!!vR@`_-(PU-R?-;sY+3xxbLIpM|-`IZy*1XV4k*3j>vqjvA&ks1@|#m~S&wh>A?E(K-)mu$r75cwCxlE@M+-Nz_b6j?L?~C-?kHFeMcBjAG+bEOpk(p-5B0Lc^b&e-!O)UX(u}OziJGR)r#{?vc~ZEDviH3hQ}FR zaX-W*2ElCKLm0#FRB8@0L{-;6Jca#G$hv=8KNPZ#?tkd(ujq%$nElYApXi4m4Yo~L z>mUDyF}$pPNS4)|0A;M1_MF_G)>DU{ITv(nyTJ7dADU5SG=J4ZiqTsdjQH*FXg$)yel}erjF@&oRUX&cPCiw^- zCyzss?kP)8Ez*l`&Dmm;>Nmpf2pS6mH&t~8ohlMVoy;4ktHi2EJ z;z@Tpvl+rnh`0kpX=He1=8;A4cZ~#Fwj-(2k%|%9kWu-&zuPb=XMmyqB4!-ACJj$Wj!yM&c$*@DB z3nGkDNdVJG38?(wn!!%zQQ`&>k&i4XUf}LNcyb$K!bWLfTq-7W_=MGmF^6$mfG2rI z{^3qo*jo185xl%5N^wxM;k;zr94&$M0`1)l4pbMeN_-UiuYWl*EDUIzk8n z8!uh*>F?0r%>m(AKM`3p+W&99y+mfA1suqP8~hd%4j`KL6o+vEJ+rnrOn#e*@vSBt z7x@@a&*{S?)OZA^fARZB=#<6ZRw7|oUS+9o5@8OcVK`uvhT(uy8iur_VK^3J!thH$ z=v*Pk)OUk$_~iD($OeM&%I40&^lRWQ?&+urgCi&qLDugld zNDYRtESfNQr$ZPMzJw)whB0*^#A2&J$ykI7e0xV5zAcYm73=`h4)l+CU^j%hVWlZz zui=4RNQ1c|mwDkz5N_l@^;dP7mkn>p2O894N@$2m1Ov!t7sA3ub@*+iQXN`)ff5 zM1q<{1J%L$1+}m(|936axGnthT5#C)yB7ZMYvI7|-?dQVTA(@<)Jo`nppmFZzyJUL z-~d)S+PG+;;T#g$c4#xBrHmFa+PY|y;Wjz6JTVSh!+8De{{0;I4og_dER+CAhjLLi zaoOunod@zZ%;6HPT5P{&1+abKQ7-VgH+US&1cFCc1b7^|q=H9z$pDWslMNn^JFEqd zqop0-k>Y(wjNgUFIKcy>2Sp3`JwoH6P&`xeKBEW}h}=`L9C3u}xt4 zvD`S`>j@sefQ#kFa>szjye5Ok@{R(JZ@-)X9{Df>JeGMrAYt+W*g-lfzZxVi1Rzdy z(pLi@g0RNmsm-O#H)eT;CJhXW4^509^Xo524vh#+4kZ(K!cEdZfgriHWYAhNXd@Z4 zl?=8a$DZQ%f4UF%j2;@D6dfN&z88tgkj%#|MfqiXSWTR~eO!b%8NnAtW){wUI3p>c z0gOQAp)essdc-FVCd-#Z>NV75_y~EAyKf`-K_F%V>LJZ^c=j`n1)5L^6eMZ__~CBq-0wJV+C0_5O3RV8$u!c yU;MM+=x96<3ooeZ47-pA)tCgn5{#(@tAhaRA+b6Us~M-J^s9-s+R^^a^#32(Ulxi0 literal 0 HcmV?d00001 diff --git a/src/testcases/org/apache/poi/hssf/usermodel/TestBugs.java b/src/testcases/org/apache/poi/hssf/usermodel/TestBugs.java index 1f7648ed75..024f94e5b7 100644 --- a/src/testcases/org/apache/poi/hssf/usermodel/TestBugs.java +++ b/src/testcases/org/apache/poi/hssf/usermodel/TestBugs.java @@ -1523,4 +1523,14 @@ public final class TestBugs extends TestCase { assertEquals(64, green.getCellStyle().getFillBackgroundColor()); assertEquals("0:FFFF:0", p.getColor(11).getHexString()); } + + /** + * ContinueRecord after EOF + */ + public void test46137() { + // This used to break + HSSFWorkbook wb = openSample("46137.xls"); + assertEquals(7, wb.getNumberOfSheets()); + wb = HSSFTestDataSamples.writeOutAndReadBack(wb); + } } -- 2.39.5