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

[Errno 104] Connection reset by peer #1

Open
kakopappa opened this issue Jan 10, 2023 · 4 comments
Open

[Errno 104] Connection reset by peer #1

kakopappa opened this issue Jan 10, 2023 · 4 comments

Comments

@kakopappa
Copy link

kakopappa commented Jan 10, 2023

First of all, thanks for sharing this project. It's a great idea!

I am seeing [Errno 104] Connection reset by peer as soon as i do curl http://localhost:8080 I was wonding whether you had the same problem and how did you solve it ?

From the ESP. Looks fine

23:55:09.927 -> WNET xxxx FORWARD TO xxxxxxx
23:55:10.443 -> WLAN_CONN_RETRYWLAN_CONNECTED
23:55:10.443 -> IP: xxxx MAC: xxx
23:55:10.443 -> CONN_LISTENER
23:55:10.443 -> CONN_TCP
23:55:10.770 -> RECV_FROM_LISTENER
23:55:11.049 -> RECV_NOP
23:55:11.049 -> RECV_NOP
23:55:11.049 -> RECV_NOP
23:55:11.049 -> RECV_NOP
23:55:11.563 ->
23:55:11.563 ->
23:55:11.563 -> RECV_AUTH

Python

Waiting for ESP-32 to connect to us...
('Got connection from ESP32:', ('xxxxx', 52826))
Starting local input socket on 127.0.0.1 port 8080...
('Got local connection from:', ('127.0.0.1', 42714))
Sending 78 bytes to the target...
[Errno 104] Connection reset by peer
Waiting for ESP-32 to connect to us...

@userlandkernel
Copy link
Owner

I think I have some flaws in the protocol design that need fixing and I’ll look into this in the coming days. Thanks for reporting these bugs!

@userlandkernel
Copy link
Owner

What I expect is that perhaps a buffer overflow vulnerability is causing this issue on the ESP side

@kakopappa
Copy link
Author

Great let me know if you need help testing

@userlandkernel
Copy link
Owner

I identified a flaw in the protocol where I was sending a DWORD from the client instead of a byte.
Nevertheless I think this project needs a redesign so I’ll be busy rewriting it and then push the fix

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