fulfillRandomWords msg.data length validation

[kidambisrinivas]

• upper bound to the length of the msg.data that can be passed to the fulfillRandomWords function.

Testing

$ forge test -vvv --match-path test/v0.8/foundry/vrf/VRFV2Plus.t.sol
[⠒] Compiling...
[⠢] Compiling 7 files with 0.8.6
[⠰] Solc 0.8.6 finished in 5.23s
Compiler run successful with warnings:
Warning (2519): This declaration shadows an existing declaration.
   --> test/v0.8/foundry/vrf/VRFV2PlusSubscriptionAPI.t.sol:568:7:
    |
568 |       address consumer = makeAddr(Strings.toString(i));
    |       ^^^^^^^^^^^^^^^^
Note: The shadowed declaration is here:
   --> test/v0.8/foundry/vrf/VRFV2PlusSubscriptionAPI.t.sol:575:5:
    |
575 |     address consumer = makeAddr("consumer");
    |     ^^^^^^^^^^^^^^^^
Running 11 tests for test/v0.8/foundry/vrf/VRFV2Plus.t.sol:VRFV2Plus
[PASS] testCancelSubWithNoLink() (gas: 159020)
Logs:
  changePrank is deprecated. Please use vm.startPrank instead.

[PASS] testCreateSubscription() (gas: 180963)
Logs:
  changePrank is deprecated. Please use vm.startPrank instead.

[PASS] testDeregisterProvingKey() (gas: 86472)
Logs:
  changePrank is deprecated. Please use vm.startPrank instead.

[PASS] testGetActiveSubscriptionIds() (gas: 3427509)
Logs:
  changePrank is deprecated. Please use vm.startPrank instead.

[PASS] testRegisterProvingKey() (gas: 101426)
Logs:
  changePrank is deprecated. Please use vm.startPrank instead.

[PASS] testRequestAndFulfillRandomWordsLINK() (gas: 765378)
Logs:
  changePrank is deprecated. Please use vm.startPrank instead.

[PASS] testRequestAndFulfillRandomWordsNative() (gas: 709857)
Logs:
  changePrank is deprecated. Please use vm.startPrank instead.

[PASS] testRequestAndFulfillRandomWords_NetworkGasPriceExceedsGasLane() (gas: 613829)
Logs:
  changePrank is deprecated. Please use vm.startPrank instead.

[PASS] testRequestAndFulfillRandomWords_OnlyPremium_LinkPayment() (gas: 765812)
Logs:
  changePrank is deprecated. Please use vm.startPrank instead.

[PASS] testRequestAndFulfillRandomWords_OnlyPremium_NativePayment() (gas: 710331)
Logs:
  changePrank is deprecated. Please use vm.startPrank instead.

[PASS] testSetConfig() (gas: 68113)
Logs:
  changePrank is deprecated. Please use vm.startPrank instead.

Test result: ok. 11 passed; 0 failed; 0 skipped; finished in 8.48ms

Ran 1 test suites: 11 tests passed, 0 failed, 0 skipped (11 total tests)

[github-actions]

Go solidity wrappers are out-of-date, regenerate them via the make wrappers-all command

[ibrajer]

LGTM

[cl-sonarqube-production]

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