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

Gordon Sim commented on QPID-6271:
----------------------------------

Suggested fix (I'm assuming detach() may want to hold onto the receiver):

{noformat}
diff --git 
a/qpid/java/amqp-1-0-client/src/main/java/org/apache/qpid/amqp_1_0/client/Receiver.java
 
b/qpid/java/amqp-1-0-client/src/main/java/org/apache/qpid/amqp_1_0/client/Receiver.java
index 826a757..1d9ec0f 100644
--- 
a/qpid/java/amqp-1-0-client/src/main/java/org/apache/qpid/amqp_1_0/client/Receiver.java
+++ 
b/qpid/java/amqp-1-0-client/src/main/java/org/apache/qpid/amqp_1_0/client/Receiver.java
@@ -530,6 +530,7 @@ public class Receiver implements DeliveryStateHandler
         {
             release(msg);
         }
+        _session.removeReceiver(this);
 
     }
 
{noformat}

> [AMQP 1.0] receivers never removed from sessions list
> -----------------------------------------------------
>
>                 Key: QPID-6271
>                 URL: https://issues.apache.org/jira/browse/QPID-6271
>             Project: Qpid
>          Issue Type: Bug
>          Components: Java Client
>    Affects Versions: 0.30
>            Reporter: Gordon Sim
>
> The amqp-1.0-client Session records every created Receiver in its private 
> _receivers list. There is a package visible removeReceiver() method, but I 
> can't find any use of that.
> So even when a receiver is closed, it remains in the sessions list.  I came 
> across this while investigating an apparent leak with a jms QueueBrowser, but 
> just from inspection it looks slightly wider in applicability.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

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

Reply via email to