From 65f3d780842d80a5fc919ce6bc8a2a2f19ff0e9b Mon Sep 17 00:00:00 2001 From: Victor Shyba Date: Tue, 2 Apr 2019 19:09:33 -0300 Subject: [PATCH] fix invalid signatures for missing certificates --- lbrynet/wallet/resolve.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lbrynet/wallet/resolve.py b/lbrynet/wallet/resolve.py index 0068a3931..f3c6795fa 100644 --- a/lbrynet/wallet/resolve.py +++ b/lbrynet/wallet/resolve.py @@ -324,10 +324,10 @@ def _verify_proof(name, claim_trie_root, result, height, depth, ledger): def validate_claim_signature_and_get_channel_name(claim_result, certificate_claim, ledger, claim_tx=None, cert_tx=None): + valid_signature = False if cert_tx and certificate_claim and claim_tx and claim_result: tx = Transaction(unhexlify(claim_tx)) cert_tx = Transaction(unhexlify(cert_tx)) - valid_signature = False try: valid_signature = tx.outputs[claim_result['nout']].is_signed_by( cert_tx.outputs[certificate_claim['nout']], ledger @@ -337,7 +337,7 @@ def validate_claim_signature_and_get_channel_name(claim_result, certificate_clai if not valid_signature: log.warning("lbry://%s#%s has an invalid signature", claim_result['name'], claim_result['claim_id']) - return valid_signature + return valid_signature # TODO: The following came from code handling lbryum results. Now that it's all in one place a refactor should unify it.