Code

Airdrop audit fixes#634

Code

Merged

kumaryash90 merged 12 commits intomainthirdweb-dev/contracts:mainfrom yash/airdrop-audit-fixesthirdweb-dev/contracts:yash/airdrop-audit-fixesCopy head branch name to clipboardApr 5, 2024

+485-183

Commits

Commits on Apr 1, 2024

• [Q-2] Make processed mapping public

  kumaryash90

  committed
  29f02cc

  Copy full SHA for 29f02cc

• [Q-3] Unused error definition

  kumaryash90

  committed
  6bb17f2

  Copy full SHA for 6bb17f2

• [Q-4] Use separate events for each airdrop type

  kumaryash90

  committed
  fb8595a

  Copy full SHA for fb8595a

• [Q-5] Use safeTransferETH instead of low-level call

  kumaryash90

  committed
  fbde069

  Copy full SHA for fbde069

• remove receive and withdraw

  kumaryash90

  committed
  3d418a9

  Copy full SHA for 3d418a9

• cleanup

  kumaryash90

  committed
  a92bbfb

  Copy full SHA for a92bbfb

• [Q-1] Missing natspec documentation

  kumaryash90

  committed
  a3f9f27

  Copy full SHA for a3f9f27

• [L-1] Airdropping tokens using push or signature-based methods can be griefed

  kumaryash90

  committed
  da646d2

  Copy full SHA for da646d2

• [G-1] Use verifyCalldata to verify Merkle tree

  kumaryash90

  committed
  7d23411

  Copy full SHA for 7d23411

Commits on Apr 3, 2024

• support EIP1271 signatures

  kumaryash90

  committed
  3e3d176

  Copy full SHA for 3e3d176

• revert L-1

  kumaryash90

  committed
  fe2d23f

  Copy full SHA for fe2d23f

Commits on Apr 5, 2024

• The statement using ECDSA for bytes32 is not needed anymore

  kumaryash90

  committed
  19c0120

  Copy full SHA for 19c0120