RFR JDK-8211728,JarFile::versionedStream() does not filter META-INF resources in versioned stream

Lance Andersen lance.andersen at oracle.com
Fri Oct 5 22:41:15 UTC 2018


Hi Sherman,

The change looks OK

Best
Lance
> On Oct 5, 2018, at 12:48 PM, Xueming Shen <xueming.shen at oracle.com> wrote:
> 
> Hi
> 
> Please help review the change for JDK-8211728
> 
> issue: https://bugs.openjdk.java.net/browse/JDK-8211728
> webrev: http://cr.openjdk.java.net/~sherman/8211728/webrev
> 
> The "intention" of JEP 238 is that the resources under META-INF can't/shouldn't
> be versioned. But it appears this is not documented & specified clearly/explicitly.
> And the JarFile implementation has loophole dealing with this case. The spec
> clarification is being traced/addressed separately via JDK-8211758. The change
> proposed here is to fix the JarFile implementation.
> 
> Note: the direct trigger of the bug is that versionedStream() does not expect the
> possibility VersinedEntry() might return null when the entry is actually under
> META-INF/versions/{n}/META-INF, in which it has a "basename" of META-INF/xxx
> but doesn't have a real entry under /META-INF (in which the getVersionEntry(...)
> is invoked with (name="META-INF/xxx", je=null).
> 
> Thanks,
> Sherman

 <http://oracle.com/us/design/oracle-email-sig-198324.gif>
 <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
 <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering 
1 Network Drive 
Burlington, MA 01803
Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>





More information about the core-libs-dev mailing list