Java sample uses Inflater.finished() which doesn't work as expected

[GoogleCodeExporter]

What steps will reproduce the problem?
1.
2.
3.

What is the expected output? What do you see instead?

Please use labels and text to provide additional information.

http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=4795299

Workaround: use Inflater.getRemaining() instead.

Original issue reported on code.google.com by acc%goog...@gtempaccount.com on 21 Dec 2007 at 6:35

[GoogleCodeExporter]

Original comment by acc%goog...@gtempaccount.com on 21 Dec 2007 at 6:35

[GoogleCodeExporter]

Original comment by acc%goog...@gtempaccount.com on 21 Dec 2007 at 6:35

• Changed state: New

[GoogleCodeExporter]

Original comment by acc%goog...@gtempaccount.com on 21 Jun 2008 at 6:10

• Changed state: Accepted

[GoogleCodeExporter]

Actually the problem here is that Inflater does not know it's finished because 
the
data is missing a single "dummy" byte that is required when Inflater is 
constructed
with "new Inflater(true)". This is in the JavaDoc for the Inflater constructor [
http://java.sun.com/j2se/1.4.2/docs/api/java/util/zip/Inflater.html#Inflater(boo
lean) ].

The fix simply involves copying the byte array and tacking a single null byte 
on to
the end.

I've attached a fix that does just this.

Original comment by ry...@ontko.com on 28 Aug 2008 at 12:17

Attachments:

• PROPER-INFLATER-USAGE-PATCH

[GoogleCodeExporter]

I face the same issue with inflater code. My byte size is 40000 and is required 
to store dbcs chars. I did add the dummy byte at the end, however, the fix is 
sporadic. Sometimes it returns true and sometimes not!
So could you please let me know why it works with less byte array size when it 
is around 200 - 1000 and not with more. 

Original comment by ashun...@gmail.com on 24 Jun 2015 at 6:25