[ 
https://issues.apache.org/jira/browse/DIRMINA-1103?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16810758#comment-16810758
 ] 

Jörg Michelberger commented on DIRMINA-1103:
--------------------------------------------

You are right, I downloaded the src of ZLib and inspected the free() method:

{{  public void free(){}}
{{    next_in=null;}}
{{    next_out=null;}}
{{    msg=null;}}
{{  }}}

It does the job!

I expected this is destroying the internal compression dictionary, but it does 
what we need and free that sticky buffers and additionally a msg, which could 
be set in error case.

 

> Avoid sticky byte array buffers in Zlib CompressionFilter
> ---------------------------------------------------------
>
>                 Key: DIRMINA-1103
>                 URL: https://issues.apache.org/jira/browse/DIRMINA-1103
>             Project: MINA
>          Issue Type: Improvement
>    Affects Versions: 2.0.21
>            Reporter: Jörg Michelberger
>            Priority: Major
>
> For each Zlib instance there is a zStream instance which is equipped with new 
> buffers for each in and out of inflate / deflate. This buffers are not 
> released after each inflate / deflate. So the last compressed and 
> uncompressed byte[] and its content for rx and tx is not garbage collectable 
> on each session.
> It would be great if there was a reset(zStream) method on Zlib which sets the 
> in and out to static empty byte[] to release these buffers after usage quite 
> before returning from Zlib.deflate and Zlib.inflate but in the synchronized 
> block. Something like:
> {{private void resetZStream (zStream) {}}
> {{  zStream.next_in = EMPTY_BYTE_ARRAY;}}
>  {{  zStream.next_in_index = 0;}}
>  {{  zStream.avail_in = 0;}}
>  {{  zStream.next_out = EMPTY_BYTE_ARRAY;}}
>  {{  zStream.next_out_index = 0;}}
>  {{  zStream.avail_out = 0;}}
> {{}}}
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to