mirror of
https://github.com/rustdesk/rustdesk.git
synced 2026-03-02 19:26:56 -05:00
[Crash] app often crashes (Android 6.0 and Android 7.1 , armv7) #1679
Labels
No labels
bug
documentation
duplicate
enhancement
enhancement
enhancement
good first issue
help wanted
invalid
question
unreproducible
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/rustdesk-rustdesk#1679
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @areful on GitHub (Apr 19, 2023).
Bug Description
Often crash on Android (as client).
How to Reproduce
When android client connected with remote server, it works well.
But when I disconnect from remote server manually by click "Disconnect" button, it's often crash. Not every time, but too often.
Expected Behavior
Works well when click disconnect.
Operating system(s) on local side and remote side
Windows
RustDesk Version(s) on local side and remote side
1.20 -> 1.1.9
Screenshots
Additional Context
part of crash log:
@rustdesk commented on GitHub (Apr 19, 2023):
Can you try out 1.2 android?
@areful commented on GitHub (Apr 19, 2023):
Tried 1.2 on Android 6.0 and Android 7.1 and still crashed.
The apk from here:
https://github.com/rustdesk/rustdesk/releases/download/nightly/rustdesk-1.2.0-armv7-signed.apk
It's OK on android 12.
@rustdesk commented on GitHub (Apr 19, 2023):
Frankly wontfix for old android version, and specifically it is armv7. Anyway, Thanks for your report.
@areful commented on GitHub (Apr 19, 2023):
So sed :(
@rustdesk commented on GitHub (Apr 19, 2023):
There is probably some threading sync issue, you may have a look.
github.com/rustdesk/rustdesk@6859bfae98/flutter/android/app/src/main/kotlin/com/carriez/flutter_hbb/MainService.kt (L132)stopCapturehere is called in another thread.@rustdesk commented on GitHub (Apr 19, 2023):
Reopen because obvious threading safe issue
@rustdesk commented on GitHub (Apr 19, 2023):
@21pages
@areful commented on GitHub (Apr 20, 2023):
Thanks, let me have a try~
@wei3 commented on GitHub (Apr 20, 2023):
I also have this crash problem
@areful commented on GitHub (Apr 20, 2023):
@21pages any help?
@rustdesk commented on GitHub (Apr 20, 2023):
@21pages does not know this part of code either, the original author has left, so we have the same level of knowledge.
@rustdesk commented on GitHub (Apr 20, 2023):
I do not think below change help. It only ensure stopCapture not called paralelly, but the crash arises from the other method is using the resource while stopCapture is releasing them. @areful hope this can help you find some clue.
github.com/rustdesk/rustdesk@f89455f31d/flutter/android/app/src/main/kotlin/com/carriez/flutter_hbb/MainService.kt (L387)@areful commented on GitHub (Apr 20, 2023):
Thanks for your reply. Have no idea now ~~
@wei3 commented on GitHub (Apr 23, 2023):
I tried, the problem can still be reproduced when disconnected
@rustdesk commented on GitHub (Apr 23, 2023):
You can try your solution then if possbile. :)
@rustdesk commented on GitHub (May 13, 2023):
Another piece of stupid code, https://github.com/rustdesk/rustdesk/pull/4353#discussion_r1192921604
github.com/rustdesk/rustdesk@4d52edf46d/libs/scrap/src/android/ffi.rs (L56)@wei3 commented on GitHub (May 24, 2023):
Thanks a lot, let's try
@as886944123 commented on GitHub (Jul 22, 2023):
I also have this crash issue, especially when my phone restarts and the ID generation card dies
@akshaydoshi2 commented on GitHub (Sep 24, 2023):
Hey, Is this issue for the flutter mobile application?
If yes, do you have some documentation to run/build the flutter application?
I am trying to do so, but the I am facing some issues with it
development OS: MacOS
@areful commented on GitHub (Sep 25, 2023):
So far we can only control the app itself on Android 6.0 with the .so library using ARMv7 ABI. Not whole system, because there is no GestureDetector APIs below android 7.
@akshaydoshi2 commented on GitHub (Sep 25, 2023):
Okay. I can try and work on this.
But I am not able to run the app. Can you help me with that?
Are there any dependencies I need to install apart from
pub getbefore I am able to run the app?@areful commented on GitHub (Sep 25, 2023):
First you should compile the .so file for ARMv7 ABI. U can according this document: (https://rustdesk.com/docs/en/dev/build/android). There are still a lot of works in it.
@akshaydoshi2 commented on GitHub (Sep 25, 2023):
Ahh got it. I need to install rust/cargo as well.
Thanks
@rustdesk commented on GitHub (Dec 2, 2023):
tracked by #3893
@rustdesk commented on GitHub (Apr 19, 2024):
Wondering if some one can test the nightly, https://github.com/rustdesk/rustdesk/releases/tag/nightly, I put some fix for this.
@haisong66 commented on GitHub (Apr 28, 2024):
Windows远程安卓7.1 armv7 鼠标滑动控制一会, 安卓设备就会报错停止运行
@haisong66 commented on GitHub (Apr 28, 2024):
Version 1.2.2 works fine on Android7.1
@rustdesk commented on GitHub (Apr 28, 2024):
crash report?
@haisong66 commented on GitHub (Apr 29, 2024):
--------- beginning of crash
@haisong66 commented on GitHub (Apr 29, 2024):
@haisong66 commented on GitHub (May 9, 2024):
Hello, can this problem be fixed?
@rustdesk commented on GitHub (May 9, 2024):
Maybe if you fund this bug, we offiically will not spend additional time on armv7 or old android.
@haisong66 commented on GitHub (May 9, 2024):
Approximately how much
@rustdesk commented on GitHub (May 9, 2024):
@sahilyeole you can give a quote, if you are interested.
@haisong66 commented on GitHub (May 9, 2024):
Give me a price and I'll see if it's worth it. After all, there are relatively few devices serving lower versions now.
@Rikj000 commented on GitHub (Jun 4, 2024):
Can confirm that RustDesk
v1.2.2works well on Androidv7.1! 🎉However it appears like the
.apkfiles have been removed from the RustDeskv1.2.2release page...After some digging I was able to find them though:
It would be nice if these could be re-included in the
v1.2.2release@rustdesk commented on GitHub (Jun 4, 2024):
The only change from 1.2.2 to 1.2.3 is the targetSdkVersion change from 31 to 33, I guess this changes the behavior of local reference table limit.
@rustdesk commented on GitHub (Jun 4, 2024):
Try out https://github.com/rustdesk/rustdesk/releases/tag/nightly please, I had a fix.
Please paste the crash log if still crash. I have removed "mouse" string, at least it wont have local reference table overflow caused by "mouse" jni string.
@Rikj000 commented on GitHub (Jun 5, 2024):
Just tried the
v1.2.6nightly build on Androidv7.1@ commit:github.com/rustdesk/rustdesk@65f7541ec7The nightly build:
v1.2.3-2, the session only crashed after remote user input)@rustdesk commented on GitHub (Jun 5, 2024):
Thanks for you testing, could you please test current nightly build again? we just put a new fix. https://github.com/rustdesk/rustdesk/pull/8266
@Rikj000 commented on GitHub (Jun 5, 2024):
Tested the new
v1.2.6nightly (2024-06-05 10:17) build on an Androidv7.1.1and av7.1.2.And can happily confirm that the fix resolved the issue! 🥳
@blankhang commented on GitHub (Jun 19, 2024):
https://github.com/rustdesk/rustdesk/releases/download/1.2.6/rustdesk-1.2.6-armv7-signed.apk
works on Android 7.1.2 again
THANKS A LOTS