#2657 Potential null pointer exception in handleConnection

Code_Cleanup
closed-fixed
Dave Blaschke
jsr48-client
5
2013-09-19
2013-08-20
Dave Blaschke
No

HttpConnectionHandler.handleConnection() assumes all instances of IOException come with a valid message and use equalsIgnoreCase() on getMessage() without checking for null.

1 Attachments

Discussion

  • Akbar Ali
    Akbar Ali
    2013-08-20

    I will be out of the office starting 08/20/2013 and will not return until
    08/26/2013.

    Please contact below in my absence.
    CIMP - Dinesh R Putchala
    Any MCP defects for SFCB - Vipin Nair
    Any CPS defects for CIMP - Dinesh R Putchala / Basabjit Sengupta
    For any other urgent issues, contact my manager Ravindra S Rao

     
  • Dave Blaschke
    Dave Blaschke
    2013-08-21

    • summary: removeListener should clear blocked IPs --> Potential null pointer exception in handleConnection
    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -1 +1 @@
    -WBEMListenerSBLIM.removeListener() should also remove the corresponding blocked IPs, if any, so that a new listener on the same port starts with a clean slate.
    +HttpConnectionHandler.handleConnection() assumes all instances of IOException come with a valid message and use equalsIgnoreCase() on getMessage() without checking for null.
    
     
  • Dave Blaschke
    Dave Blaschke
    2013-08-21

    • status: open --> open-fixed
     
  • Dave Blaschke
    Dave Blaschke
    2013-08-21

    Patch sent for community review. During a 2 week period any exploiter may comment on the patch, request changes or turn it down completely (with good reason). For the time being the patch is part of the "Experimental" branch in CVS.

     
  • Dave Blaschke
    Dave Blaschke
    2013-09-12

    • status: open-fixed --> pending-fixed
     
  • Dave Blaschke
    Dave Blaschke
    2013-09-12

    The community review is completed and we received no substantial criticism. Therefore the patch has been approved and merged into the "HEAD" branch. The next release will pick it up.

     
  • Dave Blaschke
    Dave Blaschke
    2013-09-13

    • status: pending-fixed --> closed-fixed
     
  • Dave Blaschke
    Dave Blaschke
    2013-09-13

    The patch was picked up by release 2.2.4 and will be closed.

     
  • Dave Blaschke
    Dave Blaschke
    2013-09-19

    • labels: --> Java Client (JSR48)