Thanks for reviewing, Max.
On 18 Dec 2014, at 06:52, Wang Weijun <weijun.w...@oracle.com> wrote: > >> On Dec 18, 2014, at 07:58, Vincent Ryan <vincent.x.r...@oracle.com> wrote: >> >> FYI I’ve updated the webrev to include the changes below: >> http://cr.openjdk.java.net/~vinnie/8044445/webrev.05/ > > PKCS12KeyStore.PKCS12_HEADER_PATTERNS. > > Is there a possibility for this? > > 30 82 -- -- 02 01 03 30 81 -- 06 09 2A 86 48 86 F7 0D 01 07 01 A0 -- 04 > > That is to say, the length of ContentInfo is only slight smaller than 128. My > understanding this is more likely than existing pattern #5 and #6. In theory it may be possible but the smallest non-empty content that I could generate was about 200 bytes. Do you have an example? > > > KeyStore.getInstance(file,pass,param,hasP): > > It seems if one engineProbe() returns true but loading fails you will try the > next storetype. Right? If so, dataStream.reset() should be called. No. If loading fails then an exception is thrown - no further storetypes are checked. > > Thanks > Max >
