iotaledger / trinity-wallet

Trinity is IOTA's old, deprecated wallet. Use Firefly instead.
https://trinity.iota.org
Other
470 stars 120 forks source link

java.lang.IllegalArgumentException when hashing password #591

Open rajivshah3 opened 5 years ago

rajivshah3 commented 5 years ago

Error in Trinity Mobile

java.lang.IllegalArgumentException in NavigationActivity Factors and values given to Argon2 must be positive and have a positive length (encodedlen may be 0)

View on Bugsnag

Stacktrace

java.lang.IllegalArgumentException Factors and values given to Argon2 must be positive and have a positive length (encodedlen may be 0) 
Argon2.java:-2 de.wuthoehle.argon2jni.Argon2.argon2jni_hash
Argon2.java:152 de.wuthoehle.argon2jni.Argon2.argon2_hash_raw
Argon2Android.java:65 org.iota.mobile.Argon2Android.hash
Method.java:-2 java.lang.reflect.Method.invoke
JavaMethodWrapper.java:374 com.facebook.react.bridge.JavaMethodWrapper.invoke
JavaModuleWrapper.java:162 com.facebook.react.bridge.JavaModuleWrapper.invoke
NativeRunnable.java:-2 com.facebook.react.bridge.queue.NativeRunnable.run
Handler.java:751 android.os.Handler.handleCallback
Handler.java:95 android.os.Handler.dispatchMessage
MessageQueueThreadHandler.java:31 com.facebook.react.bridge.queue.MessageQueueThreadHandler.dispatchMessage
Looper.java:154 android.os.Looper.loop
MessageQueueThreadImpl.java:194 com.facebook.react.bridge.queue.MessageQueueThreadImpl$3.run
Thread.java:762 java.lang.Thread.run

View full stacktrace

Created by Rajiv Shah via Bugsnag

cvarley100 commented 5 years ago

Closing due to age.

rajivshah3 commented 5 years ago

An error linked to this issue has been marked as fixed in Bugsnag java.lang.IllegalArgumentException in MainActivity

rajivshah3 commented 5 years ago

An error linked to this issue has been reopened in Bugsnag java.lang.IllegalArgumentException in Argon2.java:152

cvarley100 commented 5 years ago

@rajivshah3 Any ideas on root cause? Bugsnag says Factors and values given to Argon2 must be positive and have a positive length

rajivshah3 commented 5 years ago

@cvarley100 this looks like an issue with the Android Argon2 module or how we pass values to it