We're updating the issue view to help you get more done. 

NullPointerException in RoutingTableImpl.

Description

https://github.com/igniterealtime/Openfire/pull/534

In #routeToComponent, it is first checked if the component is being hosted in the local JVM. If it was not found, then all nodes in the ComponentsCache are checked to find if the component is hosted by one of those. In this process, the local routing table is again searched for the component, because it may have been added after the previous check. However, after this search, there is no null check done for the searched route. This results in an NPE when the component was not found locally and the #process(packet) method is called on the object.

This also prevents the code from searching other nodes that may have been in the nodes set. Added a null check to prevent this scenario.

Environment

None

Acceptance Test - Entry

None

Assignee

Dave Cridland

Reporter

Daryl Herzmann

Labels

None

Expected Effort

None

Ignite Forum URL

None

Components

Fix versions

Affects versions

Priority

Major
Configure