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

Solana offchain plugin #15974

Draft
wants to merge 16 commits into
base: develop
Choose a base branch
from
Draft

Solana offchain plugin #15974

wants to merge 16 commits into from

Conversation

huangzhen1997
Copy link
Contributor

No description provided.

* add solana support to contract transmitter

* add changeset

* one more place

* address unsupported chain family

* encode offramp addr

* fix breaking change

* rename

* test

* revert
* no-op for solana estimate providers

* add changeset

* token data noop

* noop for solana rmn

* fix lint
…reader-config-to-plugin-creator

Add Solana chain reader config to plugin creator
Copy link
Contributor

AER Report: CI Core

aer_workflow , commit , Clean Go Tidy & Generate , Scheduled Run Frequency , Detect Changes , GolangCI Lint (core/scripts) , test-scripts , Core Tests (go_core_tests) , GolangCI Lint (.) , Core Tests (go_core_tests_integration) , GolangCI Lint (integration-tests) , Core Tests (go_core_ccip_deployment_tests) , GolangCI Lint (deployment) , Core Tests (go_core_fuzz) , Core Tests (go_core_race_tests) , lint , SonarQube Scan

1. Runtime error: invalid memory address or nil pointer dereference: go_core_tests

Source of Error:
Run tests	2025-01-21T21:26:52.6925939Z panic: runtime error: invalid memory address or nil pointer dereference
Run tests	2025-01-21T21:26:52.6926624Z [signal SIGSEGV: segmentation violation code=0x1 addr=0x20 pc=0xff30b5]
Run tests	2025-01-21T21:26:52.6926987Z 
Run tests	2025-01-21T21:26:52.6927189Z goroutine 231 [running]:
Run tests	2025-01-21T21:26:52.6927928Z github.com/smartcontractkit/chainlink/v2/core/chains/evm/gas.(*FeeHistoryEstimator).RefreshGasPrice(0xc000764280)
Run tests	2025-01-21T21:26:52.6929014Z 	/home/runner/work/chainlink/chainlink/core/chains/evm/gas/fee_history_estimator.go:182 +0x75
Run tests	2025-01-21T21:26:52.6929943Z github.com/smartcontractkit/chainlink/v2/core/chains/evm/gas.(*FeeHistoryEstimator).run(0xc000764280)
Run tests	2025-01-21T21:26:52.6930936Z 	/home/runner/work/chainlink/chainlink/core/chains/evm/gas/fee_history_estimator.go:155 +0x18f
Run tests	2025-01-21T21:26:52.6932150Z created by github.com/smartcontractkit/chainlink/v2/core/chains/evm/gas_test.TestFeeHistoryEstimatorLifecycle.func4.(*FeeHistoryEstimator).Start.1 in goroutine 230
Run tests	2025-01-21T21:26:52.6933346Z 	/home/runner/work/chainlink/chainlink/core/chains/evm/gas/fee_history_estimator.go:122 +0x165
Run tests	2025-01-21T21:26:52.6934162Z FAIL	github.com/smartcontractkit/chainlink/v2/core/chains/evm/gas	0.056s

Why: The error is caused by a nil pointer dereference in the RefreshGasPrice method of the FeeHistoryEstimator struct. This typically happens when an object or variable that is expected to be initialized is actually nil.

Suggested fix: Ensure that all pointers and objects are properly initialized before they are used. Add nil checks before dereferencing pointers to prevent runtime errors.

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.

1 participant