[jira] [Commented] (NET-477) TFTP sendFile retry broken

["Niranjana Mai TM (JIRA)" <ji...@apache.org>]

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

Niranjana Mai TM commented on NET-477:
--------------------------------------

I have similar issue. TFTP sendFile does not throw exception if an invalid/ not reachable IP address is given in the place of host name.
The "timeouts" variable is getting reset to 0 each time continue _sendPacket; is called from exception, so SocketException / IOException Connection timed out. is never thrown.

> TFTP sendFile retry broken
> --------------------------
>
>                 Key: NET-477
>                 URL: https://issues.apache.org/jira/browse/NET-477
>             Project: Commons Net
>          Issue Type: Bug
>          Components: TFTP
>    Affects Versions: 3.1, 3.2
>         Environment: Linux **** 3.2.0-29-generic #46-Ubuntu SMP Fri Jul 27 17:03:23 UTC 2012 x86_64 x86_64 x86_64 GNU/Linux
> java version "1.6.0_26"
> Java(TM) SE Runtime Environment (build 1.6.0_26-b03)
> Java HotSpot(TM) 64-Bit Server VM (build 20.1-b02, mixed mode)
>            Reporter: John Walton
>
> TFTPClient.java sendFile() method does not retry after initial timeout.
> Between revisons 1088720 & 1229692 the 'continue' statements were changed to 'continue _sendPacket'. After the initial timeout of bufferedReceive() these cause the control to flow to the outer loop and never retry. If the control did flow accordingly, the retry would loop indefinitely since the timeouts variable is reset to zero each time as well.
> I am using a workaround by calling bufferedSend(sent) just before continue, in place of the 'continue _sendPacket' statement.
> It looks as if similar logic was applied to the receiveFile() method as well.
> Revision:1229692
> Time:01/10/2012 01:14 PM
> Author:sebb
> Path:http://svn.apache.org/repos/asf/commons/proper/net/trunk/src/main/java/org/apache/commons/net/tftp/TFTPClient.java
> Message:NET-410 Apache Commons TFTP does not handle RFC 783 retransmits



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)