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

850 Million transaction total consensus on the Blockchain #35

Open
EmberCoin opened this issue Oct 25, 2017 · 5 comments
Open

850 Million transaction total consensus on the Blockchain #35

EmberCoin opened this issue Oct 25, 2017 · 5 comments
Milestone

Comments

@EmberCoin
Copy link
Owner

[5:19 AM] Kris BTC: @AndrewBC @everyone found a new bug I think, you cant have more then ~850mil in change it will cause a runtime error and if your weight goes past ~850mil when it tries to stake it causes a runtime error and both force close

@EmberCoin EmberCoin changed the title (change) address triggers MSVC runtime errors under special conditions on 1.1.4 and maybe below. (change) address triggers MSVC runtime errors and closures under special conditions on 1.1.4 and maybe below. Oct 25, 2017
@KrisBTC
Copy link

KrisBTC commented Oct 26, 2017

On here I said "if your weight goes past ~850mil when it tries to stake it causes a runtime error and both force close"
Correction
if your input+reward goes past ~850mil when it tries to stake it causes a runtime error and both issues cause a force close.

@EmberCoin
Copy link
Owner Author

EmberCoin commented Oct 26, 2017

if (txout.nValue > MAX_MONEY)
    return DoS(100, error("CTransaction::CheckTransaction() : txout.nValue too high"));

and MAX_MONEY is:

/** No amount larger than this (in satoshi) is valid */
static const int64_t MAX_MONEY = 850000000 * COIN; // 850 Million

There's already a check for negative, and it resides in an int64_t, so why is there a random value check when you already are using a well defined value...

@EmberCoin EmberCoin changed the title (change) address triggers MSVC runtime errors and closures under special conditions on 1.1.4 and maybe below. (change) address triggers MSVC runtime errors and closures under special conditions on all versions including old Oct 26, 2017
@EmberCoin
Copy link
Owner Author

Here is an interesting discussion which talks specifically about this variable: https://github.com/bitcoin/bitcoin/pull/6198/files

Credit to @ZbufferRing in Discord

@EmberCoin
Copy link
Owner Author

Since this requires forking changes we will not be fixing this for quite a while.

@EmberCoin EmberCoin removed the wontfix label Oct 29, 2017
@EmberCoin EmberCoin added this to the v1.2.x milestone Oct 29, 2017
@EmberCoin EmberCoin reopened this Oct 30, 2017
@EmberCoin EmberCoin changed the title (change) address triggers MSVC runtime errors and closures under special conditions on all versions including old 850 Million transaction total consensus on the Blockchain Nov 17, 2017
@jmsanta
Copy link

jmsanta commented Dec 12, 2017

please, fix, make changes to put ember to 1 satoshi again before coin swap (2018), and you will win millions.. :) and hold some!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants