hc-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Oleg Kalnichevski (JIRA)" <j...@apache.org>
Subject [jira] Resolved: (HTTPCORE-163) AbstractMultiworkerIOReactor.execute() swallows exceptions
Date Sun, 15 Jun 2008 12:52:45 GMT

     [ https://issues.apache.org/jira/browse/HTTPCORE-163?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Oleg Kalnichevski resolved HTTPCORE-163.
----------------------------------------

       Resolution: Fixed
    Fix Version/s: 4.0-beta2

Committed the fix. Many thanks, Patrick

Oleg

> AbstractMultiworkerIOReactor.execute() swallows exceptions
> ----------------------------------------------------------
>
>                 Key: HTTPCORE-163
>                 URL: https://issues.apache.org/jira/browse/HTTPCORE-163
>             Project: HttpComponents HttpCore
>          Issue Type: Bug
>          Components: HttpCore NIO
>    Affects Versions: 4.0-beta2
>            Reporter: Patrick Moore
>             Fix For: 4.0-beta2
>
>
> org.apache.http.impl.nio.reactor.AbstractMultiworkerIOReactor.execute() has try{}finally
construct that hides the real problem.
> at about line 145:
> try {
>     for (;;) {
>         int readyCount;
>          ....
>    }
> } catch (ClosedSelectorException ex) {
> } finally {
>     // Shutdown
>     try {
>         doShutdown();
>     } catch (IOException ex) {
>         throw new IOReactorException(ex.getMessage(), ex);
>     }
> }
> should really be:
> boolean success = false;
> try {
>     for (;;) {
>         int readyCount;
>          ....
>    }
>    success = true;
> } catch (ClosedSelectorException ex) {
>       success = true;
> } finally {
>     // Shutdown
>     try {
>         doShutdown();
>     } catch (IOException ex) {
>         if ( success)  {
>            throw new IOReactorException(ex.getMessage(), ex);
>         }
>     }
> }
>       

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


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


Mime
View raw message