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

[MINER] Nonce err. Is 12, expected 7 #476

Closed
jorisbontje opened this issue Mar 13, 2015 · 6 comments
Closed

[MINER] Nonce err. Is 12, expected 7 #476

jorisbontje opened this issue Mar 13, 2015 · 6 comments
Milestone

Comments

@jorisbontje
Copy link

with develop 80592f2

creating trivial contract https://github.com/jorisbontje/sleth/blob/master/contracts/simple.se with hex code 336000556001600155604f8060146000396063567c01000000000000000000000000000000000000000000000000000000006000350463119fbbd48114156036576001600154016001555b63ea66cf6d811415604d5760015460405260206040f35b505b6000f3

via rpc:

{'params': [{'gas': '0x186a0', 'from': '0x48574ed8323453c2c5deaebc69eb068c1cc516fa', 'data': '0x336000556001600155604f8060146000396063567c01000000000000000000000000000000000000000000000000000000006000350463119fbbd48114156036576001600154016001555b63ea66cf6d811415604d5760015460405260206040f35b505b6000f3', 'value': '0x0', 'gasPrice': '0x9184e72a000'}], 'jsonrpc': '2.0', 'id': '64014be1-3810-4964-904b-ad14b86fd945', 'method': 'eth_sendTransaction'}

result:

{u'jsonrpc': u'2.0', u'id': u'64014be1-3810-4964-904b-ad14b86fd945', u'result': u'0x320fdb74aa01379b2e8189b2daf889a45f6dc306'}

ethereum started with:

$ ethereum -mine=true -rpc=true -rpcport=8080 --unlock=48574ed8323453c2c5deaebc69eb068c1cc516fa:bla

the ethereum logs:

2015/03/13 13:10:30 [XETH] Contract addr 320fdb74aa01379b2e8189b2daf889a45f6dc306
2015/03/13 13:10:32 [MINER] Hash rate: 88 Khash
2015/03/13 13:10:33 [CHAIN] processed block #1539 (d5aea773...)
2015/03/13 13:10:33 [MINER] committing new work with 1 txs
2015/03/13 13:10:33 [MINER] Nonce err. Is 12, expected 7
2015/03/13 13:10:33 [MINER] (re)started agent[0]. mining...
2015/03/13 13:10:34 [MINER] Hash rate: 91 Khash
2015/03/13 13:10:35 [CHAIN] processed block #1540 (eaecc1ad...)

Why does this cause a Nonce err? My contract doesn't seem to be mined (getData doesnt return anything), but otherwise I get no error messages.

@obscuren
Copy link
Contributor

Could you update to f1fcda4 and see if the error persists. Nonce errors may still be caused when doing something like:

send TX1 with too little gas = GAS_FAILURE (nonce 0)
send TX2 with enough gas = OK (nonce 1)

Both of these transactions will fail because: TX1 fails because of GAS_FAILURE and gets removed. TX2 fails because it actually should have gotten N = 0.

The following test should pass

send TX1 with too little gas = GAS_FAILURE (nonce 0)
send TX2 with enough gas = OK (nonce 1)
MINE TX1 & 2
send TX3 with enough gas = OK (nonce 0)

When TX3 is send it knows of the failure of TX1 and removes both 1 & 2.

#477

@obscuren obscuren added this to the Frontier milestone Mar 13, 2015
@jorisbontje
Copy link
Author

2015/03/13 17:07:37 [MINER] Nonce err. Is 18446744073709551615, expected 1

@obscuren
Copy link
Contributor

Fixed

@jorisbontje
Copy link
Author

Ok, not getting this error anymore; however my transaction still isn't mining & I am unable to really test this due to

[CHAIN] Block timestamp equal or less than previous block (1426267746 - 1426267746)

@obscuren
Copy link
Contributor

That error doesn't remove the transaction. If a transaction is included in a block that throws an error like the one above it will also be included in the next valid one.

@obscuren
Copy link
Contributor

Please re-open if the issues persists. I haven't been able to find any nonce errors anymore

ngtuna added a commit to ngtuna/tomochain that referenced this issue Mar 29, 2019
tony-ricciardi pushed a commit to tony-ricciardi/go-ethereum that referenced this issue Jan 20, 2022
* add input length checks

* check exact input length. add a new error for input length. check input in a few more places

* add tests that verify the input-length checks for contracts that don't require an evm instance

* fix formatting

* add comments to explain input length checks

* run the formatter

* e2e transfer test was failing because it passes in a transaction options object, making the input larger than 96 bytes

* e2e tests have revealed that our precompiled contracts need to be tolerant of inputs that are longer than the bytes that are actually read
tony-ricciardi pushed a commit to tony-ricciardi/go-ethereum that referenced this issue Jan 20, 2022
* Log on ValidatorElections

