Apache
/
activemq-artemis
mirror of https://github.com/apache/activemq-artemis.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

adding missing check for closed on proton receiver

This commit is contained in:
Andy Taylor 2016-12-15 13:37:51 +00:00 committed by Martyn Taylor
parent e8eaf15300
commit a9b32bdf0b
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
artemis-protocols/artemis-amqp-protocol/src/main/java/org/apache/activemq/artemis/protocol/amqp/proton/ProtonServerSenderContext.java
View File

@ -427,6 +427,7 @@ public class ProtonServerSenderContext extends ProtonInitializable implements Pr
@Override
public void close(boolean remoteLinkClose) throws ActiveMQAMQPException {
try {
closed = true;
sessionSPI.closeSender(brokerConsumer);
// if this is a link close rather than a connection close or detach, we need to delete
// any durable resources for say pub subs
@ -466,6 +467,9 @@ public class ProtonServerSenderContext extends ProtonInitializable implements Pr
@Override
public void onMessage(Delivery delivery) throws ActiveMQAMQPException {
if (closed) {
return;
}
Object message = delivery.getContext();
boolean preSettle = sender.getRemoteSenderSettleMode() == SenderSettleMode.SETTLED;
@ -544,7 +548,6 @@ public class ProtonServerSenderContext extends ProtonInitializable implements Pr
*/
public int deliverMessage(Object message, int deliveryCount) throws Exception {
if (closed) {
System.err.println("Message can't be delivered as it's closed");
return 0;
}