Add report vs message source chain selector check

smartcontractkit / ccip

Cross Chain Interoperability Protocol

Other

119 stars 47 forks source link

Add report vs message source chain selector check #1413

Closed RayXpub closed 1 month ago

RayXpub commented 1 month ago

Motivation

The goal of this PR is to add an extra check in the execute function to enforce that the report source chain selector matches its messages source chain selector. Extra gas cost is about 50 gas per message.

Solution

Revert when report.sourceChainSelector != (message.header.sourceChainSelector

github-actions[bot] commented 1 month ago

LCOV of commit `77db94d` during Solidity Foundry #8024

Summary coverage rate:
  lines......: 97.6% (2154 of 2206 lines)
  functions..: 94.9% (408 of 430 functions)
  branches...: 93.4% (512 of 548 branches)

Files changed coverage rate: n/a

cl-sonarqube-production[bot] commented 1 month ago

Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube