A NullPointerException in DataHandler

[arinban]

Hi, I just see a NullPointerException in DataHandler.java.

The following is my proposal fix: /export/grizzly/src/trunk/modules/cometd/src/main/java/com/sun/grizzly/cometd > svn diff DataHandler.java Index: DataHandler.java

— DataHandler.java (revision 1115) +++ DataHandler.java (working copy) @@ -82,7 +82,10 @@ try{ if (obj instanceof DeliverResponse){ DeliverResponse deliverRes = (DeliverResponse)obj;

• if (deliverRes.getClientId().equals(getClientId())){

  • String deliverResClientId = deliverRes.getClientId();
  • if ((deliverResClientId != null &&
  • deliverResClientId.equals(getClientId()) ||
  • deliverResClientId == null && getClientId() == null)) { return; }

  res.write(deliverRes.toJSON());

Regards, Shing Wai Chan

Environment

Operating System: All Platform: All

[arinban]

• Issue Imported From: https://github.com/javaee/grizzly/issues/135
• Original Issue Raised By:@glassfishrobot
• Original Issue Assigned To:@glassfishrobot
• Original Issue Closed By:@glassfishrobot

[arinban]

@glassfishrobot Commented Reported by jfarcand@java.net

[arinban]

@glassfishrobot Commented jfarcand@java.net said: Patch applied. Thanks!

Sending src/main/java/com/sun/grizzly/cometd/DataHandler.java Transmitting file data . Committed revision 1125.

[arinban]

@glassfishrobot Commented Was assigned to jfarcand@java.net

[arinban]

@glassfishrobot Commented This issue was imported from java.net JIRA GRIZZLY-135

[arinban]

@glassfishrobot Commented Marked as fixed on Wednesday, December 16th 2009, 6:13:23 pm