EmberCoin / Ember

Official Ember Blockchain
http://embercoin.io/
MIT License
25 stars 31 forks source link

850 Million transaction total consensus on the Blockchain #35

Open EmberCoin opened 6 years ago

EmberCoin commented 6 years ago

[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

KrisBTC commented 6 years ago

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 commented 6 years ago
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 commented 6 years ago

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 commented 6 years ago

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

jmsanta commented 6 years ago

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