Skip to content

Conversation

@udfvirus
Copy link

@udfvirus udfvirus commented Jul 26, 2025

Previous implementation copied and executed the OpenVPN binary from cache, which is no longer necessary or reliable on Android 7+. This patch ensures that for API < 28 the binary is executed directly from nativeLibraryDir, aligning the behavior with API >= 28.

This resolves an issue where c_pie_openvpn.armeabi-v7a was missing from cache and caused crashes on older devices like Samsung Galaxy A3 (Android 7).

@schwabe
Copy link
Owner

schwabe commented Jul 26, 2025

What platforms mobile phones have you actually tested before making this change? This is a huge potential to cause regression on other phones that worked for years if the behaviour there is changed without retesting them with this change.

…s device's ABIs [armeabi-v7a, armeabi] on Samsung SM-A310F(Galaxy A3(2016))
@udfvirus
Copy link
Author

udfvirus commented Jul 27, 2025

Thanks for the comment, I fixed the commit)
I checked the work on Samsung SM-A310F (Galaxy A3 (2016)) and on the emulator with Android 7, also checked the work on Xiaomi Note 12 (Android 14), Xiaomi Note 7 (Android 10)

@schwabe
Copy link
Owner

schwabe commented Jul 27, 2025

But still the question remain what happens on your A3 device:

   // Fixed: java.lang.RuntimeException: Cannot find any executable for this device's ABIs [armeabi-v7a, armeabi] on Samsung SM-A310F(Galaxy A3(2016))

is that the official build of mine or some custom build that doesn't have all ABIs presents?

Is this an 64 bit device or 32 bit device?

Also what Android API does it have.

I really would like to understand why your device fails and needs a fix when I never heard of this problem before.

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

Successfully merging this pull request may close these issues.

2 participants