https://issues.apache.org/bugzilla/show_bug.cgi?id=52665

--- Comment #3 from Yegor Kozlov <ye...@dinom.ru> 2012-02-15 08:16:08 UTC ---
What do you expect from  "getEntries()" and "getInputStream()": return null or
throw a more sensible exception? 

I would say that instead of the NPE they should throw
IllegalStateException("zip file closed"), the way java.util.ZipFile does. Does
it seem the right fix for you? 

Yegor

(In reply to comment #2)
> If the method "close" is called, zipArchive will be null. After that, if the
> methods "getEntries()" and "getInputStream()" are called, NPE will happen at
> the code line 44 and 48.  
> 
> Please check the bug issue 51949 at
> https://issues.apache.org/bugzilla/show_bug.cgi?id=51949  for the method
> "close": 
> 
> 
>     public void close() throws IOException {
>         zipArchive.close();
>         zipArchive = null;
>     }
> 
> 
> 
> Thanks.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

Reply via email to