bumpfee fails with "Transaction does not have a change output" when setting label for newly generated raw change address #14773

issue equalgeek opened this issue on November 21, 2018
  1. equalgeek commented at 3:07 AM on November 21, 2018: none

    Issue: The issue seems to be when you generate a new change address via "getrawchangeaddress" and setting a new label for it. When creating a new raw transaction, setting it to be replaceable, and using the new change address for one of the outputs in the raw tx. bumpfee throws an error "Transaction does not have a change output". If you do not perform the setlabel on the newly generated raw change address then bumpfee performs the "bump" correctly.

    Setup: v 0.17.0.1

    Behavior expected: bumpfee to notice any address in wallet as a potential "change" address or at least allowing a label to be set for being able to track specific labels being used for change addresses.

    Steps to reproduce:

    1. Generate new raw change address via "getnewchangeaddress"
    2. Set label for newly generated raw change address to whatever you want.
    3. Create raw transaction as replaceable with newly generated raw change address as one of the outputs.
    4. Sign & Send tx
    5. Perform bumpfee on raw txid
    6. Error "Transaction does not have a change output"

    Thanks to all the devs btw, just wanted to express my appreciation for all you do

  2. meshcollider added the label RPC/REST/ZMQ on Nov 22, 2018
  3. meshcollider commented at 5:57 AM on November 22, 2018: contributor

    If you do not perform the setlabel on the newly generated raw change address then bumpfee performs the "bump" correctly.

    As you correctly observe, if you add a label to an address, it is no longer considered change (change addresses are defined as those which aren't in the address book, adding a label will add it to the address book). Internal addresses can not have labels.

  4. promag commented at 9:53 AM on November 22, 2018: member

    Related discussion in #14724.

  5. MarcoFalke commented at 3:10 PM on April 26, 2020: member

    Let's continue discussion in #14724

  6. MarcoFalke closed this on Apr 26, 2020

  7. meshcollider commented at 10:05 PM on April 26, 2020: contributor

    #14724 is related but it is certainly a different issue, I don't think this has been resolved

  8. MarcoFalke reopened this on Apr 26, 2020

  9. glozow added the label Wallet on Aug 10, 2022
  10. glozow commented at 2:41 PM on August 10, 2022: member

    no longer a problem since #15557 AFAIU, but feel free to reopen if not

  11. glozow closed this on Aug 10, 2022

  12. bitcoin locked this on Aug 10, 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-13 15:15 UTC

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