signrawtransaction returns "complete" : false sometimes when it really is complete #2265

issue gmaxwell opened this issue on February 2, 2013
  1. gmaxwell commented at 4:05 AM on February 2, 2013: contributor

    When doing offline signing of unknown txins with the scriptpubkey provided on the command line signrawtransaction returns complete false when the transaction is really completed.

  2. gavinandresen commented at 6:32 PM on February 3, 2013: contributor

    If the txin is unknown, then how does signrawtransaction know that the signature for that scriptSig is correct?

    I suppose it could assume that if there is a signature then it is correct... but that seems dangerous.

  3. luke-jr commented at 6:58 PM on February 3, 2013: member

    How about omitting the "complete" flag entirely, when it's not known? It could be supplemented with a count of signatures (and total signatures in transaction) if there's no other way to tell that info already.

  4. gmaxwell commented at 4:33 AM on February 4, 2013: contributor

    An 'unknown' state of any kind would be fine— false is confusing though.

  5. laanwj removed the label Priority Low on Dec 6, 2017
  6. kouloumos commented at 2:47 PM on August 2, 2022: contributor

    This issue is reported for txins unknown to bitcoin, i.e not in the mempool or blockchain, thus the scriptPubKey has to be provided explicitly.

    The existing functional tests at wallet_signrawtransactionwithwallet.py, rpc_signrawtransactionwithkey.py are already checking against this behavior and what is reported seems to not be an issue anymore or it's reproduction is more complicated. I've also try to reproduce this manually on regtest but I still was not able to do so.

    Note:

    • The referenced signrawtransaction RPC was fully removed in v0.18. and since replaced with signrawtransactionwithkey and signrawtransactionwithwallet.
    • Functional tests for this RPC were introduced much later with #5937
  7. MarcoFalke commented at 3:45 PM on August 2, 2022: member

    what is reported seems to not be an issue anymore or it's reproduction is more complicated.

    Ok, I guess if it is not yet fixed, it can be reopened the next time someone has steps to reproduce.

  8. MarcoFalke closed this on Aug 2, 2022

  9. bitcoin locked this on Aug 2, 2023

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-18 21:16 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me