tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 47216] Possible NPE in Http11Processor.action()
Date Mon, 25 May 2009 11:00:46 GMT
https://issues.apache.org/bugzilla/show_bug.cgi?id=47216


Mark Thomas <markt@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |WONTFIX




--- Comment #3 from Mark Thomas <markt@apache.org>  2009-05-25 04:00:45 PST ---
The answer is in the svn history. See r298828 ( https://svn.apache.org/viewcvs.cgi?view=rev&rev=298828
) and r299051 ( https://svn.apache.org/viewcvs.cgi?view=rev&rev=299051 ).

It may be the case that later changes have removed the need for some of these
null tests. A cursory review suggests that a few of the tests could be removed
but the risk of introducing an NPE for an unusual code path outweighs the
minimal benefit of removing these tests.

I have changed the scope of parseHost() to protected for trunk. It will not be
backported as it breaks the API.

Since the (socket != null) was the main thrust of this report, I am marking it
as WONTFIX.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org


Mime
View raw message