Igor Yanchenko

I tried to remove the assert condition and it works, but I guess it's there for a reason, maybe @romanz has ideas.

This error happens because you are running `leftoverdata = util.read_frame(i)` on empty `i`. if you ignore the leftover then it works. Probably the protocol has changed, and the data there...

I see, I was testing with ed25519

I cannot reproduce the error after removing assert string. even if I use nistp256.

@mavaa what version trezorlib are you you using? mine is `0.13.0`

I am running trezord-git https://aur.archlinux.org/packages/trezord-git I tried your script, but I still didn't manage to reproduce, could you try to use trezord-git and check?

By the same error you mean the original error? ``` ... assert not i.read() AssertionError ``` or ``` raise interface.DeviceError(msg) libagent.device.interface.DeviceError: Trezor error: DataError: bytes overflow ```

I can't reproduce that error

@mavaa I tried that version previous time, removing the assert condition helped me, but as you said it doesn't fix the problem for you, so I didn't create any PR...

@nkichukov could you please check if it works for you if you comment the assert line? It fixed the problem for me, but it didn't fix for others. Thank you.