Some XMPP messages are causing NPE

Description

r181 has the offending line as:
97 final JID legacyJID = transport.convertIDToJID(message.getFrom());

Logs are:
2010.04.19 18:13:44 E001:null
java.lang.NullPointerException
at net.sf.kraken.protocols.xmpp.XMPPListener.processMessage(XMPPListener.java:97)
at org.jivesoftware.smack.Chat.deliver(Chat.java:169)
at org.jivesoftware.smack.ChatManager.deliverMessage(ChatManager.java:221)
at org.jivesoftware.smack.ChatManager.access$200(ChatManager.java:42)
at org.jivesoftware.smack.ChatManager$2.processPacket(ChatManager.java:118)
at org.jivesoftware.smack.PacketReader$ListenerWrapper.notifyListener(PacketReader.java:819)
at org.jivesoftware.smack.PacketReader$ListenerNotification.run(PacketReader.java:799)
at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:441)
at java.util.concurrent.FutureTask$Sync.innerRun(FutureTask.java:303)
at java.util.concurrent.FutureTask.run(FutureTask.java:138)
at java.util.concurrent.ThreadPoolExecutor$Worker.runTask(ThreadPoolExecutor.java:886)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:908)
at java.lang.Thread.run(Thread.java:619)

Environment

Kraken r181

Activity

Show:
daniel vultur
September 22, 2010, 11:06 PM

Moving this to 1.1.4 simply because I can not reproduce it anymore but I suspect there is something going on here still.

daniel vultur
April 21, 2010, 3:29 PM

This was seem in Kraken 1.1.2 as well!
At the time it was line 90.
That lines up with:
90 getSession().getTransport().sendMessage(
91 getSession().getJID(),
92 getSession().getTransport().convertIDToJID(message.getFrom()),
93 message.getBody()
94 );

I wonder which one here is "wrong".

Assignee

Daniel Henninger

Reporter

Daniel Henninger

Labels

None

Expected Effort

None

Ignite Forum URL

None

Components

Fix versions

Affects versions

Priority

Major