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

Hudson commented on AXIOM-354:
------------------------------

Integrated in ws-axiom-trunk #413 (See 
[https://builds.apache.org/job/ws-axiom-trunk/413/])
    AXIOM-354: Refactored UIDGenerator to avoid a class loader leak.

veithen : 
Files : 
* 
/webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/util/UIDGenerator.java
* 
/webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/util/UIDGeneratorImpl.java


> Potential class loader leak caused by the thread local in 
> org.apache.axiom.util.UIDGenerator
> --------------------------------------------------------------------------------------------
>
>                 Key: AXIOM-354
>                 URL: https://issues.apache.org/jira/browse/AXIOM-354
>             Project: Axiom
>          Issue Type: Bug
>          Components: API
>    Affects Versions: 1.2.10
>            Reporter: Andreas Veithen
>            Assignee: Andreas Veithen
>             Fix For: 1.2.12
>
>
> UIDGenerator uses a ThreadLocal with a value class that belongs to Axiom 
> (namely UIDGeneratorImpl). Since the ThreadLocal is never cleaned up 
> explicitly (doing that would break the UIDGenerator), this may cause a class 
> loader leak (i.e. in a Web application, it may prevent the class loader from 
> being garbage collected when the application is stopped). Since 
> UIDGeneratorImpl basically stores 3 values of type long, it should be easy to 
> refactor the code so that the value class ThreadLocal is not loaded from the 
> application class loader (e.g. by using long[] as value class). However, I 
> first need to confirm that there is really a class loader leak here.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to