v0.19.0: Problem with variable m_absurd_fee #17464

issue fujicoin opened this issue on November 13, 2019
  1. fujicoin commented at 1:26 PM on November 13, 2019: none

    The variable m_absurd_fee has only two lines written in the entire source code. It is the following two lines. validation.cpp: L451 const CAmount & m_absurd_fee; const CAmount & nAbsurdFee = args.m_absurd_fee;

    As a result, isn't nAbsurdFee indefinite? This causes a problem with the following line: validation.cpp: L717 if (nAbsurdFee && nFees> nAbsurdFee) return state.Invalid (ValidationInvalidReason :: TX_NOT_STANDARD, false, REJECT_HIGHFEE, "absurdly-high-fee", strprintf ("% d>% d", nFees, nAbsurdFee));

    As a result, may an error occur with the "sendrawtransaction" command?

  2. fujicoin added the label Bug on Nov 13, 2019
  3. MarcoFalke commented at 1:33 PM on November 13, 2019: member

    No. The const makes the compiler enforce that the value is set on initialization.

  4. MarcoFalke closed this on Nov 13, 2019

  5. MarcoFalke locked this on Dec 16, 2021
Contributors
Labels

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-29 03:15 UTC

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