You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hc.apache.org by "Karl Wright (JIRA)" <ji...@apache.org> on 2013/01/10 03:08:12 UTC

[jira] [Commented] (HTTPCLIENT-1296) Null pointer exception when using default host and virtual host with port -1

    [ https://issues.apache.org/jira/browse/HTTPCLIENT-1296?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13549290#comment-13549290 ] 

Karl Wright commented on HTTPCLIENT-1296:
-----------------------------------------

I also have a trivial test that demonstrates the problem and fix; I will include this if I get the go-ahead for inclusion in 4.2.3.  Meanwhile I'm going to apply both the fix and the test to trunk.

                
> Null pointer exception when using default host and virtual host with port -1
> ----------------------------------------------------------------------------
>
>                 Key: HTTPCLIENT-1296
>                 URL: https://issues.apache.org/jira/browse/HTTPCLIENT-1296
>             Project: HttpComponents HttpClient
>          Issue Type: Bug
>    Affects Versions: 4.2.2
>            Reporter: Karl Wright
>            Assignee: Karl Wright
>            Priority: Critical
>         Attachments: HTTPCLIENT-1296.patch
>
>
> The following code in DefaultRequestDirector is incorrect:
> {code}
>         // HTTPCLIENT-1092 - add the port if necessary
>         if (virtualHost != null && virtualHost.getPort() == -1) {
>             int port = target.getPort();
>             if (port != -1){
>                 virtualHost = new HttpHost(virtualHost.getHostName(), port, virtualHost.getSchemeName());
>             }
>         }
> {code}
> The reason it is incorrect is that if the target has not been specified in the URL, then an NPE will result.  The target is resolved later in the same method in that case.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

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