-
Notifications
You must be signed in to change notification settings - Fork 587
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
v2.2 does crashes in a while #85
Comments
Could you run with:
instead, please? What's your operating system? |
I think this is the same bug as #61. |
yes that makes beter sense please see output below i use this program to sync photos to google photos and it usually requires several attempts for the connenction to start working properly. then, after cca 100 photos it stops anyways mand i need to restart it. i use lg g6 thanks for development, great idea and still works better than any other requiring root
|
OK, the binary is stripped, so the trace is useless, but I know where the problem occurs with the error message. The last time I tried to understand the cause, I came to the conclusion that this may not happen (so I am obviously wrong). I'll try to understand again. Meanwhile, the java version should not suffer from the problem. What is your operating system (Linux, Mac, Windows…)? |
In a single poll() result, the client may be ready to receive a packet from the device, that will be sent to a specific network connection, and in the same time that connection may be ready to send a packet to the client. There is a possibility that both events will lead to close the connection, and remove it from the router. In that case, the second removal did panic. Instead, remove it only if it does not exist. See: - <#61> - <#85>
OK, I got it. I fixed it on
|
Thank for lightning fast fix, Ill give it a try in the afternoon. Im running KDE Neon latest stable. |
Not fixed: #61 (comment) |
Fixed. See #61 (comment) |
The text was updated successfully, but these errors were encountered: