Re: bug in ZoneTransferIn.run

[Brian W. <bwe...@xb...>]

On Tue, 24 Jan 2006, Can Bican wrote:

> Where zone is valid zone to transfer and ns is a server that raises
> ConnectException for this zone. The code above fails quite quickly
> via:
>
> [snip]
>
> $ diff original/org/xbill/DNS/ZoneTransferIn.java
> new/org/xbill/DNS/ZoneTransferIn.java
> 553,554c553,554
> <       openConnection();
> <       try {
> ---
>>     try {
>>         openConnection();
>
> Does that make sense as a bugfix? :)

Looks correct.  I'll apply this, and also make closeConnection() check 
that client is non-null before using it.

Brian