Skip to content

Commit 7511559

Browse files
authored
fix: failing certora-ci (#410)
* fix: try installing solc-select * fix: addShares selector
1 parent 5b15e1e commit 7511559

File tree

2 files changed

+5
-6
lines changed

2 files changed

+5
-6
lines changed

.github/workflows/certora-prover.yml

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -46,9 +46,8 @@ jobs:
4646
run: pip install certora-cli
4747
- name: Install solc
4848
run: |
49-
wget https://github.com/ethereum/solidity/releases/download/v0.8.12/solc-static-linux
50-
sudo mv solc-static-linux /usr/local/bin/solc
51-
chmod +x /usr/local/bin/solc
49+
pip install solc-select
50+
solc-select use 0.8.12 --always-install
5251
- name: Verify rule ${{ matrix.params }}
5352
run: |
5453
bash ${{ matrix.params }}

certora/specs/core/StrategyManager.spec

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ methods {
2020
// external calls to StrategyManager
2121
function _.getDeposits(address) external => DISPATCHER(true);
2222
function _.slasher() external => DISPATCHER(true);
23-
function _.addShares(address,address,uint256) external => DISPATCHER(true);
23+
function _.addShares(address,address,address,uint256) external => DISPATCHER(true);
2424
function _.removeShares(address,address,uint256) external => DISPATCHER(true);
2525
function _.withdrawSharesAsTokens(address, address, uint256, address) external => DISPATCHER(true);
2626

@@ -97,7 +97,7 @@ definition methodCanIncreaseShares(method f) returns bool =
9797
f.selector == sig:depositIntoStrategy(address,address,uint256).selector
9898
|| f.selector == sig:depositIntoStrategyWithSignature(address,address,uint256,address,uint256,bytes).selector
9999
|| f.selector == sig:withdrawSharesAsTokens(address,address,uint256,address).selector
100-
|| f.selector == sig:addShares(address,address,uint256).selector;
100+
|| f.selector == sig:addShares(address,address,address,uint256).selector;
101101

102102
/**
103103
* a staker's amount of shares in a strategy (i.e. `stakerStrategyShares[staker][strategy]`) should only decrease when
@@ -129,7 +129,7 @@ rule newSharesIncreaseTotalShares(address strategy) {
129129
uint256 stakerStrategySharesBefore = get_stakerStrategyShares(e.msg.sender, strategy);
130130
uint256 totalSharesBefore = totalShares(strategy);
131131
if (
132-
f.selector == sig:addShares(address, address, uint256).selector
132+
f.selector == sig:addShares(address, address, address, uint256).selector
133133
|| f.selector == sig:removeShares(address, address, uint256).selector
134134
) {
135135
uint256 totalSharesAfter = totalShares(strategy);

0 commit comments

Comments
 (0)