RFR: 8277087: ZipException: zip END header not found at ZipFile#Source.findEND [v2]
Sergey Bylokhov
serb at openjdk.java.net
Wed Nov 17 10:37:37 UTC 2021
On Wed, 17 Nov 2021 09:59:43 GMT, Alan Bateman <alanb at openjdk.org> wrote:
>> ZipEntry::setComment indicates that the comment will be truncated if needed and ZipOutputStream takes care of this.
>>
>> Perhaps writeEND() should also be updated to something like:
>> `writeBytes(comment, 0, Math.min(comment.length, 0xffff))`
>>
>> Which is similar to what happens in writeCEN
>>
>> Yes it would be nice to clarify that a null is accepted by setComment. When null is specified, the comment length is written as 0
>
>> Yes it would be nice to clarify that a null is accepted by setComment. When null is specified, the comment length is written as 0
>
> @mrserb Are you taking the spec clarification or should we just created another issue in JBS to track it?
I would like to update the spec for the null value for this method, and probably others in a separate CR, since this fix could be backported to the early releases. Will create such CR after agreement.
-------------
PR: https://git.openjdk.java.net/jdk/pull/6380
More information about the core-libs-dev
mailing list