* merge master (ethereum#496)

* Check message address against signature (ethereum#477)

* Check signing validator's address matches msg address

* Add comments about use of sig data in tests

* Try fix Circle build test failures

* Try fix Circle build test failures, take 2

* tx price heap fix (ethereum#471)

* contract_comm/currency/currency.go

* fixed the txn price-sorted min-heap

* merge master (ethereum#490)

* Add precompiles to access validator set (ethereum#441)

* set max gas to double of the charged gas for the 'intrinsic' smart contract calls (ethereum#472)

* set max gas to double of the charged gas for the 'intrinsic' evm operations

* addressed PR comments

* addressed pr comment

* Adds Prepared Certificates to ensure Istanbul liveness (ethereum#366)

* Check message address against signature (ethereum#477)

* Check signing validator's address matches msg address

* Add comments about use of sig data in tests

* Try fix Circle build test failures

* Try fix Circle build test failures, take 2

* added new option --use-in-memory-discovery-table (ethereum#479)

* added new option --use-in-memory-discovery-table

* merge master (ethereum#489)

* Adds Prepared Certificates to ensure Istanbul liveness (ethereum#366)

* Check message address against signature (ethereum#477)

* Check signing validator's address matches msg address

* Add comments about use of sig data in tests

* Try fix Circle build test failures

* Try fix Circle build test failures, take 2

* Allow v4/v5 on a bootnode simultaneously, allow mobile to use discv5 (ethereum#454)

* changes for isolating celo networks

* changes to get unit tests working

* changes to add salt in the discovery packets

* removed checking for the ip address when handling a reply

* added ping-ip-from-packet option to bootnode

* for handling expected replies, don't filter on expected sender ip address if --pingIPFromPacket is used

* Add v4 flag

* Add unhandled and quicken docker builds

* Add salt & logs

* Add v4 flag

* Add PeerDiscovery to mobile node config

* Remove logs

* Remove hardcoded bootnodes

* Add salt & turn on discv5

* Delete hardcoded eth bootnodes

* Make Discoveryv5 configurable

* Lint

* Add comment to bootnode v4/v5 handling

* Change PeerDiscovery -> NoDiscovery

* Remove mobile geth no discovery

* Reduce istanbul default timeout, cap exp backoff (ethereum#475)

* Reduce istanbul default timeout, cap exp backoff

* Ensure round 0 timeout factors in block period

* Sanitize logs (ethereum#495)

* Change registry lookup and infrastructure lookup error logs to debug level

* Sanitize logs regarding registry deployment

* Change empty abi logging and comment

* Lower log level from error to warning for potentially outdated istanbul messages

* Change back to an error message

* Add input length checks for precompiled contracts (ethereum#476)

* add input length checks

* check exact input length. add a new error for input length. check input in a few more places

* add tests that verify the input-length checks for contracts that don't require an evm instance

* fix formatting

* add comments to explain input length checks

* run the formatter

* e2e transfer test was failing because it passes in a transaction options object, making the input larger than 96 bytes

* e2e tests have revealed that our precompiled contracts need to be tolerant of inputs that are longer than the bytes that are actually read
tanishqjasoria pushed a commit to tanishqjasoria/go-ethereum that referenced this issue Oct 31, 2023
)

* fix: exclude L1 message from block payload size validation

* fix the bug when calculating l2TxCount. (ethereum#479)

* fix bug when calculate l2 tx count

* Update version

* bump version

---------

Co-authored-by: maskpp <maskpp266@gmail.com>
tanishqjasoria pushed a commit to tanishqjasoria/go-ethereum that referenced this issue Oct 31, 2023
* feat: store skipped txs in local db

* bump version

* bump version

* fix test

* include L2 txs in skipped index

* goimports

* rename more

* bump version

* fix missed renames

* fix the bug when calculating l2TxCount. (ethereum#479)

* fix bug when calculate l2 tx count

* Update version

* fix: exclude L1 message from block payload size validation (ethereum#476)

* fix: exclude L1 message from block payload size validation

* fix the bug when calculating l2TxCount. (ethereum#479)

* fix bug when calculate l2 tx count

* Update version

* bump version

---------

Co-authored-by: maskpp <maskpp266@gmail.com>

* fix: update row estimation with scroll-prover `v0.7.2` (ethereum#475)

* Fix row estimation.

* Update libzkp.

* more

* prepare

* finish

* upgrade

* bump version

* fix tests

* Update to scroll-prover `v0.7.2`.

* fix tests

* Update miner/worker.go

Co-authored-by: Péter Garamvölgyi <peter@scroll.io>

* Update miner/worker.go

Co-authored-by: Péter Garamvölgyi <peter@scroll.io>

* Reset ccc when skips first tx.

* do not unnecessarily skip L1 message

* fix ccc reset and improve code readability

* seal block on circuitcapacitychecker.ErrUnknown

---------

Co-authored-by: HAOYUatHZ <haoyu@protonmail.com>
Co-authored-by: Péter Garamvölgyi <peter@scroll.io>

* refactor: simplify ccc revert to snapshot (ethereum#480)

* simplify ccc revert to snapshot

* Update version.go

---------

Co-authored-by: HAOYUatHZ <37070449+HAOYUatHZ@users.noreply.github.com>

* feat: use --gcmode=archive and --cache.noprefetch=true by default (ethereum#482)

* feat: use --gcmode=archive and --cache.noprefetch=true by default

* refuse to start with invalid config

* lint

* lint

* feat(sdk): support compressed response (ethereum#469)

* enable use compression algorithm

* fix ci

* Just enable decode compressed content at ethclient

* fix comments

---------

Co-authored-by: Haichen Shen <shenhaichen@gmail.com>

* fix: disable pruning and prefetch if not flags are provided (ethereum#483)

Co-authored-by: colin <102356659+colinlyguo@users.noreply.github.com>

* fix: update libzkp to use scroll-prover `v0.7.5` (ethereum#484)

* Update libzkp to use scroll-prover `v0.7.5`.

* Update version.

* address comments

* bump version

* nit

---------

Co-authored-by: maskpp <maskpp266@gmail.com>
Co-authored-by: Steven <asongala@163.com>
Co-authored-by: HAOYUatHZ <haoyu@protonmail.com>
Co-authored-by: iczc <iczcalan@gmail.com>
Co-authored-by: HAOYUatHZ <37070449+HAOYUatHZ@users.noreply.github.com>
Co-authored-by: Haichen Shen <shenhaichen@gmail.com>
Co-authored-by: colin <102356659+colinlyguo@users.noreply.github.com>
tanishqjasoria pushed a commit to tanishqjasoria/go-ethereum that referenced this issue Sep 3, 2024
Signed-off-by: Ignacio Hagopian <jsign.uy@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants