Skip to content
This repository has been archived by the owner on Jan 5, 2019. It is now read-only.

manyFunctions100 vmtest failing #217

Open
lrettig opened this issue Apr 1, 2018 · 3 comments
Open

manyFunctions100 vmtest failing #217

lrettig opened this issue Apr 1, 2018 · 3 comments

Comments

@lrettig
Copy link
Member

lrettig commented Apr 1, 2018

This one test is failing in the current master branch (e050d24):

evm2wasm/ master> npm run vmTests
> evm2wasm@0.0.0 vmTests /Users/lanerettig/wip/eth/evm2wasm
> node --expose-wasm --harmony ./tests/runVmTests.js
TAP version 13
# VMTESTS
...
# manyFunctions100 manyFunctions100
not ok 1425 Error: Command failed: /Users/lanerettig/wip/eth/evm2wasm/tools/wabt/bin/wat2wasm /Users/lanerettig/wip/eth/evm2wasm/tmp/manyFunctions100.wast -o /Users/lanerettig/wip/eth/evm2wasm/tmp/manyFunctions100.wasm
  ---
    operator: fail
    at: runner.catch.err (/Users/lanerettig/wip/eth/evm2wasm/tests/runVmTests.js:171:9)
    stack: |-
      Error: Error: Command failed: /Users/lanerettig/wip/eth/evm2wasm/tools/wabt/bin/wat2wasm /Users/lanerettig/wip/eth/evm2wasm/tmp/manyFunctions100.wast -o /Users/lanerettig/wip/eth/evm2wasm/tmp/manyFunctions100.wasm

          at Test.assert [as _assert] (/Users/lanerettig/wip/eth/evm2wasm/node_modules/tape/lib/test.js:225:54)
          at Test.bound [as _assert] (/Users/lanerettig/wip/eth/evm2wasm/node_modules/tape/lib/test.js:77:32)
          at Test.fail (/Users/lanerettig/wip/eth/evm2wasm/node_modules/tape/lib/test.js:318:10)
          at Test.bound [as fail] (/Users/lanerettig/wip/eth/evm2wasm/node_modules/tape/lib/test.js:77:32)
          at runner.catch.err (/Users/lanerettig/wip/eth/evm2wasm/tests/runVmTests.js:171:9)
          at <anonymous>
  ...

Attempting to run this command manually produces:

~/wip/eth/evm2wasm master> /Users/lanerettig/wip/eth/evm2wasm/tools/wabt/bin/wat2wasm /Users/lanerettig/wip/eth/evm2wasm/tmp/manyFunctions100.wast -o /Users/lanerettig/wip/eth/evm2wasm/tmp/manyFunctions100.wasm
[1]    48249 segmentation fault  /Users/lanerettig/wip/eth/evm2wasm/tools/wabt/bin/wat2wasm  -o
@lrettig lrettig changed the title manyFunctions100 test failing manyFunctions100 vmtest failing Apr 1, 2018
@lrettig
Copy link
Member Author

lrettig commented Apr 1, 2018

I tried this with the latest master version of wabt but it's still failing. The only lead I found is WebAssembly/wabt#685 which suggests that certain characters can cause a segfault.

@cdetrio
Copy link
Collaborator

cdetrio commented Apr 2, 2018

does wabt crash if using wabt.js rather than the wabt binary? see also #174

@lrettig
Copy link
Member Author

lrettig commented Apr 23, 2018

@cdetrio interestingly enough, this is what happens when I try to use wabt js (see https://github.com/lrettig/web-wabt-test):

RangeError: Maximum call stack size exceeded
    at pv (/Users/lanerettig/wip/eth/web-wabt-test/node_modules/wabt/index.js:11:100915)
    at $b (/Users/lanerettig/wip/eth/web-wabt-test/node_modules/wabt/index.js:5:292191)
    at sc (/Users/lanerettig/wip/eth/web-wabt-test/node_modules/wabt/index.js:7:45889)
    at pc (/Users/lanerettig/wip/eth/web-wabt-test/node_modules/wabt/index.js:7:13284)
    at Ec (/Users/lanerettig/wip/eth/web-wabt-test/node_modules/wabt/index.js:7:56896)
    at uc (/Users/lanerettig/wip/eth/web-wabt-test/node_modules/wabt/index.js:7:48024)
    at pc (/Users/lanerettig/wip/eth/web-wabt-test/node_modules/wabt/index.js:7:13450)
    at Ec (/Users/lanerettig/wip/eth/web-wabt-test/node_modules/wabt/index.js:7:56896)
    at uc (/Users/lanerettig/wip/eth/web-wabt-test/node_modules/wabt/index.js:7:48024)
    at pc (/Users/lanerettig/wip/eth/web-wabt-test/node_modules/wabt/index.js:7:13450)

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

No branches or pull requests

2 participants