Skip to content

Commit

Permalink
fix: nil pointer exception (#451)
Browse files Browse the repository at this point in the history
* fix: nil pointer exception

* chore: update readme
  • Loading branch information
maharifu authored Oct 25, 2024
1 parent 2a1843c commit 023d93a
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 4 deletions.
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ This repo does not accept issues. Please use <https://github.com/palomachain/pal
### To get the latest prebuilt `pigeon` binary

```shell
wget -O - https://github.com/palomachain/pigeon/releases/download/v2.3.2/pigeon_Linux_x86_64.tar.gz | \
wget -O - https://github.com/palomachain/pigeon/releases/download/v2.3.3/pigeon_Linux_x86_64.tar.gz | \
sudo tar -C /usr/local/bin -xvzf - pigeon
sudo chmod +x /usr/local/bin/pigeon

Expand All @@ -94,7 +94,7 @@ mkdir ~/.pigeon
```shell
git clone https://github.com/palomachain/pigeon.git
cd pigeon
git checkout v2.3.2
git checkout v2.3.3
make build
sudo mv ./build/pigeon /usr/local/bin/pigeon

Expand Down
6 changes: 4 additions & 2 deletions chain/evm/compass.go
Original file line number Diff line number Diff line change
Expand Up @@ -1517,8 +1517,10 @@ func (t compass) skywayEstimateBatches(ctx context.Context, batches []chain.Skyw
return nil, fmt.Errorf("failed to estimate batch: %w", err)
}

logger.WithField("estimate", tx.Gas()).Debug("Estimated gas for batch")
estimates = append(estimates, tx.Gas())
if tx != nil {
logger.WithField("estimate", tx.Gas()).Debug("Estimated gas for batch")
estimates = append(estimates, tx.Gas())
}
}

return estimates, nil
Expand Down

0 comments on commit 023d93a

Please sign in to comment.