You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Sebb (JIRA)" <ji...@apache.org> on 2017/02/09 15:57:41 UTC

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

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

Sebb resolved NET-477.
----------------------
       Resolution: Fixed
    Fix Version/s: 3.6

The TFTP send/receive logic has been completely rewritten and now appears to handle retries properly.

URL: http://svn.apache.org/viewvc?rev=1782352&view=rev
Log:
NET-477 TFTP sendFile retry broken

Modified:
    commons/proper/net/trunk/src/changes/changes.xml
    commons/proper/net/trunk/src/main/java/org/apache/commons/net/tftp/TFTPClient.java



> 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
>             Fix For: 3.6
>
>
> 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.15#6346)