Unsubscribing from an endpoint that no longer exists throws NPE [SPR-11832]

[spring-projects-issues]

Jon Schneider opened SPR-11832 and commented

Exception in thread "clientInboundChannel-4" java.lang.NullPointerException
at java.util.concurrent.ConcurrentHashMap.hash(ConcurrentHashMap.java:332)
at java.util.concurrent.ConcurrentHashMap.get(ConcurrentHashMap.java:987)
at org.springframework.messaging.simp.broker.DefaultSubscriptionRegistry$SessionSubscriptionInfo.getSubscriptions(DefaultSubscriptionRegistry.java:315)
at org.springframework.messaging.simp.broker.DefaultSubscriptionRegistry.removeSubscriptionInternal(DefaultSubscriptionRegistry.java:98)
at org.springframework.messaging.simp.broker.AbstractSubscriptionRegistry.unregisterSubscription(AbstractSubscriptionRegistry.java:97)
at org.springframework.messaging.simp.broker.SimpleBrokerMessageHandler.handleMessageInternal(SimpleBrokerMessageHandler.java:154)
at org.springframework.messaging.simp.broker.AbstractBrokerMessageHandler.handleMessage(AbstractBrokerMessageHandler.java:171)
at org.springframework.messaging.support.ExecutorSubscribableChannel$1.run(ExecutorSubscribableChannel.java:70)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1110)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:603)
at java.lang.Thread.run(Thread.java:722)

---

Affects: 4.0.5

Issue Links:

• The NullPointerException occur when stomp client unsubscribing [SPR-11838] #16457 The NullPointerException occur when stomp client unsubscribing ("is duplicated by")

Referenced from: commits 5a8e470, e505072

[spring-projects-issues]

Rossen Stoyanchev commented

Pull request submitted #549

[spring-projects-issues]

Juergen Hoeller commented

Fixed through a "destination != null" check in removeSubscriptionInternal.

Juergen