From f9ebc1ae74d290098127f945b960556d2a64c0f9 Mon Sep 17 00:00:00 2001
From: Nick Burch <nick@apache.org>
Date: Mon, 12 Nov 2007 13:13:25 +0000
Subject: [PATCH] Fix for BOFRecord from files generated by access etc (bug
 #42794)

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@594102 13f79535-47bb-0310-9956-ffa450edef68
---
 src/documentation/content/xdocs/changes.xml   |   1 +
 src/documentation/content/xdocs/status.xml    |   1 +
 .../org/apache/poi/hssf/record/BOFRecord.java |  19 +++++--
 .../org/apache/poi/hssf/data/bug_42794.mdb    | Bin 0 -> 143360 bytes
 .../org/apache/poi/hssf/data/bug_42794.xls    | Bin 0 -> 10240 bytes
 .../apache/poi/hssf/record/TestBOFRecord.java |  50 ++++++++++++++++++
 6 files changed, 67 insertions(+), 4 deletions(-)
 create mode 100644 src/testcases/org/apache/poi/hssf/data/bug_42794.mdb
 create mode 100644 src/testcases/org/apache/poi/hssf/data/bug_42794.xls
 create mode 100644 src/testcases/org/apache/poi/hssf/record/TestBOFRecord.java

diff --git a/src/documentation/content/xdocs/changes.xml b/src/documentation/content/xdocs/changes.xml
index d2298ea8be..f88c1582f6 100644
--- a/src/documentation/content/xdocs/changes.xml
+++ b/src/documentation/content/xdocs/changes.xml
@@ -36,6 +36,7 @@
 
 		<!-- Don't forget to update status.xml too! -->
         <release version="3.0.2-FINAL" date="2007-??-??">
+            <action dev="POI-DEVELOPERS" type="fix">42794 - [PATCH] Fix for BOF records from things like Access</action>
             <action dev="POI-DEVELOPERS" type="fix">43648 - Fix for IntPtg and short vs int</action>
             <action dev="POI-DEVELOPERS" type="fix">43751 - [PATCH] - Fix for handling rotated text in HSSFSheet.autoSizeColumn</action>
             <action dev="POI-DEVELOPERS" type="add">Include an Excel text extractor, and put all existing text extractors under a common superclass</action>
diff --git a/src/documentation/content/xdocs/status.xml b/src/documentation/content/xdocs/status.xml
index f8c0eb6de8..1dc3b60bcd 100644
--- a/src/documentation/content/xdocs/status.xml
+++ b/src/documentation/content/xdocs/status.xml
@@ -33,6 +33,7 @@
 	<!-- Don't forget to update changes.xml too! -->
     <changes>
         <release version="3.0.2-FINAL" date="2007-??-??">
+            <action dev="POI-DEVELOPERS" type="fix">42794 - [PATCH] Fix for BOF records from things like Access</action>
             <action dev="POI-DEVELOPERS" type="fix">43648 - Fix for IntPtg and short vs int</action>
             <action dev="POI-DEVELOPERS" type="fix">43751 - [PATCH] - Fix for handling rotated text in HSSFSheet.autoSizeColumn</action>
             <action dev="POI-DEVELOPERS" type="add">Include an Excel text extractor, and put all existing text extractors under a common superclass</action>
diff --git a/src/java/org/apache/poi/hssf/record/BOFRecord.java b/src/java/org/apache/poi/hssf/record/BOFRecord.java
index df532d036e..dd5335a420 100644
--- a/src/java/org/apache/poi/hssf/record/BOFRecord.java
+++ b/src/java/org/apache/poi/hssf/record/BOFRecord.java
@@ -110,10 +110,21 @@ public class BOFRecord
     {
         field_1_version  = in.readShort();
         field_2_type     = in.readShort();
-        field_3_build    = in.readShort();
-        field_4_year     = in.readShort();
-        field_5_history  = in.readInt();
-        field_6_rversion = in.readInt();
+        
+        // Some external tools don't generate all of
+        //  the remaining fields
+        if (in.remaining() >= 2) {
+            field_3_build = in.readShort();
+        }
+        if (in.remaining() >= 2) {
+            field_4_year = in.readShort();
+        }
+        if (in.remaining() >= 4) {
+            field_5_history  = in.readInt();
+        }
+        if (in.remaining() >= 4) {
+            field_6_rversion = in.readInt();
+        }
     }
 
     /**
diff --git a/src/testcases/org/apache/poi/hssf/data/bug_42794.mdb b/src/testcases/org/apache/poi/hssf/data/bug_42794.mdb
new file mode 100644
index 0000000000000000000000000000000000000000..431eff734eb97e40ddd1fe3694c012d50244d62e
GIT binary patch
literal 143360
zcmeI53w&JFedo^|jii~;Xzo}x#(;tGBR~xjqmd<B5U55PKangO85;)`ELmebwxp3Y
zgRujN1#DW9_CwM<37<G5ZE2deNy|3PZW=btt0f7vkTjp&H2X<tTDEDMkj=hBH_`0x
zckVqicSa*ymW3AN|Czb>{_lDI&-b4DIFEa-vXmOg4~>ruO^(ENrt`5>w}O@W)A5?&
zgTXJ~@z0;Uce3l3)kog`<a-M5c%t<$PW{;{@n>Fo{>-7jefcX-ZvDUSdS=%Thkv~3
z&ENaTzkcG`>;7@_#&;cTZ+xKR{Xcr?>6K&4dp_OqblV+2J9XFpxZ%MKfAr#m9}Zmo
z_%Gi7y$^lyK+_|K@A<&bUVP{Jcij7zS6`d@)=yqYzf!1wcI(Gat_;8O&&|W%-dm^C
z#+EoV5hDbIfDjM@LO=)z0U;m+gn$qb0t-RlO3P=V!Y99rlz?T8*sSl<_|)g^6SgDE
zS_lXMAs_^VfDjM@LO=)z0U;m+gn$rueG&M1VgANveZPUZx!kYR0ra`AaI<FG1WZl`
zmZ=7{3{3=VCV+ug3k!`2;+4aUE2wH<7evfWbkjyV1autb4grSn>#+xPSgw7CR6uEm
z+6spzIQSW&z8IAg;&Q|hqC*HmdE(QS4&mrgN`t93FhnCFD-WR@hbSVe>R=;_RUINw
zuhKzlrP-AsnYMyVysntR1VYld>N*Faq^{qn8|E=6iwb*;szKFOI80kLbf_t&;jrRi
zh^QWwRpa=j)ghHvQI%3d*bb>-EIHV&aBb4?If;F`dTsAe6Kce@4XLea6!N_Yxe{_#
zhsa=mT<wDW7()8h4*2)0UUdSgbG=2KM4XdwkLvwCgd0)!A*QqnYn$D0%_5yZF_w_(
zSNChJClP8l!cw<!#KYKEs@stk<#BZwA$zfmLyJ`v_5<n&TqfZ&p!TX$NFl8rK`B6E
zpifzaf~xwDf+dRz%|m+4FbGxEw_~{sVOS<;oG?5hJh=J*^8}`o->-M+Rx~9yZRPM=
zH6zc4Z&3LWx>l#>!@34bqY7Xh#1d5ztZgnTjCCWHhzeodfCXg=>v}AXIz=5s5CTF#
z2nYcoAOwVf5D)@FU<nZLS)YtB8~vzvu-0*tn<rfj@#JrSRdc6$PNyb#U5<susZ65o
z!tkgz!K%EvYSE#x^2v(Nto3nSk9AaFR8Ez+9#!Hxl|(hP4;8tsx^i~}6}Cy0K?RN#
zR#q6Js2C2bqmWpgWo58V4Pcwn6+2HQE$FGO`3r?6?5=>GtkEYC@=<0@Bhdm^6&10W
zDM_WN$VvnuAOwVf5D)@FKnMr{As_^VfDm|n5%5_*YNSSu=$ZERI?ky7qrV*;n6&0{
z32MqKv3QM-;Y*cvr&)ka2D;C`wsJ54>h-!;x(*S0TMTN^hTwMXmpjzMqwYj6H6^&?
z3Z|zm?cgsC1}Hh4y^Uc;e>|dU()E9KNN3QJa39(N*xk)e>IvP+oI~e!Mt3L&irw9n
z9oeB`M|CR(516jmM3vSFc4SYYm4fXUZa1>0_jz15QoL2K4`Y295OXj>8X+Fi@Vy7k
z(?X$-!`c~bA#B+w&snXy6mhP?BCQwR(k!wN5CTF#2nYcoAOwVf5D)@FV2Kg%S<i);
zrDhD(_f{pHdVDFf0|AUG59m5P2lZc$`r;K>=3-_Y7oZ(D?}E4b&*Avmp;XH^qt)H&
zKN|>~5N<6W;{x33zlo~Cfh`ABL)29^N|>med~xhSDXIuY6dXdme-v#3V;D{z!hrHg
zv=DIgK?G~BTNG^sZ`DKVjc>=sJ2qjx_H#Sx@`rWppTpM=>6$%<<ePA_`|`%Mx)>LS
z-{;--i5!NvXV7YK?&0aHaV+(FAjtT`I`a&J#v9%8z@~wj|0@wjmz}aOc#&bI*P_i;
z2&3DZieXHnkuGo*BP{5~W35|hs<>%IxvFqGAJ%iHh)jqP0zyCt2mv7=1cZPP5CTF#
z2)uC#Sk?jaDzmy;&;QqI|2XoA$Z+J!$afmw*SM=O*!b1(>F`bA|JCrv4aXX;Y4}0t
zBcc7FrqEvn9}V^drvsl0<O4Sb{-yq@VmxnL8lt%n5CTF#2nd1K9f8<FXcS$DNX$eg
z3lZ1rAP#m8h<ObN=yv-Hjd$uoBVhOiiKMbmY(9!Dh$2XMLC`KpB=e#0nuJ-1!{d%$
zeogd~NQFQdfv&Q>dG1_bBz|i&Jo{w;H+l<~8ay<_C<U;Yf}YMkf?4h^ETRgUz}JF2
zU?2@K3qs)ZyAjRl{+(kxclPz|Id<&u&O7!T<CQWJEjjh>Jmy+jj^)5e=Ah%N!Lc|v
z(x1KC^%HTdGJEdcId))V_}KmfhaJB@<8XK+Gv-)@9jjw~nFEJit47!A#4*I6HL)F=
zv7Nmp2}K+0^uc2%jyY<N^xn0n&y65Z>3H8kC!Qv+gR5+9$<7;xzhr$RyC-wN^<7u8
zo|(OR$8l`Wu`#XoGVc3MI||;t^TbHjv0v`kQ~Ca0?X$+QIhH+maA)Skf#HD>Z5?#1
zv&Z&i`u7YR7~aE4I8=o1cZd-JLO=)z0U;m+gn$qb0zyCtEI0ww|M}{FRpavqqOq<i
z%!Te1s(Ka6*C~P1j$V-<1ynC1AqqI+kp>EQmm?CU;D^vi0pAXYL?~eFeZ;1KaUEzZ
zfPmJZ$nw)uPH;a32Ct>S;B^!jyq>}`2muP0LRd!OatKZp9fGdmQw`nXz<7M8-=k<E
ze7{7D5D)@FKnMr{As_^VfDjM@LO=*CFaf`{GCW&_tE}2TvmxNzHVdq)M0frKSZ{~1
zFQ<U|zkLM-RJ`rgM!3=lR~g}I3MdtJ%m{0Yu+|9cD4^PJuQ$S*jBt$+uBCt$0sA^5
z{E`ti7~y&fXlt-<Fha8tT8wZb1vEq0aU--EAz_3zHZd??$_Qa2G*Mu7EF&~hpz=N=
zM2uilpw2Z$h#En+VxV2azLYi;RvF<k3TWxDHyWYc2ptsA_F-=_!e%3MasU7iiZdJm
zf-_tJf*wRbrxkjT00cekfw|Cw6e#Ec2^21&9||koHWWK(gpd);><&td5D)@FKnMr{
zAs_^VfDjM@7b^k36^o!EBt{4b0U;m+gn$qb0zyCt2mv7=1cZPPSi%HY|3`197$G18
zgn$qb0zyCt2mv7=1cZPP5CTGA2@{a|{}Mj9k~JY91cZPP5CTF#2nYcoAOwVf5D)?`
z0jd8>Q4j(`KnMr{As_^VfDjM@LO=)z0U@yT30T$>0nD1F%?kCW_K4kJe<pHIBpUgA
z<69bE4L=^<6#hxW#~b!G)Hgg6x+fG4<$^CRvG4yb{i7^76aqp(2nYcou(Sy*<5y~?
zI1g;Y)V0s{-`#%0KXl#o_NRupWzLi85PoTu!>?BjsZ%Pe?o)Yn{LDs#qYs+C+Y_9!
z6&6kEHuz4f3HZ(;<d7OwE43fYSqp=Ywc_tnd)3;A>Qa|!OZBVjt@zzk_S>vtszDXH
ztYtcC&h2Wc<^TDHJE@x4@=>f$4M442nW}ffw)X;*v|J@0RQ$cU?01u^7-qD^47Pec
zT1(aayX&=L|Ls#d1`WHs=0;Yl{Hn`#zx3Rp_}ftSyRH-^rtO_VtagohcKJ(t8I{?>
zR;2(mkHvdfc4Pluoz(F&U0_=IyVZ%DGw|XVoF#Gx<L!uQ6gs!4nZFxKQQ;-nvqaU0
zgFk|TGV1CTS2KStrO5nejcfqrXi{ZRrt&z<$Ir}+Dh^$BX<;TYvF}+lWA-MX-Q?UN
zLjgNWWLtGaV_K^l)Xd+8l2#TXLS#72%<(i^1#<_YNJFO-)`yYRacoDB)ud8gwQj*l
z!8N5em;E}N1iCo8@>vrYz`+>9ZwQH;Qag~qVd#GR%)dbb{<$T<B#;Z7h41_9_jSoM
zTjZ<iS=g_J)luZ+kjvW1LHWAd$pH@>->Y^K_|^A*PbXlu%$qsQo$|@kAcGu?1Kbf^
zHgam>$?~Y3Jx>BCW^+hj4^Z(~WUywW<;0ymm%Cfs<`8L8^{6d6x`dkfi<cmf<+&l0
zlRc-7h%?H<=TnBL*?P&QaJY`2`7TOC#4Qod6}kL6;9dh*<T!;kBEYrEL}#{)_G|~L
z{EF)Axz|-PrLMC_Y%YSsD9t==c#t-#nZHDdDv8b>;a)^E>|U!To-EI<vsb+ES=LYe
z$hI~&saNdp+Rxgbu|IA<W<OvbvG1_E?HlcXA4x{O(Kyi9)%f-BXT!f8z9D>B__rF~
z-SA*TZNq;Ly%c&r^tn)X=*H0MP%!k~;OXFK@IdhI11|)g34Ah;2wWWq2VSjzxISAy
zSihtGTXlb4_lI>Ku3KAY*ZsWq9kr7(`G8AbHbOuMEPVog{rF!f*lOZT7?lT0)t9Vt
z+lJo%oA&$Ol6l3~{!i^+tbcCLh6dmD`}<Q*pLs#~e)l`?9bWaJkKOkhyZ`p<zn}cf
zeILB`V>LG(`N`(?)UO=>&bvN)$MEkQed2@v<-@=Dho^q|&V8$X{YdhmPygDVANbN;
z>B3`Ie(z_85?}b=+yC+VzxAIsO;jd(yrc=M0EOu?<&4!LD`%4RD96f#Cil2Q&i3=O
zAll0K@>ngVzMf>wVTBdzCl&e{E04a$YLWFd$$Io<C8$DQW98A;SS_-?CRvZZcmS@{
z*I0S<HCBtPuSwRUFROADe2tYyUt_h%`kG`t`r^5@QeR``(brflvc4u+kG^;;o?>;w
zSmDYFQ)$JDpFRqutDdo1%q&1TldORWD;x#es~oN(#%eK@L$a)6c<%%rt3}oXM_R4G
z;yJ9i7*w~aQ;Vq_4j7AAg>sJ7qHBV3f_BJ7PnJjL|3k7!H5ys`+&amYl|`e8=PU*j
z)X(eC{7`(v<5`S$)k>>rc$_Fb&X|+V1awcj3STV!_)@wzE`6n&5j}jBrO0DdW@TAd
zgnIWSdufeEk{6xdgZ9n#%`cSyh9W!Ah%8132mv7=1QwY9yCl1D`JPg%)jqWEj-ow(
z1Y6T>aJ$<(Ro!(jnw3w&?}f*5;W1E;m8#$Ekr>BbDHmb275N&49eXSy+SiDdyV1?l
zRI%TW?vjYwh45qW(}(Q@Vs`qRhBS{nSh3&Zesl|OL$A~hxUNLkMHXqW18H3C*BtZc
zbSbx|eoo4(%KUHF{^>IArj_$_=Q&+C%msVo*gcg;a}PB-gmBy+RlBhaVV@2CO}gu9
z0=nSXz&59jAxAxs*g5FLwrWO>7z_J!qKbVI$028s8hhAQ!*7Ni2s({ZT5}Sf)H!ZK
z&oXsjUm^Pl?uE~*(DyouUlxqsf;i~27u#{Tl1c;`_MxlC=>bi{zq3D}{qQ)A>%18E
z^`JZs9S$KLQX=ay=tKS3>F(sI0na)5v#(eOmF4rXwh=_tt(L;`N;TA}ud2}Vkvr(`
zrPdGJ`Ug8N>Hh2`AN-$3zw)7<((b974jeEY)7&?m)0CAu{M+CClY#YL?Ekah8;$?<
z%|DExm1ssR)NoX_YX3_8=+(mU@cO5>ec;YrzZrdTYh?Bi2{%OLCJYPb`O52pO!?}O
zDc>q(1p_!_m6^E<zAa!@D56FsL{$?+&7uH9IRu%+NT`7lnMEKJ(0&+Z|5BV*RukP|
zRkIvcW5``))CpBBYIrM)+KGn|UV;cQcECi)WLKK>FNFXtEeknQmRuKjJ(lZ&To-sY
zc@56Ei~PFqC<sjHJIyZKfp(~vGGh`>K{j`e>u^sxg1ZFo<s+~+_t03yT`Y$?s&~B5
zKKP$3-dRGbTi*{y5k~~~THeo&;wgZBcIm@{c5fIQ6;zp>S~F$Ky~H-*0_wvxG>&m8
zEvg+`y4Jw80~dG5HEi-(hZuOpyAIna=*s&r@43A1@t&V9rtFLsxr9lgDo(?TIRE;R
z8HdFblG}@*c|?##5=s3ya0H6;5y;#?hj3dxg=lG<h%gQfHS!1^>_(A&a|47`2)7$3
zwO(gsH6$~_U|ipfsbmZ{`8*K$tRM#xsQ-%P0HSq(Xbp9ty&A*mVpw+Z4z2IfR9@l1
zMd2=?jR~@h;T;o%9b2Pl9=H8;(4iC^oO|T6kNI)KGm809DpKkZ+;qXEP{553|CDF_
zc%jf(D7@%Hn#&pM#Md5sC%0N@EpWyACAWSYPM8Q#_9NzTWR%apjw4;f2mv7=1cZPP
z5CTF#2nYcoAOwVf5V)uaSk{%g{?AO;sL$Ic?AGG`MFm1KD+Gjq5D)@FKnMr{As_^V
zfDjM@LZAu(bcw%C%!J#uK0kH**3TRH7E(3i^9S%sfS5Njfzwl3`xS};21Q(=1q{Jh
zsRazuh-nY95&}X%2nYcoAOwVf5D)@FKnMtdrA@%HZqu`nFqf9%yJ!n$e0l;*%nPgT
zeAm@F)_?lp8YgzntxJP<6~6W#*5GvnGbL~NJdnrFoOJ?BCk|N)0U;m+gn$qb0zyCt
z2mv7=1cZPP5CXFj_^AB^rX#F*P3!+JzOd}?KK}IOKd4mVR=jF3T?o3<7&xxzo5tsH
zT!Uc;Xd2K18>Zb?`Z?f&ulI|2;}G}-RORdtJce92Z6)W3pe$zT6Sy7Ijbt&+NDT9b
zIkS+R!qK0*S&0@zL&;1hGF#}W8&Cc2=bm}iqQ5V?o}AfVCW^C+%ocVw`zB0}!ZM3l
zUy4(U%;wi??|dLk6=$%?c^%I~4r6jHo-aIy$e~(LlW(*2`f+BZ;WRS*(A4>xDAMO_
zExb0*Cw+6?%lU3f@x`@eiSuQ!;z{W{iq*>~%FoT&Ab|2m&4;r1IVYMq2Y-&`mdo5j
zh|h&2Q2y0C9=r21y#8<=MswCw3vI4@X6yA+{-Hgu%iLt9Z_?(rS}2pQ*?Rr#!(3!Z
zuf=)F0Z;mzAhMb<>Hc){kG{(LpVv<h=29EMS;HAW=Xw1vvY$`>&Fo&@+1g59cl2sn
zU5?@Gy%7E;{nCfks;4uX?Fvu&tdCSPhJV-I|M-JS{+!(HLJTJT)a9=ckreAj6%m^B
zs~!JtObnNGubLIX=Cm8a^Ut~XS4PBpy|L@TM0T8*jx)e+LmA|!RwvG*BwmjEW41iZ
z6F0x-j~YDrab{do4vb!N-QxN0oaL`FE}nqq`Oi83%^ZZzOn{aC&$f@LRyBXVkw4q{
zw|n_t==0uez47t9b7tLjG%*~g<6_kRy}F!DzI+~>`CMvV!jN*t`^>_WGj~$Q8HdZ1
zuWHws*~;0BNM^B^@9}57jK+J7qRcv*d2ZnKgY6K!fAZSIx)iUcJg2zNH(dAv4D-#I
zC8e$cC~q*c0MA*)GQ=`a&77_L@5cPE{c0y#IeIXW>;1Ufou64TkLw?=eY^(Jl&A05
zozFUo`@A_1c<AR6mQ}h{?$hQym|vGypnK-23C@3upHMQzZZ7HboPF870q~qP3(s%U
z&d6(kvFGy}pP_io;Q7z%0j~kfH~pCNpL%e%Uf#p`1;SNW_*HifS-S$uYAjb`xeCkG
zSYlY#U|EZ09hUW2-h|~EEZ1VW4$CiL*?{GGEH_|j#?pf2Ml5_r<Q((~ENxgeVxc}_
z=1yQS+ZoMsKFd)T3lfa9GkU*4?=U%^^@X-GGOgD`I}`PoPy5M^)Gcct=-!lQOQlk+
z%^jPzCYv{QZ|!VOb|$);H*V}`>*(l-w<b5YzHMz|XxGqK`j)jPM|1gfE|2}~>CuUv
z?07zXGS7J8u-ccN%w@9UJ=w8|p?qd|G~JfCWo=t$TOyH&$LXP`Z`&<v6Ppvsp4Qe)
zJ?-8FuBqMKx2$cCw<o$g+mhW$XMz2;o?F&#+Po>*+1iolOmOK;!MZVY*Y<2KAKP*=
zpB^7cPsVnPAI|<tW9XXgNQ|cSmIvB4w<S^?8{^GATRY>;t*t#<n>TOS*w!46C*wVx
zt*!BRd-82J@9W-jb1_uBF*H|}^P^sh-Fo5aDbD}S*24G(pYJ_EHL;>@dyRV7r$%1(
z^CPpXl}o&K4=?Fmz7=)86}NoK7pnJ-S}Ou|O+jDN)Td9^Y_C;e-_E8t`$DL^P3H0=
z(d=mYR^Mf;2BR&8UmQvA#trC*dehyiW`i2s_5cVa;(xrcE#9mGet(Y&)%YS-Q#`)0
z<8A9yq9wjAzOLscHMoDru2ld2fxuwyl={?ITVhZR>`nFeZb`I6_x5(LQ-N3#Uy;d<
z;fQC)*QtXFU+Sot@}-jg{#$)JQ=AIDT_qn^p~t4I+GP8d74g<9lI^w0-IG~$@8r-}
zY-<L(<pxn|#<IR~e_dbZ&}25JvWN4rfg?kctI{KbQ#U_kwROY~wxoJ{*ZrEU!ZFp>
z(h=()K8nh1KGvHVo*e3(Jf*fP3)%m2aw^rI>b@0yxq(NMsD-~Jg~x=!w|;o4{@qiS
zfAFD?)(y4>;?0{MIg?Dr<6Rx8#Fpg7FYRh=_{y$TEdvh^3=WNC1A}9^E-TU6lJ}1e
zf9?9@p?v1VRQk?XYA8SCZ~j;=mS|}m(Y1V@ee*Mcc|3oyUhy~X^UwTJEf3|r+m*X4
z{2reh-TFTocy!C+tFYoU{id0VIit7Y{QosLH$3zI<FN?eebL4QDNgfm>1qD?fXJ>Z
zzX~y8M&@(o|JT%>eQX5vck{s;BPB}T0Pq4S%04`t)*xLzdA-wJ&i)O675eV%e0ymU
zA5gdrEh4Azq=~?e;m!Lj&c>T;g@6zc0zyCt2mv7=1cZPP5CTF#2)uy_Sk@Kn1<}T@
z=l?(Pf_wqM5Rt7A5CTF#2nYcoAOwVf5D)@FKnPs)1f~{*0DHwR*jz{?mi5<G@4jR&
zm9W(J7jQts1!qelxL^X`_<~=*d&!83V<<cV?eqE-5sXf{XbnfIa5#?`5^%=t2n&aR
z(D0T7p&>N91iZ48jcPRbnh0MNn8uS;+Vgb9n=p6p#$uk|z7Cm&_IwpYj1Uk4LO=)z
zf!B(F&l=D#u|0*G5Qpg-!+Q|#OU3d3b(p!WPFD`}jUKN;T6`UH4o1Hh;P?yT9UCW+
zzx%ddH88<i{kA3@%xw_wWzxaY_ra3Gw;H47O#=EXIW($=9fumNUdiF*PS=4iA$UV{
zeskLLs~vbBC#-t$?&=WU%DE5K;1zg_cUT>TbO_brZLrJ0Pr2<b_#T6gXz{JTG+rRc
z<DG(#C#FaE>Z!uy4KEi8jfKKD3x&=Kxd!z3E@|gC8(-#|49dLufEO)h3*$!^zI)0r
z4;91Ut&FpTsYMt*gp|T~53_C+U`&2LkGL)Z6!Z?CQD})5plBG5<n?i?aC2y`ugcH7
z9ceg~`!YN$ebp+TTfRK+YG^%VB?N?k5D)@FKnMr{As_^VfDjM@^B`bZ+x7eZ3|gnh
z|F_ydj(j3AT(q4>q!L632mv7=1cZPP5CTF#2nYcoAOtRA0$mG{_t-*2!YO~nI0XDw
zEW)Xc#Rvf*AOwVf5D)@FKnMr{As_^VfDjM@OP7Gu|CjEum7ECyAs_^VfDjM@LO=)z
z0U;m+gn$q*1f>3NNXS+Q2mv7=1cZPP5CTF#2nYcoAOwWK(k0-x{<87XS+eKz2cogA
zDP{Tmv-l9}*E@mJQ+hJLHHrde?ptF~z!Z*ad=zkC*VIs`h2W=vN&eQ<Qosy`Yw9Qj
zAk<U99QkVk6haVW!h9u02nYcoAOwVf5D)@FKnMr{A@D{bU|F}RWv;@Ox@#d-*Ym{B
zh>#P<8#BXk5iCsF+@5c8x3$>a?|)RO<zKe8JQP(M5dOLIG0zW;u{K@^qDsxr6uj(A
zy06u+PzCp?e?_E&`0~I5Dz1MCtT(DHYO88?ZQ^B8k84|Ub;3F8jjBVvO|3%=E%3Ka
zbz<pJHz9@H>UPZ4zYXi8db?_cO%L|_@atE*T$?_8A)!b68*pv*sc!fm#CHz3?Qo?n
zkXlp<UsC8r{QL0DgdDz{z;7zV;BN?f8GKJ6hCF2PrGywlPT)7HZHMsXgbeJ*;md29
zEW=?V{BOmt2ihl*i)Pi1y-xg^p?4fhHzfKYCx=5)`;6lwN?YNB%O=gC)8#-GhQlOM
zzZZJ(s}9sL1I^OVW)Li>H{Eks4nxxz=p4q6_G$R&hySgb(iZ4JihXJTY4_mQ0@q&X
zwytVCNyNZ7a`?q`SRNt9U;6$;4B<1_r*4z@zJ%92Q?;D5A&yc>@Je$lCy5z+_9J(@
zaBTb4e&lQr+MdF~PezV`ZUQy~YOi+h1$~C!>*kud-+~+^Q5N#xH-=o8W9`k!>f(_$
zCB7ZucwD{un^N5jvW{GGHE!IR65l=}C(idJsvqedt=|~d<IwnAWne%Z(MKeWZ-WdX
z?@7=x+cumFaind|hH7QV2@wOo7Wkg~F~6#U6!XjL!%R{<FPhb6tod_DnRB88x+Oq3
z30Ax|@R+q?JGXOUc2WbLk}(KABcM5mqn>l+1X5{12^d9*p8cF)2|0n%btkx_bcq?N
zAjI><(fMrW2U|9MxB>Znma4-xfNjL*T>Dx2c<ul6>6-1eRX>aBy+@o&w!DSemA5?P
zd~VgU0!aH@F^K!pCvXxaV@j8fpw*xX>bf^^$EWK2$wP<Ixm=BZUw88V{C!l_`uAtX
zTRXn=c2{gm)Sr&38h_u=p+EVzQGALA{yILjE~*0l-ILj)=|lO}(4|qtlgp1}N7L^+
z7*+NDlcTwOI+uU)8iq_I`#<_HV@sy`Q{6d4g7W~=J|q38uLh;F$>%SY(K<e!6xdO4
zu459$=~XD0bBdohbk2nHc^(qKLO=)z0U@xs1o#ENkJ?Yzma4I=S0kqXKcxP}{<i&$
z{gnNfebPQ?Z?{|RD{S}`BLswi5D)@FKnMr{As_^VfDjM@LSX3<U@bX>AKF5v^XYLL
zB03WsTaJD%Opj;A)A%TT$r}B-uuf<0J(6GMwddF13)8vuiS&4xJqBCTK6A1xa*V}>
z?cvOc^2j*|AJ*CXCd<;KLQwF?WIA0IH`;J%e}A?tZZ^^urnBQ|*EVSV)b8Dv?1eQO
za9`wB%u*rnjW75))?b@y4f=FoXD+*R6}B;KhtNUCF7Oz-;@BOU((V~_l5_6_Y}4v~
z9ph42k&wL3V2-9KFJ}MZ-U<!x@fR9G!%M&`OWCMK<0@FPb9~yu{S}Kbyc=`(ZY+FK
bqxrhC(H-e+t}GiNAOwVf5D)^d1%dwup`rm<

literal 0
HcmV?d00001

diff --git a/src/testcases/org/apache/poi/hssf/data/bug_42794.xls b/src/testcases/org/apache/poi/hssf/data/bug_42794.xls
new file mode 100644
index 0000000000000000000000000000000000000000..2827d5ac6d2a72943cd8813d9084aa6789241d4c
GIT binary patch
literal 10240
zcmeHM-D@0G6hE_>O?KNP<~y-Aaon`gd~Dss)LKHCG+G2D8v21kQer+P&C-&E-L>k2
zx_-SW3cd<b^u?D}UlgHW`((j@B1plf)W1N%f)8!i-|tMOI~h0WuBMW<bC+|^o%1_)
z?!9O3$GK;J`muiH%vU?Eh%X(G3b~PwN~K3`c*;^IvJKCa-N@(jgjhx~NOzD04oeEZ
ztXOhLmcY~U8b4*psYSv!zxkT^QREGYU%tEg?fLJGT41BhkX}IRX*nsg+K-ctu#CJx
zLWbAerB;oswqL6LMtc9I{f*vNF7{vA=i0sQeZe|q>Q$p-H6WJn1IHF1^`HjOcF;Oi
z1MwEN!2h}aC#uf{>mTc6vy*pvmHTeD?ydHLVEwz6y1O{f>=38|WP-vV`g~QO2q+4o
zpU3^(-T!%X`uG1v$eKXSpcYUos10;KXb0#4&`wZ0r~~vM$O1hCih(*oyFj}^U7&|S
zoO40Cy%vZ@@RcjWayj}#)fLQyFU^ZOAAW=Vr)0l8E^^Sy%w&_9g`_=ijVI^QsZ44i
zYdycPkesvkTgNhX+A5O9^ku_F2-kv7P_D&P;6#JNhP+Ypb3jFgAzL(_l~2c>>v;mt
zyhywl8dZ%CG>>Usi`<ZJw9c<8>6ae41pWM*A8{UAvRq?~%Mlcg$PC=sjFxOTm`Pa>
zN85n*c{W>;cBrg(SCsYc^7?IJ`Eis+N&Plq?^3io%r-1}EZ*2>vCh8E&cx7E+%0yP
z?br&ih+V<Z<Y9aMxL2VA`vg|FtFR~Gt&gV;vlF`r*1K23_ZC&#i!YfFnP%fXmiA;i
z?)0VbxQfQ&F<{g2(dl@H*(crHE2iTGwJA@T@-_Oy(kGZ?-iFWtUt5l((A>yKp}|4v
zl}9}dy0ur^cdKf%g?cNxIf`7(TbO!ox?oX`kLt+(EKSbItMBpYuP5hQi|OCZas$h|
zkQ_UQ@#WMGf4PV&?_NjVw~oB0sGLV;F+1}(EhZ1+aP{}+<B%zJxr#h_eXde@L(Wv*
zn5$CWl#42F&P9|<PL#Lg!ugkUzkMG^c^`h)^ayuGo0pKu13gHA1p*7)s}`Uuzb5hU
z;lZ&L{r^YMuWnEes29`+ii7rma0Js~fKw+j0D25G2-*vR&no*sIOl^q#4WB5FwvGy
zoUqOvrB3fC+e5r%I*E5urVkO6tkKIsHS&4fO?gqM*LhusBk<Z6fBZZ?J25h?@*b7<
zYW}uLa-L&yHx6Dn=JFPhi)Is?W2}y@gUz@E<9wf(IbmB}6Ypr{cQrpFG4^4?K$C_%
zCCf+@h^&mbe0lj8baDEIoT#(YaW1<^h(S~?LH!2>rB;<W%W|!Z`hGnwTfdA0@XL6u
z_si&)_+|8A{4#nCei_38ei=WNS9V<k2TmwJ#Cb9>K922;&^F}^6Z}3xjMi&KeFq~a
zCZojoIa_s$Vm*$TSVXZ3#W-eS<izR}GZo|5iIEd)QY@?(Z9t5iSes&%iqRIt$cgP#
ztV%K3gcvz7OR<Pzv<)$GVll;{iqS^I$cgPzjH7SW5X)r*OX@KAO9+<y^EaqlAp<3o
zg^TzzhmKlLrtO(5uIEQ*W>4A26N!PuzW&64P=kEUfypSd)zpsVK?@@BLol&3LAp0B
z5X66Eqi7NKvoc~!LHs9(|7b92HS83`e~QIDxY6Ej`JEd7apQzh>`Lm)=&5wU+MKx1
zdx#-_uBW{zSv!+GYtTb7O!%BPoq;y}dqzyxxkd-5I>e5!sv(8w(xm<+#J`2$cg>|v
z^*d|;{zYoZMxK2o1+Oc+n6_;|3#pgWWXaRSGRaqgW$e@T61bf_v6uz8lyoZEOHRP*
WX*e)!;C1jZ@E`>i2rRI97Wf;@QXk3y

literal 0
HcmV?d00001

diff --git a/src/testcases/org/apache/poi/hssf/record/TestBOFRecord.java b/src/testcases/org/apache/poi/hssf/record/TestBOFRecord.java
new file mode 100644
index 0000000000..63f5cc317e
--- /dev/null
+++ b/src/testcases/org/apache/poi/hssf/record/TestBOFRecord.java
@@ -0,0 +1,50 @@
+
+/* ====================================================================
+   Licensed to the Apache Software Foundation (ASF) under one or more
+   contributor license agreements.  See the NOTICE file distributed with
+   this work for additional information regarding copyright ownership.
+   The ASF licenses this file to You under the Apache License, Version 2.0
+   (the "License"); you may not use this file except in compliance with
+   the License.  You may obtain a copy of the License at
+
+       http://www.apache.org/licenses/LICENSE-2.0
+
+   Unless required by applicable law or agreed to in writing, software
+   distributed under the License is distributed on an "AS IS" BASIS,
+   WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+   See the License for the specific language governing permissions and
+   limitations under the License.
+==================================================================== */
+        
+
+package org.apache.poi.hssf.record;
+
+import java.io.File;
+import java.io.FileInputStream;
+
+import org.apache.poi.hssf.usermodel.HSSFWorkbook;
+import org.apache.poi.poifs.filesystem.POIFSFileSystem;
+
+import junit.framework.TestCase;
+
+public class TestBOFRecord extends TestCase
+{
+    private String _test_file_path;
+    private static final String _test_file_path_property = "HSSF.testdata.path";
+
+    public TestBOFRecord()
+    {
+        super();
+        _test_file_path = System.getProperty( _test_file_path_property ) +
+        	File.separator + "bug_42794.xls";
+    }
+
+    public void testBOFRecord() throws Exception {
+    	POIFSFileSystem fs = new POIFSFileSystem(
+    			new FileInputStream(_test_file_path)
+    	);
+    	
+    	// This used to throw an error before
+    	HSSFWorkbook hssf = new HSSFWorkbook(fs);
+    }
+}
-- 
2.39.5