Skip to content

Fix: update with latest change for overflow issues #143

Open
ianlapham wants to merge 3 commits into
mainfrom
ian/overflow-fix
Open

Fix: update with latest change for overflow issues #143
ianlapham wants to merge 3 commits into
mainfrom
ian/overflow-fix

Conversation

@ianlapham

Copy link
Copy Markdown
Contributor

No description provided.

@cozakocur

cozakocur commented Dec 14, 2023

Copy link
Copy Markdown

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
Copy Markdown

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
Copy Markdown

@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