Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Lockup problem #50

Closed
arromdee opened this issue Jun 20, 2016 · 3 comments
Closed

Lockup problem #50

arromdee opened this issue Jun 20, 2016 · 3 comments

Comments

@arromdee
Copy link

arromdee commented Jun 20, 2016

Using heimdal-ftp on Linux, I try to connect and manage to connect. Then I do a ls or get. The result: ftpd displays "502 Invalid command "EPRT|1|192.168.1.2|52727"

It then locks up and refuses to respond to the Home button or anything else until the ls command times out.

It shouldn't lock up, and also you'd probably be better off returning a response code 522 indicating that you don't support ipv6.

I am able to connect using other ftp clients that apparently don't try to use ipv6.

@mtheall
Copy link
Owner

mtheall commented Jun 20, 2016

The EPRT command failing looks fine (it's not trying to use IPv6 here btw); heimdal-ftp appears to handle this fine, then proceeds to fallback to a PORT command, followed by a LIST command (for ls). I did get the hang you are talking about. However, a PORT command means the 3DS will connect to your client, so after turning off my firewall, it worked with no problems.

The firewall should have caused the connection to fail immediately (i.e. connection refused) but this might be yet another quirk of SOC:U, which means the whole process is hanging waiting for connect() to return. That would explain why not even the Home button works. I will change the connect() to be non-blocking which should fix the hang.

@arromdee
Copy link
Author

I was confused by the fact that most Google searches I get for EPRT happen when people are having trouble with ipv6. This was indeed a firewall problem (though the lockup did still happen).

@mtheall
Copy link
Owner

mtheall commented Jun 21, 2016

Fixed in 81ec2ea

@mtheall mtheall closed this as completed Jun 21, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants