spesmilo / spesmilo/electrum

InvalidECPointException:

Open
#5,488 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Python
Stars
8.6k
Forks
3.5k
Avg merge
2d 8h
Merged PRs (30d)
61

Description

Crash Report

This crash report was reported through the automatic crash reporting system 🤖

Traceback

  File "electrum/gui/qt/__init__.py", line 215, in wrapper
  File "electrum/gui/qt/__init__.py", line 239, in start_new_window
  File "electrum/gui/qt/__init__.py", line 294, in _start_wizard_to_select_or_create_wallet
  File "electrum/wallet.py", line 1990, in __new__
  File "electrum/wallet.py", line 1821, in __init__
  File "electrum/wallet.py", line 1675, in __init__
  File "electrum/wallet.py", line 1784, in synchronize
  File "electrum/wallet.py", line 1771, in synchronize_sequence
  File "electrum/wallet.py", line 1758, in create_new_address
  File "electrum/wallet.py", line 1751, in derive_address
  File "electrum/wallet.py", line 1850, in derive_pubkeys
  File "electrum/keystore.py", line 458, in derive_pubkey
  File "electrum/keystore.py", line 453, in get_pubkey_from_mpk
  File "electrum/ecc.py", line 188, in __init__
  File "electrum/ecc.py", line 128, in _ser_to_python_ecdsa_point

InvalidECPointException: 

Reporter

This issue was reported by 1 user(s):

Electrum Version Python Version Operating System Wallet Type Locale Stack
3.3.7 3.6.4 (default, Feb 28 2018, 03:24:19) [GCC 4.2.1 Compatible Apple LLVM 8.0.0 (clang-800.0.42.1)] Darwin-17.7.0-x86_64-i386-64bit standard ?

Additional Information

The reporting user(s) did not provide additional information.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the traceback in electrum/gui/qt/init.py and follow the wallet flow through electrum/wallet.py, electrum/keystore.py, and electrum/ecc.py, especially _ser_to_python_ecdsa_point. Reproduce the reported standard-wallet creation path if possible and determine why the public key becomes an invalid EC point. Done means that this flow no longer raises InvalidECPointException and has a regression test.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
blockchain, cryptography
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.