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

scott comer updated ETCH-25:
----------------------------

         Priority: Minor  (was: Major)
    Fix Version/s: 1.0.2

voted to include in 1.0.2 release: +1 james, jd, scott; no negatives.

> DefaultDeliveryService ends call with mb.CloseDelivery() and it should end it 
> with mb.CloseRead()
> -------------------------------------------------------------------------------------------------
>
>                 Key: ETCH-25
>                 URL: https://issues.apache.org/jira/browse/ETCH-25
>             Project: Etch
>          Issue Type: Bug
>          Components: csharp-binding, java-binding
>    Affects Versions: 1.0.0, 1.0.1
>            Reporter: scott comer
>            Assignee: scott comer
>            Priority: Minor
>             Fix For: 1.0.2
>
>
> when done reading the mailbox, the right method to call is CloseRead(). 
> somehow the code actually calls CloseDelivery(), a method used to shutdown 
> the writing side of the mailbox. CloseRead() calls CloseDelivery() first off, 
> but then takes the additional steps of redelivering any messages left in the 
> mailbox.
> The distinction is rather fine, and this bug was found by reading the source 
> code. There are certain ways you could exercise this bug, but it would be 
> very rare and not have any real impact. So, it isn't an emergency to fix it, 
> but it is also a relatively benign fix. I've already made the change in my 
> tree and tested it. The patch is attached.

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

Reply via email to