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

Fix: update with latest change for overflow issues #143

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

ianlapham
Copy link
Member

No description provided.

@cozakocur
Copy link

cozakocur commented Dec 14, 2023

Again indexing error with overflow problems:

Subgraph instance failed to run: transaction 4b6ad5c5b68b0963e11ff5c1525ae026fbcc59e190385dc0adc4d1b252fb7e8a: Mapping aborted at ~lib/@graphprotocol/graph-ts/index.ts, line 380, column 8, with message: overflow converting 0x0000000000000000000000000000000000000000000000000000000067e53c31 to i32 wasm backtrace: 0: 0x1fd2 - <unknown>!~lib/@graphprotocol/graph-ts/index/ByteArray#toI32 1: 0x20e0 - <unknown>!~lib/@graphprotocol/graph-ts/chain/ethereum/ethereum.Value#toI32 2: 0x2113 - <unknown>!src/types/Factory/ERC20/ERC20#try_decimals 3: 0x215b - <unknown>!src/mappings/helpers/fetchTokenDecimals 4: 0x24fe - <unknown>!src/mappings/factory/handleNewPair in handler handleNewPair at block #18746374 (9b095ecb52fd2d2e8a549aac4e639dbb961ecc6b1e91e6990f3d5ba93e3c6658), code: SubgraphSyncingFailure

we need permament fix @ianlapham :(

Here issue:
#144

@0xSamWitch
Copy link

Looks like there is a heap access out of bounds error now

Subgraph instance failed to run: transaction ba8ed60fac7531a18f886cae5b81611e6cc833d3d57989076a9dcbf4b309374b: Heap access out of bounds. Offset: 42 Size: 6291556 wasm backtrace: 0: 0x26c1 - <unknown>!src/types/schema/Token.load 1: 0x34c6 - <unknown>!src/mappings/core/handleSync in handler `handleSync` at block #18746371 (a5b160ee3e8c76765dfc6d1816d55dd2e6ca73577658af4c585e8ce7cb3bd518), code: SubgraphSyncingFailure

17/12/2023, 14:06:08

@balthazar
Copy link

@ianlapham Your latest commit should fix the heap access right? And is this graph deployed against your branch? If that's the case and since it's the only one currently working, this PR should probably merged

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

Successfully merging this pull request may close these issues.

4 participants