EdgeApp / airbitz-android-gui

35 stars 29 forks source link

The following bad address, as a QR code, crashes the wallet #7

Open PaulSnow opened 8 years ago

PaulSnow commented 8 years ago

Note that the given BTC address is invalid (missing last two characters, ok)

18Y4qwziefoZZyhA8yU2LjRsWh1c6J5E http://i.imgur.com/qBVeABA.png

Steps to create crash on Android Airbitz wallet:

1) Open App 2) Log in with pin 3) Press Send 4) Point at QR Code above 5) Wallet crashes.

Restart, Login, Repeat, Crash.

Add missing characters. Regenerate the QR Code. Go through steps above, and all is good.

18Y4qwziefoZZyhA8yU2LjRsWh1c6J5Eok http://i.imgur.com/GBw1Aud.png

swansontec commented 8 years ago

Yikes! Let me see If I can figure out what's going on.

paullinator commented 8 years ago

Thanks for the heads up, @PaulSnow. We'll try to squeeze a fix into the next release.

swansontec commented 8 years ago

@teneighty: The core seems to be working properly, returning an error code from ABC_ParseUri. This appears to be a GUI issue.

paullinator commented 8 years ago

@swansontec iOS seems ok. Simply pops up Invalid QR Code. I can take a look at this on Android. Should be easy.

teneighty commented 8 years ago

@PaulSnow thanks for the report. This will be fixed in the next release which should be out next week.