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.

Sign up for GitHub

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: prevent nil memory access #130

Merged
merged 1 commit into from
Dec 9, 2024

prevent nil memory access by checking tx status before fetching receipt

58b076d
Select commit
Loading
Failed to load commit list.
Merged

fix: prevent nil memory access #130

prevent nil memory access by checking tx status before fetching receipt
58b076d
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Dec 9, 2024 in 1s

56.93% (+0.00%) compared to 55ab7cf

View this Pull Request on Codecov

56.93% (+0.00%) compared to 55ab7cf

Details

Codecov Report

Attention: Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Project coverage is 56.93%. Comparing base (55ab7cf) to head (58b076d).

Files with missing lines Patch % Lines
jsonrpc/backend/tx.go 0.00% 1 Missing and 1 partial ⚠️
Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #130   +/-   ##
=======================================
  Coverage   56.93%   56.93%           
=======================================
  Files         110      110           
  Lines       10063    10063           
=======================================
  Hits         5729     5729           
  Misses       3508     3508           
  Partials      826      826
Files with missing lines Coverage Δ
jsonrpc/backend/tx.go 53.98% <0.00%> (ø)
View more details on Codecov