NPE in SessionManager.processCommand (#1225)

Closed

Activities

wojciech.kapcia@tigase.net opened 4 years ago

[2020-12-03 10:27:34:424] [SEVERE  ] [        in_10-sess-man ] AbstractMessageReceiver$QueueListener.run(): [in_10-sess-man] Exception during packet processing: from=c2s@ip-10-0-8-212.us-west-2.compute.internal/10.0.8.212_5222_61.173.100.241_49316, to=sess-man@ip-10-0-8-212.us-west-2.compute.internal, DATA=<iq id="moved" type="set" from="c2s@ip-10-0-8-212.us-west-2.compute.internal/10.0.8.212_5222_61.173.100.241_49316" to="sess-man@ip-10-0-8-212.us-west-2.compute.internal"><command node="STREAM_MOVED" xmlns="http://jabber.org/protocol/commands"><x type="submit" xmlns="jabber:x:data"><field var="old-conn-jid"><value>CData size: 81</value></field></x></command></iq>, SIZE=431, XMLNS=null, PRIORITY=NORMAL, PERMISSION=NONE, TYPE=set, STABLE_ID=null
java.lang.NullPointerException
	at tigase.server.xmppsession.SessionManager.processCommand(SessionManager.java:1631)
	at tigase.cluster.SessionManagerClustered.processPacket(SessionManagerClustered.java:240)
	at tigase.server.AbstractMessageReceiver$QueueListener.run(AbstractMessageReceiver.java:1397)

wojciech.kapcia@tigase.net commented 4 years ago

Most likely due to concurrency - old session is removed while it's resumed. I added check and notification to CM to properly handle it.

Type	Bug
Priority	Normal
Assignee	wojciech.kapcia@tigase.net
Version	tigase-server-8.2.0
Spent time	1h

Issue Votes (0)

Watchers (0)

Reference

tigase/_server/server-core#1225