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

Updated Acktie-Zbar-Android and libiconv.so to fix crashes #14

Merged
merged 2 commits into from
Jul 14, 2016

Conversation

jkotchoff
Copy link

Merging latest fixes from https://github.com/jkotchoff/Acktie-Zbar-Android and updated libiconv.so to address intermittent crashes in this module.

Addresses stability issues specified at:
#12
#6
#2

Merging latest fixes from https://github.com/jkotchoff/Acktie-Zbar-Android to address intermittent crashes in this module.

Addresses stability issues specified at:
acktie#12
acktie#6
acktie#2
Updated with rebuilt libiconv.so files as per:
dm77/barcodescanner@00d4792

Also, removed the unused zbar.jar files as per:
acktie/Acktie-Zbar-Android@e671ed0

This fixes QR scanner module crashes on a test nexus device running Android 6 that were showing stack traces like:
07-14 09:49:27.086: E/linker(26751): /data/app/com.teamapp.teamapp1-1/lib/arm/libiconv.so: has text relocations
07-14 09:49:27.087: D/AndroidRuntime(26751): Shutting down VM
07-14 09:49:27.088: E/AndroidRuntime(26751): FATAL EXCEPTION: main
07-14 09:49:27.088: E/AndroidRuntime(26751): Process: com.geoplusapp.geoplusapp, PID: 26751
07-14 09:49:27.088: E/AndroidRuntime(26751): java.lang.UnsatisfiedLinkError: dlopen failed: /data/app/com.geoplusapp.geoplusapp/lib/arm/libiconv.so: has text relocations
07-14 09:49:27.088: E/AndroidRuntime(26751):  at java.lang.Runtime.loadLibrary(Runtime.java:372)
07-14 09:49:27.088: E/AndroidRuntime(26751):  at java.lang.System.loadLibrary(System.java:1076)
07-14 09:49:27.088: E/AndroidRuntime(26751):  at com.acktie.mobile.android.qr.AcktiemobileandroidqrModule.<clinit>(AcktiemobileandroidqrModule.java:23)
@TNuzzi TNuzzi merged commit a9ec9c6 into acktie:master Jul 14, 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

Successfully merging this pull request may close these issues.

2 participants