Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Wasm: Test for reaching stack overflow #9042

Merged
merged 1 commit into from
Jul 5, 2018
Merged

Conversation

pepyakin
Copy link
Contributor

@pepyakin pepyakin commented Jul 4, 2018

With this test we can check what kind of stack overflow is happened: trap by resource exhaustion (non-deterministic, execution environment dependent) or unreachable trap, which is most likely generated by deterministic stack height metering.

See also openethereum/wasm-tests#50

@pepyakin pepyakin added the A0-pleasereview 🤓 Pull request needs code review. label Jul 4, 2018
@pepyakin pepyakin requested a review from NikVolf July 4, 2018 11:06
@parity-cla-bot
Copy link

It looks like this contributor signed our Contributor License Agreement. 👍

Many thanks,

Parity Technologies CLA Bot

@debris debris added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Jul 4, 2018
@5chdn 5chdn added M4-core ⛓ Core client code / Rust. M5-dependencies 🖇 Dependencies. labels Jul 5, 2018
@5chdn 5chdn added this to the 1.12 milestone Jul 5, 2018
@NikVolf NikVolf merged commit 4839294 into master Jul 5, 2018
@NikVolf NikVolf deleted the ser-recursive-test branch July 5, 2018 07:09
ordian added a commit to ordian/parity that referenced this pull request Jul 9, 2018
…rp_sync_on_light_client

* 'master' of https://github.com/paritytech/parity:
  Fixes for misbehavior reporting in AuthorityRound (openethereum#8998)
  A last bunch of txqueue performance optimizations (openethereum#9024)
  reduce number of constraints for triedb types (openethereum#9043)
  bump fs-swap to 0.2.3 so it is compatible with osx 10.11 again (openethereum#9050)
  Recursive test (openethereum#9042)
  Introduce more optional features in ethcore (openethereum#9020)
  Update ETSC bootnodes (openethereum#9038)
  Optimize pending transactions filter (openethereum#9026)
  eip160/eip161 spec: u64 -> BlockNumber (openethereum#9044)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well. M4-core ⛓ Core client code / Rust. M5-dependencies 🖇 Dependencies.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants