[JCRVLT-92] Package rewrap broken for packages smaller than 1mb since introduction of memory archive

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.1.16
• Fix Version/s: 3.1.24
• Component/s: Packaging
• Assignee: Tobias Bocanegra
• Reporter: Ankush Malhotra
• Created: 19/May/15 17:03
• Updated: 28/Sep/15 16:22
• Resolved: 19/May/15 18:02

Description

Method getPackage() in the class JcrPackageImpl creates an in memory archive instead of a temporary zip.file, but the method PackageManagerImpl.rewrap(JcrPackage arg0, ProgressTrackerListener arg1) still expects the JcrPackage arg0 to be a file, and not a MemoryArchive.

The call src.getFile() returns null and an NullPointer Exception is thrown in the ZipFile method.

/* 175 */ ZipFile zip = new ZipFile(src.getFile(), 1);
Here is the relevant stack trace

*ERROR* [Default Executor-thread-111] de.fi.myif.modul.impl.ModulServiceImpl null 
java.lang.NullPointerException: null 
at java.util.zip.ZipFile.<init>(ZipFile.java:106) 
at org.apache.jackrabbit.vault.packaging.impl.PackageManagerImpl.rewrap(PackageManagerImpl.java:175) 
at org.apache.jackrabbit.vault.packaging.impl.PackageManagerImpl.rewrap(PackageManagerImpl.java:143) 
at org.apache.jackrabbit.vault.packaging.impl.JcrPackageManagerImpl.rewrap(JcrPackageManagerImpl.java:539) 
at com.day.jcr.vault.packaging.impl.JrVltJcrPackageManagerAdapter.rewrap(JrVltJcrPackageManagerAdapter.java:144)