Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TX Fee can be set to anything by anyone #1715

Closed
iancoleman opened this issue Aug 24, 2012 · 7 comments
Closed

TX Fee can be set to anything by anyone #1715

iancoleman opened this issue Aug 24, 2012 · 7 comments
Labels

Comments

@iancoleman
Copy link

The transaction fee can be set to any value by any person. This may lead to the user sending a small amount of coins and accidentally paying a giant tx fee without ever realising it.

See https://github.com/thedawnrider/malicious-bitcoin-example for an example of how this is done.

Options for resolving this are outlined here
https://bitcointalk.org/index.php?topic=102606.msg1126479#msg1126479

@hllhll
Copy link

hllhll commented Mar 12, 2013

If this is a trojan horse of some kind, malware, or any kind of malicious EXE, What is the problem for the torjan to replace all the Bitcoin-QT software altogether, maybe even leaking out private keys...

@jonasschnelli
Copy link
Contributor

Should we display the txfee in the transcation-confirm-promt in bitcoin-qt?
Display the txfee only when it's higher than the tx amount?

Protection (some security mechanism) of the config/registry value would make no sense in my eyes.

@laanwj
Copy link
Member

laanwj commented May 3, 2013

Not sure, it would be nice to do so, but I don't think the actual fee is known yet at the time the confirmation dialog is shown (that's also why there is an separate askFee prompt).

@jonasschnelli
Copy link
Contributor

@laanwj could we not just show the "nTransactionFee" from walletmodel in the dialog according to mockups in #1714? Because we also do the exceeds calculations based on nTransactionFee,... so i would say it's also okay to place them there. The real txfee will might be different. Maybe we show " and ~0.0005 BTC to transaction fees"?

@laanwj
Copy link
Member

laanwj commented May 3, 2013

Nah, showing an approximate value could confuse people even more. I'm for either showing nothing or the actual fee that will be paid.

@jonasschnelli
Copy link
Contributor

@laanwj okay,... let me check if i can somehow calculate (use the current txfee calc code) the fee before the confirmation dialogue pops up.

@laanwj
Copy link
Member

laanwj commented Nov 7, 2013

This is fixed by #2651, closing

@laanwj laanwj closed this as completed Nov 7, 2013
suprnurd pushed a commit to chaincoin-legacy/chaincoin that referenced this issue Dec 5, 2017
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

4 participants