[CXF-3429] ServerImpl should shutdown Destination before setMessageObserver=null

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.2.10
• Fix Version/s: 2.4, 2.3.4
• Component/s: Core
• Assignee: Willem Jiang
• Reporter: David Liu
• Created: 29/Mar/11 06:43
• Updated: 18/Apr/11 12:42
• Resolved: 29/Mar/11 13:49

Description

Hi,

During shutdown the CXF server, we encounter the NullPointerException from time to time.
The issue is that ServerImpl will set MessageObserver to null, and then it shutdown desination. however, if the server is multi-thread and probably some thread is runing, then it can result in NullPointerException because MessageObserver is null.

Can we fix this bug by shutdown the Destination first and then set MessageObserver to null?

Also, do we have a workaround for this issue?

public class ServerImpl implements Server {
public void stop() {
    LOG.fine("Server is stopping.");
    ...
    getDestination().setMessageObserver(null);
    getDestination().shutdown();
}

David