Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

DAWN-261 ⁃ Investigate and solve reported activity that causes EOSC to crash #876

Closed
blockone-syncclient opened this issue Dec 6, 2017 · 2 comments
Assignees
Milestone

Comments

@blockone-syncclient
Copy link

blockone-syncclient commented Dec 6, 2017

Need to investigate (and resolve if we need to) why this program crashes EOSD?

From Dan: We should integrate his example contract into our unit tests

https://github.com/michaeljyeates/eosshop/tree/eosccrash

ATC:

  • I've updated tests/eosd_run_test.sh to verify eosc does not crash in this situation.
  • Step to manually reproduce:
    • Use the simpledb contract as it exhibited the same crashing as the above github shop contract.
    • Set the contract without first creating the simpledb account. This is key as it is the error message from the account not being there that was causing eosc to crash.
    • programs/eosc/eosc set contract simpledb contracts/simpledb/simpledb.wast contracts/simpledb/simpledb.abi
@blockone-syncclient
Copy link
Author

➤ Kevin Heifner commented:

The core dump was in eosc when trying to localize an error message. The actual error it was trying to localize was that the account did not exist for the contract being pushed. Will be putting better error handling around the error handling code itself since it was the attempt to generate a better error message that was core dumping.

@blockone-syncclient blockone-syncclient changed the title STAT-261 ⁃ Investigate and solve reported activity that causes EOSD to crash STAT-261 ⁃ Investigate and solve reported activity that causes EOSC to crash Dec 6, 2017
heifner added a commit that referenced this issue Dec 7, 2017
heifner added a commit that referenced this issue Dec 7, 2017
heifner added a commit that referenced this issue Dec 7, 2017
@heifner heifner mentioned this issue Dec 7, 2017
@blockone-syncclient
Copy link
Author

➤ Bezwada Satyapravin commented:

Tested with 8th Dec 2017 tag release and it does not crash:

Output is as follows:

641648ms main.cpp:1166 main ] Failed with error: Assert Exception (10)
status_code == 200: Error code 500
: {"code":500,"message":"Internal Service Error","details":"unknown key"}

heifner added a commit that referenced this issue Jan 4, 2018
heifner added a commit that referenced this issue Jan 4, 2018
(cherry picked from commit f56139b)
heifner added a commit that referenced this issue Jan 4, 2018
heifner added a commit that referenced this issue Jan 4, 2018
(cherry picked from commit 86bffbf)
@thomasbcox thomasbcox added this to the Q1, 2018 milestone Jan 5, 2018
@blockone-syncclient blockone-syncclient changed the title STAT-261 ⁃ Investigate and solve reported activity that causes EOSC to crash DAWN-261 ⁃ Investigate and solve reported activity that causes EOSC to crash Jan 16, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants