SofaFTP bug report thread

Page 2/3
1 | | 3

By raymond

Champion (364)

raymond's picture

15-02-2019, 12:10

Hello Louthrax,

I will be in Nijmegen and very happy to show you, however, I do not have a public FTP available for that purpose. So we have to take a look on how we can achieve the test.

By raymond

Champion (364)

raymond's picture

15-03-2019, 14:00

I have done a test with the /u option as mentioned by you. With the /u option it works fine, without that option I get a corrupted file.

By raymond

Champion (364)

raymond's picture

25-03-2019, 20:54

Is anyone able to reproduce this error?

By ducasp

Rookie (32)

ducasp's picture

31-05-2019, 07:00

Hi,

I have an OCM-Like FPGA system that is running Nextor. It is using an UNAPI driver for an ESP8266 Wifi device that I'm developing. I could get sftp to work really fine, about 20KB/s download and about 16KB/s upload (once I've turned off delayed ack response in Windows). But, when using recursive commands, after sending or receiving the first file in the "todo" list, SFTP is aborting with this message:

*** User aborted

The file is uploaded/downloaded just fine, just that the recursive feature seems to not go on... As far as FTP server goes, it seemed to reply just fine with "226 Transfer complete.\r\n"

I believe that it is most likely some issue on my side, but any hints/help will be appreciated. Tongue

By ducasp

Rookie (32)

ducasp's picture

31-05-2019, 18:37

It seems the problem is not network related, doing a single file transaction everything goes fine, SFTP receives the status of transfer finished just fine, but after Transfer Completed message, the *** User aborted message comes (in this test I just did everything manually through commands like VERBOSE 1, USER anonymous, PASS, DIR, GET PENGO.ROM), then the file do not have the same time as the FTP server but instead it has the computer time (that is the actual time in my time zone) when receiving the file. I believe the problem is occurring in this part of the code:

if(ulFTPFileTime)
{
DOS_cSetFileTime(_szFile, ulFTPFileTime);
}

Most likely it is not being able to set the file time , causing a DOS error that is catched because of this function:

DOS_defineDiskAbortRoutine(vExit);

My system is an OCM like system, like I've said, running an SD card with Nextor (probably not the latest from the latest KDL pack, I believe it is using the kernel in SDBIOS pack of 3.7 that do not include the latest beta Nextor, I will try the latest to see if it changes anything).

By the way, would it be possible to consider the possibility of having SFTP getting the PORT for non 21 port FTP servers?

By Louthrax

Paragon (2044)

Louthrax's picture

31-05-2019, 21:17

Hi Ducasp,

Thanks for the bug reporting. The fact that the error returned is a "User error" is weird (it should be a MSX-DOS error when failing on an MSX-DOS function). What is the time of your source PENGO.ROM file on the FTP server ? Is it valid ?

Also, it could be that the problem is related to the "pre-allocation" mode, that is now forced in all my Sofa tools. I decided to force the option to have that finally fixed. I think that I've found a working patch (tested on SofaRun and SofaRunIt), and that fix should also have been applied to SFTP. You can try to download SFTP again (I put that online last week IIRC), and see if it changes something.

If not, I can provide you some instrumented versions of SFTP to identify the problem.

PS: Changing the default FTP port is definitively doable (I can add a new command for that purpose).

By ducasp

Rookie (32)

ducasp's picture

01-06-2019, 02:57

Hi Louthrax,

The dir command do not have the year information, but mdtm seems to return the correct year/complete date. I'm using Windows 7 built-in FTP server.

I'm using version 1.1, I've just re-downloaded it just in case and got the same results (images are from that version). I'll be glad to test any instrumented version of it, you can just send it to my e-mail.

Thanks!

By Louthrax

Paragon (2044)

Louthrax's picture

01-06-2019, 08:34

Mail sent !

By Louthrax

Paragon (2044)

Louthrax's picture

01-06-2019, 18:42

OK, found the problem: I was trying to set the date / time on the file when it was not closed yet. The error was not catched in the past (I just added a generic error catching mechanism in my lower MSX-DOS functions). I've uploaded a fixed version on my website. Let me know if that works!

By raymond

Champion (364)

raymond's picture

09-06-2019, 08:13

Hello Louthrax,

Is the upload bug fixed in this release as well?

Page 2/3
1 | | 3